From f307e36e66c53f89eee1ac0dfdd691518abacc91 Mon Sep 17 00:00:00 2001 From: Martin Trigaux Date: Wed, 8 Jun 2016 10:57:21 +0200 Subject: [PATCH] [FIX] base_import: disable tracking during validation Force the value tracking_disable to be make sure that the validation (which is supposed to have no impact), will not send any email. Fixes #3173 --- addons/base_import/static/src/js/import.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/addons/base_import/static/src/js/import.js b/addons/base_import/static/src/js/import.js index 0ee88e0a203..916605d699c 100644 --- a/addons/base_import/static/src/js/import.js +++ b/addons/base_import/static/src/js/import.js @@ -363,9 +363,11 @@ openerp.base_import = function (instance) { var fields = this.$('.oe_import_fields input.oe_import_match_field').map(function (index, el) { return $(el).select2('val') || false; }).get(); + var tracking_disable = 'tracking_disable' in kwargs ? kwargs.tracking_disable : !this.$('#oe_import_tracking').prop('checked') + delete kwargs.tracking_disable kwargs.context = _.extend( {}, this.parent_context, - {tracking_disable: !this.$('#oe_import_tracking').prop('checked')} + {tracking_disable: tracking_disable} ); return this.Import.call('do', [this.id, fields, this.import_options()], kwargs) .then(undefined, function (error, event) { @@ -381,7 +383,7 @@ openerp.base_import = function (instance) { }) ; }, onvalidate: function () { - return this.call_import({ dryrun: true }) + return this.call_import({ dryrun: true, tracking_disable: true }) .done(this.proxy('validated')); }, onimport: function () {