attempt to fix https://trac.osgeo.org/grass/ticket/1300 (WxGUI measure tool gives wrong results)

wxGUI: fix formatting distance, multi-line output
	(merge https://trac.osgeo.org/grass/changeset/45883 & https://trac.osgeo.org/grass/changeset/45884 from devbr6)


git-svn-id: https://svn.osgeo.org/grass/grass/trunk@45885 15284696-431f-4ddb-bdfa-cd5b030d7da7

gui/wxpython/gui_modules/mapdisp.py

@@ -1649,32 +1649,37 @@ class MapFrame(wx.Frame):
         """
         if self._layerManager.notebook.GetSelection() != 1:
             self._layerManager.notebook.SetSelection(1)
-
+        
         dist, (north, east) = self.MapWindow.Distance(beginpt, endpt)
-
+        
         dist = round(dist, 3)
         d, dunits = self.FormatDist(dist)
-
+        
         self.totaldist += dist
         td, tdunits = self.FormatDist(self.totaldist)
-
+        
         strdist = str(d)
         strtotdist = str(td)
-
+        
         if self.Map.projinfo['proj'] == 'xy' or 'degree' not in self.Map.projinfo['unit']:
             angle = int(math.degrees(math.atan2(north,east)) + 0.5)
             angle = 180 - angle
             if angle < 0:
                 angle = 360+angle
-
-            mstring = 'segment = %s %s\ttotal distance = %s %s\tbearing = %d deg' \
-                % (strdist,dunits,strtotdist,tdunits,angle)
+            
+            mstring = '%s = %s %s\n%s = %s %s\n%s = %d %s\n%s' \
+                % (_('segment'), strdist, dunits,
+                   _('total distance'), strtotdist, tdunits,
+                   _('bearing'), angle, _('deg'),
+                   '-' * 60)
         else:
-            mstring = 'segment = %s %s\ttotal distance = %s %s' \
-                % (strdist,dunits,strtotdist,tdunits)
-
+            mstring = '%s = %s %s\n%s = %s %s\n%s' \
+                % (_('segment'), strdist, dunits,
+                   _('total distance'), strtotdist, tdunits,
+                   '-' * 60)
+        
         self._layerManager.goutput.WriteLog(mstring)
-
+        
         return dist
 
     def Profile(self, event):
@@ -1697,13 +1702,14 @@ class MapFrame(wx.Frame):
         """!Format length numbers and units in a nice way,
         as a function of length. From code by Hamish Bowman
         Grass Development Team 2006"""
-
+        
         mapunits = self.Map.projinfo['units']
-        if mapunits == 'metres': mapunits = 'meters'
+        if mapunits == 'metres':
+            mapunits = 'meters'
         outunits = mapunits
         dist = float(dist)
         divisor = 1.0
-
+        
         # figure out which units to use
         if mapunits == 'meters':
             if dist > 2500.0:
@@ -1726,7 +1732,7 @@ class MapFrame(wx.Frame):
                 divisor = (1/60.0)
             else:
                 outunits = 'deg'
-
+        
         # format numbers in a nice way
         if (dist/divisor) >= 2500.0:
             outdist = round(dist/divisor)
@@ -1736,10 +1742,9 @@ class MapFrame(wx.Frame):
             outdist = round(dist/divisor,int(math.ceil(3-math.log10(dist/divisor))))
         else:
             outdist = float(dist/divisor)
-
+        
         return (outdist, outunits)
-
-
+    
     def Histogram(self, event):
         """!Init histogram display canvas and tools
         """

gui/wxpython/gui_modules/mapdisp_window.py

@@ -1743,7 +1743,7 @@ class BufferedWindow(MapWindow, wx.Window):
         if tmpreg:
             os.environ["GRASS_REGION"] = tmpreg
         
-    def Distance(self, beginpt, endpt, screen=True):
+    def Distance(self, beginpt, endpt, screen = True):
         """!Calculete distance
         
         LL-locations not supported
@@ -1754,13 +1754,14 @@ class BufferedWindow(MapWindow, wx.Window):
         @param endpt second point
         @param screen True for screen coordinates otherwise EN
         """
-        x1, y1 = beginpt
-        x2, y2 = endpt
         if screen:
-            dEast  = (x2 - x1) * self.Map.region["ewres"]
-            dNorth = (y2 - y1) * self.Map.region["nsres"]
+            e1, n1 = self.Pixel2Cell(beginpt)
+            e2, n2 = self.Pixel2Cell(endpt)
         else:
-            dEast  = (x2 - x1)
-            dNorth = (y2 - y1)
+            e1, n1 = beginpt
+            e2, n2 = endpt
+            
+        dEast  = (e2 - e1)
+        dNorth = (n2 - n1)
         
-        return (math.sqrt(math.pow((dEast),2) + math.pow((dNorth),2)), (dEast, dNorth))
+        return (math.sqrt(math.pow((dEast), 2) + math.pow((dNorth), 2)), (dEast, dNorth))

gui/wxpython/gui_modules/render.py

@@ -428,8 +428,10 @@ class Map(object):
         
         for line in ret.splitlines():
             if ':' in line:
-                key, val = line.split(':')
-                projinfo[key.strip()] = val.strip()
+                key, val = map(lambda x: x.strip(), line.split(':'))
+                if key in ['units']:
+                    val = val.lower()
+                projinfo[key] = val
             elif "XY location (unprojected)" in line:
                 projinfo['proj'] = 'xy'
                 projinfo['units'] = ''