Merge lp:~openerp-dev/openerp-web/7.0-opw-584981-cpa into lp:openerp-web/7.0

[Chirag Patel (OpenERP)]

Hello,

Can't save object with deleted invalid row in editable tree.

Reproduce Issue: check the related bug #1103507 description.

Thanks

[Xavier (Open ERP) (xmo-deactivatedaccount)]

This is not the right place to perform this operation, if it should be performed at all: at worst it should be performed in the editor's `cancel` method, but at best (and more importantly) wouldn't the actual problem more likely be in the way the O2M field (or its component One2ManyListView) checks for its own validity?

review: Needs Fixing

Unmerged revisions

3732. By Chirag Patel (OpenERP) on 2013-01-30

[FIX] Can't save object with deleted invalid row in editable tree.

=== modified file 'addons/web/static/src/js/view_list_editable.js'
--- addons/web/static/src/js/view_list_editable.js	2012-12-10 10:09:14 +0000
+++ addons/web/static/src/js/view_list_editable.js	2013-01-30 13:32:52 +0000
@@ -88,6 +88,8 @@
         do_delete: function (ids) {
             var nonfalse = _.compact(ids);
             var _super = this._super.bind(this);
+            var form = this.editor.form;
+            form.$el.removeClass('oe_form_dirty');
             var next = this.editor.is_editing()
                     ? this.cancel_edition(true)
                     : $.when();