Mohammed Shekha(Open ERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-577817-msh 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-577817-msh/+merge/119335

Hello,

Fixed the issue of request_context not set in get_context method of 
BufferedDataset.

Demo :- Open any editable one2many which has translatable field, now open 
Translation dialog and make modification and save it, you will see changes will 
not be reflected, with translation dialog when we save the translation changes 
rpc should called,. but here for one2many if you edit and save the translation 
it will try to save it in BufferedDataset which has been fixed in 
lp:~openerp-dev/openerp-web/6.1-opw-577204-msh.

Still when you open translation dialog from editable one2many you will not get 
translation terms according to language instead you will get same translation 
terms for all field with current language term, that is due to request_context 
not set in get_context for BufferedDataset.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-577817-msh/+merge/119335
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-577817-msh.
=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2012-08-10 09:09:55 +0000
+++ addons/web/static/src/js/view_form.js	2012-08-13 12:12:23 +0000
@@ -2700,8 +2700,11 @@
 });
 
 openerp.web.form.One2ManyDataSet = openerp.web.BufferedDataSet.extend({
-    get_context: function() {
+    get_context: function(request_context) {
         this.context = this.o2m.build_context([this.o2m.name]);
+        if (request_context) {
+            return new openerp.web.CompoundContext(this.context, request_context);
+        }
         return this.context;
     }
 });

_______________________________________________
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