Xavier ALT (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-client-web/6.0-opw-18475-xal into 
lp:openobject-client-web.

Requested reviews:
  Xavier (Open ERP) (xmo)

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-18475-xal/+merge/108115

Hi,

This fix an issue when trying to opening one2many field in cascade, crashing 
after a number of levels.

Steps:
- Create an Invoice
- Add one invoice line with one tax
- Open that tax line
- Add a new Sub Tax and 'Save and Close'
- Edit that Sub Tax
- Add a new Sub Tax and 'Save and Close'

Current: client crashes
Expected: client should not crash

Regards,
Xavier
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-18475-xal/+merge/108115
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-client-web/6.0-opw-18475-xal.
=== modified file 'addons/openerp/static/javascript/o2m.js'
--- addons/openerp/static/javascript/o2m.js	2011-09-19 12:50:15 +0000
+++ addons/openerp/static/javascript/o2m.js	2012-05-31 08:05:27 +0000
@@ -55,12 +55,12 @@
     this.params_parent = false
     if (this.m2o == "False"){
         this.params_parent = {
-            '_terp_view_params/_terp_model': openobject.dom.get('_terp_model').value,
-            '_terp_view_params/_terp_id': openobject.dom.get('_terp_id').value,
-            '_terp_view_params/_terp_view_id':openobject.dom.get('_terp_view_id').value,
-            '_terp_view_params/_terp_view_ids': openobject.dom.get('_terp_view_ids').value,
-            '_terp_view_params/_terp_view_mode': openobject.dom.get('_terp_view_mode').value,
-            '_terp_view_params/_terp_context': openobject.dom.get('_terp_context').value || {},
+            '_terp_view_params/_terp_model': openobject.dom.get(parent_prefix + '_terp_model').value,
+            '_terp_view_params/_terp_id': openobject.dom.get(parent_prefix + '_terp_id').value,
+            '_terp_view_params/_terp_view_id':openobject.dom.get(parent_prefix + '_terp_view_id').value,
+            '_terp_view_params/_terp_view_ids': openobject.dom.get(parent_prefix + '_terp_view_ids').value,
+            '_terp_view_params/_terp_view_mode': openobject.dom.get(parent_prefix + '_terp_view_mode').value,
+            '_terp_view_params/_terp_context': openobject.dom.get(parent_prefix + '_terp_context').value || {},
             '_terp_view_params/_terp_view_type': 'form'
         };
     }

_______________________________________________
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