[GRASS-dev] Re: [GRASS GIS] #1488: v.in.dxf wrapper not working

[GRASS GIS]

#1488: v.in.dxf wrapper not working
-------------------------+--------------------------------------------------
 Reporter:  cmbarton     |       Owner:  grass-dev@…              
     Type:  defect       |      Status:  new                      
 Priority:  normal       |   Milestone:  7.0.0                    
Component:  wxGUI        |     Version:  svn-trunk                
 Keywords:  v.in.dxf     |    Platform:  Unspecified              
      Cpu:  Unspecified  |  
-------------------------+--------------------------------------------------

Comment(by cmbarton):

 After a bit of testing, using 'ignore' works, but 'replace' creates
 further errors down the line

 {{{
 Traceback (most recent call last):
   File "/Applications/GRASS/GRASS-7.0.app/Contents/MacOS/etc
 /python/wx/lib/filebrowsebutton.py", line 140, in OnChanged

 self.changeCallback(evt)
   File "/Applications/GRASS/GRASS-7.0.app/Contents/MacOS/etc
 /gui/wxpython/gui_modules/gdialogs.py", line 1962, in
 OnSetDsn

 grassName = utils.GetValidLayerName(layerName)
   File "/Applications/GRASS/GRASS-7.0.app/Contents/MacOS/etc
 /gui/wxpython/gui_modules/utils.py", line 171, in
 GetValidLayerName

 retName = str(name).strip()
 UnicodeEncodeError
 :
 'ascii' codec can't encode character u'\ufffd' in position
 0: ordinal not in range(128)
 }}}

 Unless there is a compelling reason to do otherwise, I suggest using
 'ignore'. Thanks for the insight Glynn.

 Also, since the "string" module is imported, using "string" as a variable
 name in a method so as to produce a code line like

 string.decode(enc)

 seems potentially confusing to those reading the code at least, and
 potentially to the program. Maybe the variable should be changed to "str"
 or something along that line.

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/1488#comment:7>
GRASS GIS <http://grass.osgeo.org>