openerp-community-reviewer team mailing list archive

[Lionel Sausin - Numérigraphe <ls@xxxxxxxxxxxxxxxx>]

Lionel Sausin - Numérigraphe has proposed merging lp:~numerigraphe-team/ocb-web/7.0-opw-605931-msh into lp:ocb-web.

Requested reviews:
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #1265705 in OpenERP Community Backports (Web): "Duplicate Records, when edit a record in OpenERP 7 one2many list editable"
  https://bugs.launchpad.net/ocb-web/+bug/1265705

For more details, see:
https://code.launchpad.net/~numerigraphe-team/ocb-web/7.0-opw-605931-msh/+merge/214482

No-change merge of lp:~openerp-dev/openerp-web/7.0-opw-605931-msh, only adding the missing --fixes in the merge commit.
Fixes a bug which creates duplicate lines under cryptic circumstances.

Runs green on runbot: http://runbot.openerp.com/numerigraphe-team-7-0-opw-605931-msh-32859/logs/
-- 
https://code.launchpad.net/~numerigraphe-team/ocb-web/7.0-opw-605931-msh/+merge/214482
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~numerigraphe-team/ocb-web/7.0-opw-605931-msh into lp:ocb-web.

=== modified file 'addons/web/static/src/js/view_form.js'
--- addons/web/static/src/js/view_form.js	2014-03-26 12:35:13 +0000
+++ addons/web/static/src/js/view_form.js	2014-04-07 08:16:45 +0000
@@ -620,6 +620,9 @@
                         });
                     }
                     return $.when();
+                }).fail(function() {
+                    self.save_list.pop();
+                    return $.when();
                 });
             };
             return iterate();