From 278a0b6a674e41f47856ba92e950dd45d8e515b5 Mon Sep 17 00:00:00 2001 From: niv-openerp Date: Wed, 27 Jul 2011 14:22:29 +0200 Subject: [PATCH] [fix] removed auto-save and put save on get_value() bzr revid: nicolas.vanhoren@openerp.com-20110727122229-hqdh03hux2lu6a3t --- addons/base/static/src/js/form.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/addons/base/static/src/js/form.js b/addons/base/static/src/js/form.js index be09c5741d2..5b21ecb7e45 100644 --- a/addons/base/static/src/js/form.js +++ b/addons/base/static/src/js/form.js @@ -1638,10 +1638,9 @@ openerp.base.form.FieldOne2Many = openerp.base.form.Field.extend({ once.resolve(); }); controller.on_form_changed.add_last(function() { - if (self.save_form_view() === false) { - // we assume one value was changed to invalid state - self.validate(); - } + // may not be the better solution but I can't imagine a better one, + // auto-save in form view is way harder to do anyway + self.on_ui_change(); }); controller.$element.find(".oe_form_button_save_edit").hide(); } @@ -1727,6 +1726,7 @@ openerp.base.form.FieldOne2Many = openerp.base.form.Field.extend({ var self = this; if (!this.dataset) return []; + self.save_form_view() var val = this.dataset.delete_all ? [commands.delete_all()] : []; val = val.concat(_.map(this.dataset.ids, function(id) { var alter_order = _.detect(self.dataset.to_create, function(x) {return x.id === id;});