<div dir="ltr"><div><div><div>Hi,<br><br></div>the original motivation for the commit was actually terminology. When talking about the raster/vector/3D raster file on disk, we should use "map" rather then "layer" to be consistent with the rest of GRASS. Let me know if you think otherwise.<br>

<br></div><div>Vaclav<br></div><div><br></div>PS: You were silent for some time so I took the liberty to edit the code.<br></div>PPS: I've used the data catalog for the first time. It is pretty cool!<br></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Thu, Jul 10, 2014 at 3:03 PM,  <span dir="ltr"><<a href="mailto:svn_grass@osgeo.org" target="_blank">svn_grass@osgeo.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Author: wenzeslaus<br>
Date: 2014-07-10 12:03:00 -0700 (Thu, 10 Jul 2014)<br>
New Revision: 61236<br>
<br>
Modified:<br>
   grass/trunk/gui/wxpython/lmgr/datacatalog.py<br>
Log:<br>
wxGUI/datacatalog: use GRASS standard message format with <>, use string formatting instead of plus (to enable good translations), use keyword arguments when parameter names are unclear, prefer shorter lines<br>
<br>
Modified: grass/trunk/gui/wxpython/lmgr/datacatalog.py<br>
===================================================================<br>
--- grass/trunk/gui/wxpython/lmgr/datacatalog.py        2014-07-10 18:51:58 UTC (rev 61235)<br>
+++ grass/trunk/gui/wxpython/lmgr/datacatalog.py        2014-07-10 19:03:00 UTC (rev 61236)<br>
@@ -402,7 +402,9 @@<br>
             string = self.GetItemText(self.selected_layer)<br>
             self.ChangeEnvironment(self.GetItemText(self.selected_location), self.GetItemText(self.selected_mapset))<br>
             removed = 0<br>
-            if (self._confirmDialog(_('Do you really want to delete layer') +string+'?', _('Delete map')) == wx.ID_YES):<br>
+            # TODO: rewrite this that it will tell map type in the dialog<br>
+            if (self._confirmDialog(question=_('Do you really want to delete map <{m}>?').format(m=string),<br>
+                                    title=_('Delete map')) == wx.ID_YES):<br>
                 label = _("Deleting") + " " + string + " ..."<br>
                 self.showNotification.emit(message=label)<br>
                 if (self.GetItemText(self.selected_type)=='vect'):<br>
<br>
_______________________________________________<br>
grass-commit mailing list<br>
<a href="mailto:grass-commit@lists.osgeo.org">grass-commit@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/grass-commit" target="_blank">http://lists.osgeo.org/mailman/listinfo/grass-commit</a><br>
</blockquote></div><br></div>