Chirag Patel (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/6.1-opw-572764-cpa 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-572764-cpa/+merge/98996

Hello,

Save editable one2many in form with IE8 gives error.

Demo: Accounting > Bank and Case > Bank Statements
1. Create new Bank Statements.
2. Create new Statement lines.
3. Click on Save(floppy icon).

Thanks.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-572764-cpa/+merge/98996
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/6.1-opw-572764-cpa.
=== modified file 'addons/web/static/src/js/view_list.js'
--- addons/web/static/src/js/view_list.js	2012-03-05 17:34:19 +0000
+++ addons/web/static/src/js/view_list.js	2012-03-23 10:18:18 +0000
@@ -1432,8 +1432,10 @@
         _(this.children)
             .each(function (child) {
                 var selection = child.get_selection();
-                ids.push.apply(ids, selection.ids);
-                records.push.apply(records, selection.records);
+                if (selection.ids){
+                    ids.push.apply(ids, selection.ids);
+                    records.push.apply(records, selection.records);
+                }
             });
 
         return {ids: ids, records: records};

_______________________________________________
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