[GRASS-dev] Re: [GRASS GIS] #842: wx location wizard: spincontrol busted for UTM zone

[GRASS GIS]

#842: wx location wizard: spincontrol busted for UTM zone
-----------------------+----------------------------------------------------
  Reporter:  hamish    |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect    |      Status:  new                      
  Priority:  critical  |   Milestone:  6.4.0                    
 Component:  wxGUI     |     Version:  svn-develbranch6         
Resolution:            |    Keywords:  location wizard, utm     
  Platform:  Linux     |         Cpu:  x86-64                   
-----------------------+----------------------------------------------------
Comment (by cmbarton):

 Replying to [comment:13 hamish]:
 > Replying to [comment:12 cmbarton]:
 > > OK. I just attached a new patch for GRASS 6.4:
 > > location_wizard.patch2. It's like the first one, patched against
 > > the svn from the root of your source tree. Hopefully, this will
 > > do the trick.
 >
 > no dice.
 >
 > I can spin it, it loops past 60 and back to 1, but if I touch the
 up/down controls at all or type in a number I get a pop up window that
 says "Something is missing!  You must enter a value for Projection Zone
 (1-60)". If I don't touch it at all I can go past and create a location
 for UTM zone 30.
 >
 > after I press ok on the error popup, this appears in the terminal:
 > {{{
 > Traceback (most recent call last):
 >   File "/usr/src/grass/svn/releasebranch_6_4/dist.x86_64-unknown-linux-
 gnu/etc/wxpython/gui_modules/location_wizard.py", line 769, in
 OnPageChange
 >     if int(self.pval[num]) > 60 or int(self.pval[num]) < 1:
 > ValueError: invalid literal for int() with base 10: ''
 > }}}
 >
 > If I put some debug messages directly before L769,
 > {{{
 >         num = self.utmzoneNum
 >         print "num=",num
 >       print "self.utmzoneNum=",self.utmzoneNum
 >       print "self.pval[num]=",self.pval[num]
 > }}}
 >
 > I get this on the terminal after pressing ok to the error popup message:
 >
 > {{{
 > num= 0
 > self.utmzoneNum= 0
 > self.pval[num]=
 > }}}
 > Hamish

 Drat!

 This is a page changing validator that someone else added since last I
 worked on the location wizard. Good idea, but it's wrong. I noticed but it
 hadn't been causing errors so I left it alone, wanting to mess with this
 as little as possible. I guess I have to fix it too.

 Michael

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/842#comment:15>
GRASS GIS <http://grass.osgeo.org>