GUI: Included Mapvalidator in forms.py to check valid raster output names

gui/wxpython/gui_core/forms.py

@@ -101,6 +101,7 @@
     FormListbook,
     FormNotebook,
     PlacementValidator,
+    MapNameValidator,
 )
 from core.giface import Notification, StandaloneGrassInterface
 from gui_core.widgets import LayersList
@@ -1509,6 +1510,9 @@ def __init__(self, parent, giface, task, id=wx.ID_ANY, frame=None, *args, **kwar
                         selection = gselect.Select(
                             parent=which_panel,
                             id=wx.ID_ANY,
+                            validator=MapNameValidator()
+                            if p.get("age") == "new"
+                            else wx.DefaultValidator,
                             size=globalvar.DIALOG_GSELECT_SIZE,
                             type=elem,
                             multiple=multiple,
@@ -3350,7 +3354,8 @@ def OnInit(self):
                 "gisprompt": False,
                 "multiple": "yes",
                 # values must be an array of strings
-                "values": utils.str2rgb.keys() + list(map(str, utils.str2rgb.values())),
+                "values": list(utils.str2rgb.keys())
+                + list(map(str, utils.str2rgb.values())),
                 "key_desc": ["value"],
                 "values_desc": [],
             },

gui/wxpython/gui_core/widgets.py

@@ -17,6 +17,7 @@
  - widgets::EmailValidator
  - widgets::TimeISOValidator
  - widgets::MapValidator
+ - widgets::MapNameValidator
  - widgets::NTCValidator
  - widgets::SimpleValidator
  - widgets::GenericValidator
@@ -889,11 +890,36 @@
         GenericValidator.__init__(self, grass.legal_name, _mapNameValidationFailed)
 
 
+class MapNameValidator(BaseValidator):
+    """Validator for map name input
+
+    See G_legal_filename()
+    """
+
+    def __init__(self):
+        BaseValidator.__init__(self)
+
+    def _validate(self, win):
+        """Validate input"""
+        text = win.GetValue()
+        if text:
+            if not grass.legal_name(text):
+                self._notvalid()
+                return False
+
+        self._valid()
+        return True
+
+    def Clone(self):
+        """Clone validator"""
+        return MapNameValidator()
+
 class GenericMultiValidator(Validator):
+
     """This validator checks conditions and calls callbacks
     in case the condition is not fulfilled.
     """
 
     def __init__(self, checks):
         """Standard constructor.