wxGUI/vdigit: don't ask to save changes when there have been none (trac https://trac.osgeo.org/grass/ticket/210)

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

gui/wxpython/gui_modules/toolbars.py

@@ -856,14 +856,15 @@ class VDigitToolbar(AbstractToolbar):
             # save changes (only for vdigit)
             if UserSettings.Get(group='advanced', key='digitInterface', subkey='type') == 'vdigit':
                 if UserSettings.Get(group='vdigit', key='saveOnExit', subkey='enabled') is False:
-                    dlg = wx.MessageDialog(parent=self.parent, message=_("Do you want to save changes "
-                                                                         "in vector map <%s>?") % layerSelected.name,
-                                           caption=_("Save changes?"),
-                                           style=wx.YES_NO | wx.YES_DEFAULT | wx.ICON_QUESTION)
-                    if dlg.ShowModal() == wx.ID_NO:
-                        # revert changes
-                        self.parent.digit.Undo(0)
-                    dlg.Destroy()
+                    if self.parent.digit.GetUndoLevel() > 0:
+                        dlg = wx.MessageDialog(parent=self.parent, message=_("Do you want to save changes "
+                                                                             "in vector map <%s>?") % layerSelected.name,
+                                               caption=_("Save changes?"),
+                                               style=wx.YES_NO | wx.YES_DEFAULT | wx.ICON_QUESTION)
+                        if dlg.ShowModal() == wx.ID_NO:
+                            # revert changes
+                            self.parent.digit.Undo(0)
+                        dlg.Destroy()
 
             self.parent.digit.SetMapName(None) # -> close map
 

gui/wxpython/gui_modules/vdigit.py

@@ -758,7 +758,7 @@ class VEdit(AbstractDigit):
 
     def Undo(self, level=-1):
         """Undo not implemented here"""
-        wx.MessageBox(parent=self.mapWindow, message=_("Undondo is not implemented in vedit component. "
+        wx.MessageBox(parent=self.mapWindow, message=_("Undo is not implemented in vedit component. "
                                                     "Use vdigit instead."),
                       caption=_("Message"), style=wx.ID_OK | wx.ICON_INFORMATION | wx.CENTRE)
 
@@ -1179,6 +1179,10 @@ class VDigit(AbstractDigit):
         
         if ret < 0: # disable undo tool
             self.toolbar.EnableUndo(False)
+
+    def GetUndoLevel(self):
+        """Get undo level (number of active changesets)"""
+        return self.digit.GetUndoLevel()
         
     def __getSnapThreshold(self):
         """Get snap mode and threshold value

gui/wxpython/vdigit/digit.h

@@ -80,6 +80,7 @@ public:
     std::vector<int> GetLayers();
 
     int Undo(int);
+    int GetUndoLevel();
 };
 
 #endif /* __DIGIT_H__ */

gui/wxpython/vdigit/undo.cpp

@@ -235,3 +235,13 @@ int Digit::RemoveActionFromChangeset(int changeset, Digit::action_type type, int
 
     return action.size();
 }
+
+/**
+   \brief Get undo level (number of active changesets)
+
+   \return number
+*/
+int Digit::GetUndoLevel()
+{
+    return changesetCurrent - changesetDead;
+}