wxGUI: fix auto-rendering (cmdAutoRender)

(merge https://trac.osgeo.org/grass/changeset/44075 from devbr6)


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

gui/wxpython/gui_modules/globalvar.py

@@ -179,6 +179,3 @@ else:
 
 """@Check version of wxPython, use agwStyle for 2.8.11+"""
 hasAgw = CheckWxVersion()
-
-"""@List of commands for auto-rendering"""
-cmdAutoRender = [ 'r.colors', 'i.landsat.rgb' ]

gui/wxpython/gui_modules/goutput.py

@@ -712,11 +712,23 @@ class GMConsole(wx.SplitterWindow):
         # set focus on prompt
         if self.parent.GetName() == "LayerManager":
             self.btn_abort.Enable(False)
-            if event.cmd[0] in globalvar.cmdAutoRender:
-                display = self.parent.GetLayerTree().GetMapDisplay()
-                if display and display.IsAutoRendered():
-                    display.GetWindow().UpdateMap(render = True)
-        
+            display = self.parent.GetLayerTree().GetMapDisplay()
+            if display and display.IsAutoRendered():
+                mapLayers = map(lambda x: x.GetName(),
+                                display.GetRender().GetListOfLayers(l_type = 'raster') +
+                                display.GetRender().GetListOfLayers(l_type = 'vector'))
+                
+                task = menuform.GUI().ParseCommand(event.cmd, show = None)
+                for p in task.get_options()['params']:
+                    if p.get('prompt', '') not in ('raster', 'vector'):
+                        continue
+                    mapName = p.get('value', '')
+                    if '@' not in mapName:
+                        mapName = mapName + '@' + grass.gisenv()['MAPSET']
+                    if mapName in mapLayers:
+                        display.GetWindow().UpdateMap(render = True)
+                        return
+            
         else:
             # updated command dialog
             dialog = self.parent.parent

gui/wxpython/gui_modules/menuform.py

@@ -890,18 +890,33 @@ class mainFrame(wx.Frame):
 
         @param returncode command's return code (0 for success)
         """
-        if self.parent and self.parent.GetName() not in ('LayerTree', 'LayerManager') or \
-                returncode != 0:
+        if not self.parent or returncode != 0:
+            return
+        if self.parent.GetName() not in ('LayerTree', 'LayerManager'):
             return
         
-        if cmd[0] in globalvar.cmdAutoRender:
-            if self.parent.GetName() == 'LayerTree':
-                display = self.parent.GetMapDisplay()
-            else: # Layer Manager
-                display = self.parent.GetLayerTree().GetMapDisplay()
+        if self.parent.GetName() == 'LayerTree':
+            display = self.parent.GetMapDisplay()
+        else: # Layer Manager
+            display = self.parent.GetLayerTree().GetMapDisplay()
             
-            if display and display.IsAutoRendered():
+        if not display or not display.IsAutoRendered():
+            return
+        
+        mapLayers = map(lambda x: x.GetName(),
+                        display.GetRender().GetListOfLayers(l_type = 'raster') +
+                        display.GetRender().GetListOfLayers(l_type = 'vector'))
+        
+        task = GUI().ParseCommand(cmd, show = None)
+        for p in task.get_options()['params']:
+            if p.get('prompt', '') not in ('raster', 'vector'):
+                continue
+            mapName = p.get('value', '')
+            if '@' not in mapName:
+                mapName = mapName + '@' + grass.gisenv()['MAPSET']
+            if mapName in mapLayers:
                 display.GetWindow().UpdateMap(render = True)
+                return
         
     def OnOK(self, event):
         """!OK button pressed"""

gui/wxpython/gui_modules/render.py

@@ -236,8 +236,11 @@ class Layer(object):
     def GetName(self, fullyQualified = True):
         """!Get map layer name
 
-        @param fullyQualified if True return 'name@mapset' otherwise
-        ('name', 'mapset')
+        @param fullyQualified True to return fully qualified name as a
+        string 'name@mapset' otherwise directory { 'name', 'mapset' }
+        is returned
+
+        @return string / directory
         """
         if fullyQualified:
             return self.name