Olivier Laurent (Open ERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-575086-olt into lp:openerp-web/6.1.

Requested reviews:
  OpenERP Core Team (openerp)

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-575086-olt/+merge/109371


In:
  fault = xmlrpclib.Fault('warning -- ' + e.name + '\n\n' + e.value, '')
if e.value is not a string an exception is raised (TypeError: coercing to 
Unicode: need string or buffer, tuple found), so the original exception message 
is not shown. You don't know what went wrong, you just know that an problem 
occurred in the exception handling mechanism.

This case sometimes happens in the reporting system.

-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-575086-olt/+merge/109371
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-575086-olt.
=== modified file 'addons/web/common/http.py'
--- addons/web/common/http.py	2012-02-10 16:43:09 +0000
+++ addons/web/common/http.py	2012-06-08 14:01:21 +0000
@@ -593,7 +593,7 @@
         except Exception,e:
         # TODO change the except to raise LibException instead of their emulated xmlrpc fault
             if isinstance(e, openerp.osv.osv.except_osv):
-                fault = xmlrpclib.Fault('warning -- ' + e.name + '\n\n' + e.value, '')
+                fault = xmlrpclib.Fault('warning -- ' + str(e.name) + '\n\n' + str(e.value), '')
             elif isinstance(e, openerp.exceptions.Warning):
                 fault = xmlrpclib.Fault('warning -- Warning\n\n' + str(e), '')
             elif isinstance(e, openerp.exceptions.AccessError):

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to