[GRASS-SVN] r35831 - grass/trunk/gui/wxpython/gui_modules

[svn_grass at osgeo.org]

Author: martinl
Date: 2009-02-09 14:49:52 -0500 (Mon, 09 Feb 2009)
New Revision: 35831

Modified:
   grass/trunk/gui/wxpython/gui_modules/preferences.py
Log:
wxGUI: use gcmd.RunCommand instead grass.*_command()


Modified: grass/trunk/gui/wxpython/gui_modules/preferences.py
===================================================================
--- grass/trunk/gui/wxpython/gui_modules/preferences.py	2009-02-09 19:29:58 UTC (rev 35830)
+++ grass/trunk/gui/wxpython/gui_modules/preferences.py	2009-02-09 19:49:52 UTC (rev 35831)
@@ -23,8 +23,11 @@
 import sys
 import copy
 import stat
-if os.name in ('posix', 'mac'):
+try:
     import pwd
+    havePwd = True
+except ImportError:
+    havePwd = False
 
 import wx
 import wx.lib.filebrowsebutton as filebrowse
@@ -1787,34 +1790,36 @@
         """Load data into list"""
         self.InsertColumn(0, _('Mapset'))
         self.InsertColumn(1, _('Owner'))
-        self.InsertColumn(2, _('Group'))
+        ### self.InsertColumn(2, _('Group'))
         gisenv = grass.gisenv()
         locationPath = os.path.join(gisenv['GISDBASE'], gisenv['LOCATION_NAME'])
 
-        ret = grass.read_command('g.mapsets',
-                                 flags = 'l')
-        ret = ret.strip(' \n')
-
+        ret = gcmd.RunCommand('g.mapsets',
+                              flags = 'l',
+                              fs = '|',
+                              read = True)
+        ret = ret.rstrip('\n')
+        
         mapsets = []
         if ret:
-            mapsets = ret.split()
+            mapsets = ret.split('|')
         
         for mapset in mapsets:
             index = self.InsertStringItem(sys.maxint, mapset)
             mapsetPath = os.path.join(locationPath,
                                       mapset)
             stat_info = os.stat(mapsetPath)
-        if os.name in ('posix', 'mac'):
-            self.SetStringItem(index, 1, "%s" % pwd.getpwuid(stat_info.st_uid)[0])
-            # FIXME: get group name
-            self.SetStringItem(index, 2, "%-8s" % stat_info.st_gid) 
-        else:
-            # FIXME: no pwd under MS Windows (owner: 0, group: 0)
-            self.SetStringItem(index, 1, "%-8s" % stat_info.st_uid)
-            self.SetStringItem(index, 2, "%-8s" % stat_info.st_gid)
+            if havePwd:
+                self.SetStringItem(index, 1, "%s" % pwd.getpwuid(stat_info.st_uid)[0])
+                # FIXME: get group name
+                ### self.SetStringItem(index, 2, "%-8s" % stat_info.st_gid) 
+            else:
+                # FIXME: no pwd under MS Windows (owner: 0, group: 0)
+                self.SetStringItem(index, 1, "%-8s" % stat_info.st_uid)
+                ### self.SetStringItem(index, 2, "%-8s" % stat_info.st_gid)
                 
         self.SetColumnWidth(col=0, width=wx.LIST_AUTOSIZE)
-        self.SetColumnWidth(col=1, width=wx.LIST_AUTOSIZE)
+        ### self.SetColumnWidth(col=1, width=wx.LIST_AUTOSIZE)
         
     def OnCheckItem(self, index, flag):
         """Mapset checked/unchecked"""