openerp-dev-web team mailing list archive

["Sananaz \(Open ERP\)" <sma@xxxxxxxxxxx>]

Sananaz (Open ERP) has proposed merging lp:~openerp-dev/openobject-client-web/6.0-opw-5073-sma into lp:openobject-client-web/6.0.

Requested reviews:
  OpenERP Core Team (openerp)
Related bugs:
  Bug #753937 in OpenERP Web Client: "Error when click on New button to create new customer invoice line"
  https://bugs.launchpad.net/openobject-client-web/+bug/753937

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-5073-sma/+merge/56917

Hello,

Problem is when duplicate customer invoice then create new invoice line it gives traceback and crash it.

It creates regression because of the revision:4566, remove view_ids and view_modes so it create problem.

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-5073-sma/+merge/56917
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-5073-sma.

=== modified file 'addons/openerp/static/javascript/o2m.js'
--- addons/openerp/static/javascript/o2m.js	2011-04-04 15:02:36 +0000
+++ addons/openerp/static/javascript/o2m.js	2011-04-08 11:27:27 +0000
@@ -39,6 +39,8 @@
     this.parent_id = openobject.dom.get(parent_prefix + '_terp_id').value;
     this.parent_context = openobject.dom.get(parent_prefix + '_terp_context').value;
     this.parent_view_id = openobject.dom.get(parent_prefix + '_terp_view_id').value;
+    this.parent_view_ids = jQuery(idSelector(parent_prefix + '_terp_view_ids')).val();
+    this.parent_view_mode = jQuery(idSelector(parent_prefix + '_terp_view_mode')).val();
 
     // hide new button when editors are visible
     if (this.mode == 'tree' && this.inline) {
@@ -128,7 +130,8 @@
             '_terp_view_params/_terp_model': this.parent_model,
             '_terp_view_params/_terp_id': this.parent_id,
             '_terp_view_params/_terp_view_id':this.parent_view_id,
-            '_terp_view_params/_terp_view_mode': this.mode,
+            '_terp_view_params/_terp_view_ids': this.parent_view_ids,
+            '_terp_view_params/_terp_view_mode': this.parent_view_mode,
             '_terp_view_params/_terp_context': this.parent_context || {},
             '_terp_view_params/_terp_view_type': 'form'
         };