Merge lp:~savoirfairelinux-openerp/openobject-server/translate_warnings_1297525-trunk into lp:openobject-server

Status: Needs review
Proposed branch: lp:~savoirfairelinux-openerp/openobject-server/translate_warnings_1297525-trunk
Merge into: lp:openobject-server
Diff against target: 25 lines (+2/-2)
2 files modified
openerp/osv/orm.py (+1/-1)
openerp/tools/translate.py (+1/-1)
To merge this branch: bzr merge lp:~savoirfairelinux-openerp/openobject-server/translate_warnings_1297525-trunk
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+212738@code.launchpad.net

Description of the change

When searching a frame for context, the _() call looks for self.localcontext but misses self.context.
The integration of this patch allows for web warnings (such as except_orm) to be translated by _()

To post a comment you must log in.
5151. By El Hadji Dem (http://www.savoirfairelinux.com)

[FIX] Fixed instance of ValidateError not being translated

Unmerged revisions

5151. By El Hadji Dem (http://www.savoirfairelinux.com)

[FIX] Fixed instance of ValidateError not being translated

5150. By Sandy Carter (http://www.savoirfairelinux.com)

Also look for context in self

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py 2014-04-01 10:08:19 +0000
+++ openerp/osv/orm.py 2014-04-06 00:17:16 +0000
@@ -1576,7 +1576,7 @@
1576 )1576 )
1577 self._invalids.update(fields)1577 self._invalids.update(fields)
1578 if error_msgs:1578 if error_msgs:
1579 raise except_orm('ValidateError', '\n'.join(error_msgs))1579 raise except_orm(_('ValidateError'), '\n'.join(error_msgs))
1580 else:1580 else:
1581 self._invalids.clear()1581 self._invalids.clear()
15821582
15831583
=== modified file 'openerp/tools/translate.py'
--- openerp/tools/translate.py 2014-03-20 09:01:05 +0000
+++ openerp/tools/translate.py 2014-04-06 00:17:16 +0000
@@ -196,7 +196,7 @@
196 lang = ctx.get('lang')196 lang = ctx.get('lang')
197 s = frame.f_locals.get('self', {})197 s = frame.f_locals.get('self', {})
198 if not lang:198 if not lang:
199 c = getattr(s, 'localcontext', None)199 c = getattr(s, 'localcontext', None) or getattr(s, 'context', None)
200 if c:200 if c:
201 lang = c.get('lang')201 lang = c.get('lang')
202 if not lang:202 if not lang: