← Back to team overview

credativ team mailing list archive

lp:~savoirfairelinux-openerp/openupgrade-server/unicode-exception into lp:openupgrade-server

 

Sandy Carter (http://www.savoirfairelinux.com) has proposed merging lp:~savoirfairelinux-openerp/openupgrade-server/unicode-exception into lp:openupgrade-server.

Requested reviews:
  OpenUpgrade Committers (openupgrade-committers)

For more details, see:
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-server/unicode-exception/+merge/191995

Fixing exception that is raised when logging other exceptions on french systems.

Error message for e.message = "DÉTAIL : Key (name)=(document_page_environment_manual) already exists.":

      File "openerp/openerp/openupgrade/openupgrade.py", line 464, in wrapped_function
        (module, filename, e))
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 60: ordinal not in range(128)

-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-server/unicode-exception/+merge/191995
Your team OpenUpgrade Committers is requested to review the proposed merge of lp:~savoirfairelinux-openerp/openupgrade-server/unicode-exception into lp:openupgrade-server.
=== modified file 'openerp/openupgrade/openupgrade.py'
--- openerp/openupgrade/openupgrade.py	2013-10-15 14:53:26 +0000
+++ openerp/openupgrade/openupgrade.py	2013-10-21 14:34:14 +0000
@@ -461,7 +461,7 @@
             except Exception, e:
                 logger.error(
                     "%s: error in migration script %s: %s" % 
-                    (module, filename, e))
+                    (module, filename, e.message.decode('utf8')))
                 logger.exception(e)
                 raise
         return wrapped_function


Follow ups