From 533f17231c1aee913b8806c1e39339851514d15b Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Wed, 13 Jun 2012 17:35:55 +0200 Subject: [PATCH] [FIX] if a form is not dirty, always consider the corresponding o2m[list] to be valid the user has not modified the editable line after opening it (whether to create a new record or to edit an existing one), so shouldn't be invalid bzr revid: xmo@openerp.com-20120613153555-fq0moy1q464l92yj --- addons/web/static/src/js/view_form.js | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/addons/web/static/src/js/view_form.js b/addons/web/static/src/js/view_form.js index 8d0fd141cdb..6257633b8d2 100644 --- a/addons/web/static/src/js/view_form.js +++ b/addons/web/static/src/js/view_form.js @@ -2561,6 +2561,12 @@ openerp.web.form.One2ManyListView = openerp.web.ListView.extend({ if (!(form = this.first_edition_form())) { return true; } + // If the form has not been modified, the view can only be valid + // NB: is_dirty will also be set on defaults/onchanges/whatever? + // oe_form_dirty seems to only be set on actual user actions + if (!form.$element.is('.oe_form_dirty')) { + return true; + } // Otherwise validate internal form return _(form.fields).chain()