Merge lp:~openerp-dev/openerp-web/7.0-opw-605743-msh into lp:openerp-web/7.0

Proposed by Mohammed Shekha(Open ERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-605743-msh
Merge into: lp:openerp-web/7.0
Diff against target: 34 lines (+6/-5)
1 file modified
addons/web/session.py (+6/-5)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-605743-msh
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Martin Trigaux (OpenERP) Pending
Review via email: mp+213025@code.launchpad.net

Description of the change

Hello,

Please go through description of server branch for details.
Branch: lp:~openerp-dev/openobject-server/7.0-opw-605743-msh

Thanks.

To post a comment you must log in.

Unmerged revisions

4166. By Mohammed Shekha(Open ERP)

[FIX]Web: Fixed the issue of translation of xmlrpclib.Fault code, warning is not translated, also fixed the same thing in server.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web/session.py'
--- addons/web/session.py 2013-05-14 10:37:36 +0000
+++ addons/web/session.py 2014-03-27 11:29:06 +0000
@@ -6,6 +6,7 @@
6import traceback6import traceback
7import sys7import sys
8import xmlrpclib8import xmlrpclib
9from openerp.tools.translate import _
910
10import openerp11import openerp
1112
@@ -84,17 +85,17 @@
84 self.jsonp_requests = {} # FIXME use a LRU85 self.jsonp_requests = {} # FIXME use a LRU
8586
86 def send(self, service_name, method, *args):87 def send(self, service_name, method, *args):
87 code_string = u"warning -- %s\n\n%s"88 code_string = u"%s -- %s\n\n%s"
88 try:89 try:
89 return openerp.netsvc.dispatch_rpc(service_name, method, args)90 return openerp.netsvc.dispatch_rpc(service_name, method, args)
90 except openerp.osv.osv.except_osv, e:91 except openerp.osv.osv.except_osv, e:
91 raise xmlrpclib.Fault(code_string % (e.name, e.value), '')92 raise xmlrpclib.Fault(code_string % (_("warning"), e.name, e.value), '')
92 except openerp.exceptions.Warning, e:93 except openerp.exceptions.Warning, e:
93 raise xmlrpclib.Fault(code_string % ("Warning", e), '')94 raise xmlrpclib.Fault(code_string % (_("warning"), _("Warning"), e), '')
94 except openerp.exceptions.AccessError, e:95 except openerp.exceptions.AccessError, e:
95 raise xmlrpclib.Fault(code_string % ("AccessError", e), '')96 raise xmlrpclib.Fault(code_string % (_("warning"), _("AccessError"), e), '')
96 except openerp.exceptions.AccessDenied, e:97 except openerp.exceptions.AccessDenied, e:
97 raise xmlrpclib.Fault('AccessDenied', openerp.tools.ustr(e))98 raise xmlrpclib.Fault(_('AccessDenied'), openerp.tools.ustr(e))
98 except openerp.exceptions.DeferredException, e:99 except openerp.exceptions.DeferredException, e:
99 formatted_info = "".join(traceback.format_exception(*e.traceback))100 formatted_info = "".join(traceback.format_exception(*e.traceback))
100 raise xmlrpclib.Fault(openerp.tools.ustr(e), formatted_info)101 raise xmlrpclib.Fault(openerp.tools.ustr(e), formatted_info)