wxGUI: fix trac https://trac.osgeo.org/grass/ticket/257, can't really cancel adding layer (merge from devbr6, https://trac.osgeo.org/grass/changeset/32785)

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

gui/wxpython/gui_modules/menuform.py

@@ -570,7 +570,7 @@ class mainFrame(wx.Frame):
         self.get_dcmd = get_dcmd
         self.layer = layer
         self.task = task_description
-        self.parent = parent
+        self.parent = parent # LayerTree | None
 
         # module name + keywords
         title = self.task.name
@@ -842,7 +842,11 @@ class mainFrame(wx.Frame):
         """Cancel button pressed"""
         self.MakeModal(False)
         if self.get_dcmd:
-            self.Hide()
+            if len(self.parent.GetPyData(self.layer)[0]['cmd']) < 1:
+                self.parent.Delete(self.layer)
+                self.Destroy()
+            else:
+                self.Hide()
         else:
             self.Destroy()
 
@@ -1464,7 +1468,10 @@ class cmdPanel(wx.Panel):
         try:
             cmd = self.task.getCmd( ignoreErrors=ignoreErrors )
         except ValueError, err:
-            dlg = wx.MessageDialog(self, str(err), _("Error"), wx.OK | wx.ICON_ERROR)
+            dlg = wx.MessageDialog(parent=self,
+                                   message=str(err),
+                                   caption=_("Error in %s") % self.task.name,
+                                   style=wx.OK | wx.ICON_ERROR | wx.CENTRE)
             dlg.ShowModal()
             dlg.Destroy()
             cmd = None

gui/wxpython/gui_modules/wxgui_utils.py

@@ -652,7 +652,7 @@ class LayerTree(treemixin.DragAndDrop, CT.CustomTreeCtrl):
                                     'ctrl' : ctrlId,
                                     'maplayer' : None,
                                     'nviz' : lnviz,
-                                    'prowin' : None}, 
+                                    'propwin' : None}, 
                                    None))
 
             maplayer = self.Map.AddLayer(type=ltype, command=self.GetPyData(layer)[0]['cmd'], name=name,
@@ -673,7 +673,7 @@ class LayerTree(treemixin.DragAndDrop, CT.CustomTreeCtrl):
                                     'type' : ltype,
                                     'ctrl' : None,
                                     'maplayer' : None,
-                                    'prowin' : None}, 
+                                    'propwin' : None}, 
                                    None))
 
         # use predefined layer name if given
@@ -1128,7 +1128,6 @@ class LayerTree(treemixin.DragAndDrop, CT.CustomTreeCtrl):
 
     def GetOptData(self, dcmd, layer, params, propwin):
         """Process layer data"""
-
         # set layer text to map name
         if dcmd:
             mapLayer = self.GetPyData(layer)[0]['maplayer']
@@ -1144,7 +1143,7 @@ class LayerTree(treemixin.DragAndDrop, CT.CustomTreeCtrl):
                 return
             
             self.SetItemText(layer, mapname + ' (opacity: ' + str(opacity) + '%)')
-
+        
         # update layer data
         if params:
             self.SetPyData(layer, (self.GetPyData(layer)[0], params))