credativ team mailing list archive

["Sandy Carter \(http://www.savoirfairelinux.com\)" <sandy.carter@xxxxxxxxxxxxxxxxxxxx>]

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

Requested reviews:
  OpenUpgrade Committers (openupgrade-committers)

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

Addendum to [1] where some exceptions such as AttributeError' message may not be a string.

[1] https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-server/OperationalError/+merge/192740
-- 
https://code.launchpad.net/~savoirfairelinux-openerp/openupgrade-server/AttributeErrorFix/+merge/193141
Your team OpenUpgrade Committers is requested to review the proposed merge of lp:~savoirfairelinux-openerp/openupgrade-server/AttributeErrorFix into lp:openupgrade-server.

=== modified file 'openerp/openupgrade/openupgrade.py'
--- openerp/openupgrade/openupgrade.py	2013-10-21 14:28:15 +0000
+++ openerp/openupgrade/openupgrade.py	2013-10-29 21:23:04 +0000
@@ -461,7 +461,7 @@
             except Exception, e:
                 logger.error(
                     "%s: error in migration script %s: %s" % 
-                    (module, filename, e.message.decode('utf8')))
+                    (module, filename, str(e).decode('utf8')))
                 logger.exception(e)
                 raise
         return wrapped_function