[6.1,7.0,trunk] Context is not propagated when writing translation on 'source' fields
Bug #1247158 reported by
Guewen Baconnier @ Camptocamp
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Odoo Server (MOVED TO GITHUB) |
Fix Released
|
Low
|
OpenERP Publisher's Warranty Team | |||
OpenERP Community Backports (Server) | Status tracked in 7.0 | |||||
6.1 |
Fix Released
|
Undecided
|
Unassigned | |||
7.0 |
Fix Released
|
Medium
|
OpenERP Community Backports |
Bug Description
In openerp.
When we do a write on a record for a translation and a translation we are writing is empty in the 'source' translation, the ORM will write that value on the record (for en_US):
http://
Precisely, the write is here: http://
This call to write() isn't propagating the context, meaning that any special key in the context will be lost in the calls chain.
The context needs to be propagated, with the 'lang' key forced to the default language, that is: en_US.
I'll propose a merge.
Related branches
lp:~camptocamp/openobject-server/7.0-propagate-context-1247158
- Martin Trigaux (OpenERP) (community): Approve
-
Diff: 12 lines (+3/-1)1 file modifiedopenerp/osv/orm.py (+3/-1)
Superseded
for merging
into
lp:openobject-server
- OpenERP Core Team: Pending requested
-
Diff: 52 lines (+15/-2) (has conflicts)2 files modifiedopenerp/addons/base/i18n/ja.po (+12/-1)
openerp/osv/orm.py (+3/-1)
lp:~camptocamp/ocb-server/7.0-propagate-context-1247158
- Holger Brunn (Therp): Approve (code review)
- Christophe CHAUVET: Approve (code review, no test)
- Pedro Manuel Baeza: Approve (code review, no test)
-
Diff: 12 lines (+3/-1)1 file modifiedopenerp/osv/orm.py (+3/-1)
lp:~openerp-dev/openobject-server/7.0-opw-600285-msh
- Naresh(OpenERP): Pending requested
- Martin Trigaux (OpenERP): Pending requested
-
Diff: 13 lines (+3/-1)1 file modifiedopenerp/osv/orm.py (+3/-1)
lp:~yann-papouin/ocb-server/6.1-bug-1247158-propagate-context
- Stefan Rijnhart (Opener): Approve
- Paulius Sladkevičius @ hbee (community): Approve (code review)
- Holger Brunn (Therp): Approve (code review)
-
Diff: 11 lines (+2/-1)1 file modifiedopenerp/osv/orm.py (+2/-1)
Changed in openobject-server: | |
assignee: | nobody → OpenERP Publisher's Warranty Team (openerp-opw) |
tags: | added: maintenance |
Changed in openobject-server: | |
status: | Confirmed → Fix Committed |
Changed in openobject-server: | |
milestone: | none → 7.0 |
To post a comment you must log in.
Hello Guewen,
I would prefer the 'lang' key to be set at False instead of en_US. This would have the same result but using less openerp magic (en_US being the default language in openerp).
Is that ok with you ?