[GRASS-dev] [GRASS-SVN] r68180 - grass/trunk/lib/python/script

Ciao,

2016-03-29 8:29 GMT+02:00 <svn_grass@osgeo.org>:

Author: zarch
Date: 2016-03-28 23:29:14 -0700 (Mon, 28 Mar 2016)
New Revision: 68180

Modified:
   grass/trunk/lib/python/script/utils.py
Log:
script.utils: fix bug in decode/encode function

candidate for backport? Ma

Modified: grass/trunk/lib/python/script/utils.py

--- grass/trunk/lib/python/script/utils.py 2016-03-29 06:24:03 UTC (rev 68179)
+++ grass/trunk/lib/python/script/utils.py 2016-03-29 06:29:14 UTC (rev 68180)
@@ -156,10 +156,8 @@
     :param str string: the string to decode
     """
     enc = locale.getdefaultlocale()[1]
- if enc:
- if hasattr(string, 'decode'):
- return string.decode(enc)
-
+ if hasattr(string, 'decode'):
+ return string.decode(enc) if enc else string.decode()
     return string

@@ -169,10 +167,8 @@
     :param str string: the string to encode
     """
     enc = locale.getdefaultlocale()[1]
- if enc:
- if hasattr(string, 'encode'):
- return string.encode(enc)
-
+ if hasattr(string, 'encode'):
+ return string.encode(enc) if enc else string.encode()
     return string

@@ -376,5 +372,5 @@
             pathname = os.path.join(modulename, dirname) if dirname else modulename
             raise ImportError("Not able to find the path '%s' directory "
                               "(current dir '%s')." % (pathname, os.getcwd()))
-
+
         sys.path.insert(0, path)

_______________________________________________
grass-commit mailing list
grass-commit@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-commit

--
Martin Landa
http://geo.fsv.cvut.cz/gwiki/Landa
http://gismentors.cz/mentors/landa

Hi Martin,

On Tue, Mar 29, 2016 at 9:16 AM, Martin Landa <landa.martin@gmail.com> wrote:

candidate for backport?

Yes, I think this could be backported with no problems.
Current implementation is not working when for example the LC_ALL=C,

see tests in r68179.

All the best.

Pietro