OpenUpgrade Server

Merge lp:~savoirfairelinux-openerp/openupgrade-server/AttributeErrorFix into lp:openupgrade-server

  1. AttributeErrorFix
  2. Merge into 7.0
Status: Merged
Merged at revision: 4639
Proposed branch: lp:~savoirfairelinux-openerp/openupgrade-server/AttributeErrorFix
Merge into: lp:openupgrade-server
Diff against target: 12 lines (+1/-1)
1 file modified
openerp/openupgrade/openupgrade.py (+1/-1)
To merge this branch: bzr merge lp:~savoirfairelinux-openerp/openupgrade-server/AttributeErrorFix
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) Approve
Review via email: mp+193141@code.launchpad.net

Description of the change

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

To post a comment you must log in.
Revision history for this message
Sylvain LE GAL (GRAP) (sylvain-legal) wrote :

LGTM.

review: Approve
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openerp/openupgrade/openupgrade.py'
2--- openerp/openupgrade/openupgrade.py 2013-10-21 14:28:15 +0000
3+++ openerp/openupgrade/openupgrade.py 2013-10-29 21:23:04 +0000
4@@ -461,7 +461,7 @@
5 except Exception, e:
6 logger.error(
7 "%s: error in migration script %s: %s" %
8- (module, filename, e.message.decode('utf8')))
9+ (module, filename, str(e).decode('utf8')))
10 logger.exception(e)
11 raise
12 return wrapped_function

Subscribers

People subscribed via source and target branches