wxGUI: small code cleanup

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

gui/wxpython/lmgr/frame.py

@@ -1072,7 +1072,7 @@ class GMFrame(wx.Frame):
         @return False on error
         """
         # dtd
-        dtdFilename = os.path.join(globalvar.ETCWXDIR, "xml", "grass-gxw.dtd")
+        # dtdFilename = os.path.join(globalvar.ETCWXDIR, "xml", "grass-gxw.dtd")
         
         # parse workspace file
         try:
@@ -1437,17 +1437,7 @@ class GMFrame(wx.Frame):
         win.Show()
         win.Refresh()
         win.Update()
-        
-    def OnProfile(self, event):
-        """!Launch profile tool
-        """
-        win = profile.ProfileFrame(parent = self)
-        
-        win.CentreOnParent()
-        win.Show()
-        win.Refresh()
-        win.Update()
-        
+
     def OnMapCalculator(self, event, cmd = ''):
         """!Init map calculator for interactive creation of mapcalc statements
         """

gui/wxpython/lmgr/layertree.py

@@ -265,8 +265,10 @@ class LayerTree(treemixin.DragAndDrop, CT.CustomTreeCtrl):
 
     def GetSelectedLayers(self, checkedOnly = False):
         """!Get selected layers as a list.
+
+        @todo somewhere we have checkedOnly default True and elsewhere False
         """
-        return self.GetSelectedLayer(multi = True, checkedOnly = False)
+        return self.GetSelectedLayer(multi = True, checkedOnly = checkedOnly)
 
     def GetSelectedLayer(self, multi = False, checkedOnly = False):
         """!Get selected layer from layer tree.

gui/wxpython/lmgr/menudata.py

@@ -24,7 +24,6 @@ from core.menudata  import MenuData
 class LayerManagerMenuData(MenuData):
     def __init__(self, filename = None):
         if not filename:
-            gisbase = os.getenv('GISBASE')
             global etcwxdir
             filename = os.path.join(ETCWXDIR, 'xml', 'menudata.xml')
         

gui/wxpython/mapdisp/main.py

@@ -50,7 +50,7 @@ monSize = list(globalvar.MAP_WINDOW_SIZE)
 
 
 class DMonMap(Map):
-    def __init__(self, gisrc = None, cmdfile = None, mapfile = None, envfile = None, monitor = None):
+    def __init__(self, cmdfile=None, mapfile=None, envfile=None, monitor=None):
         """!Map composition (stack of map layers and overlays)
 
         @param cmdline full path to the cmd file (defined by d.mon)
@@ -282,6 +282,7 @@ class LayerList(object):
 
 
 class DMonGrassInterface(StandaloneGrassInterface):
+    """!@implements GrassInterface"""
     def __init__(self, mapframe):
         StandaloneGrassInterface.__init__(self)
         self._mapframe = mapframe

gui/wxpython/mapdisp/overlays.py

@@ -25,6 +25,9 @@ class OverlayController(object):
         self._pdcType = 'image'
         self._propwin = None
         self._defaultAt = ''
+        self._cmd = None  # to be set by user
+        self._name = None  # to be defined by subclass
+        self._id = None  # to be defined by subclass
 
     def SetCmd(self, cmd):
         hasAt = False
@@ -92,7 +95,6 @@ class OverlayController(object):
         else:
             self.Hide()
 
-
     def Hide(self):
         if self._overlay:
             self._overlay.SetActive(False)

gui/wxpython/mapdisp/statusbar.py

@@ -439,8 +439,9 @@ class SbShowRegion(SbItem):
             for mapWindow in self.mapFrame.GetWindows():
                 mapWindow.regionCoords = []
         elif hasattr(self.mapFrame.GetWindow(), 'regionCoords'):
+            # TODO: this maybe never happends
             for mapWindow in self.mapFrame.GetWindows():
-                mapWindow.regionCoords
+                mapWindow.regionCoords = []
         
 class SbAlignExtent(SbItem):
     """!Checkbox to select zoom behavior.