From 4d4e1a1f0ef7ee363c27451bd403d0db3f7cac30 Mon Sep 17 00:00:00 2001 From: "Yogesh (OpenERP)" Date: Wed, 12 Oct 2011 13:35:40 +0530 Subject: [PATCH] [FIX] import :- default value set on required field then remove this field on required fields list. lp bug: https://launchpad.net/bugs/871625 fixed bzr revid: ysa@tinyerp.com-20111012080540-1m6x6npy7e0jaoif --- addons/web/static/src/js/data_import.js | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/addons/web/static/src/js/data_import.js b/addons/web/static/src/js/data_import.js index 9c686846751..6eee2bd8ded 100644 --- a/addons/web/static/src/js/data_import.js +++ b/addons/web/static/src/js/data_import.js @@ -49,12 +49,18 @@ openerp.web.DataImport = openerp.web.Dialog.extend({ }); }; this.ready = $.Deferred.queue().then(function () { - self.required_fields = _(self.fields).chain() + self.required_default_fields = _(self.fields).chain() .filter(function (field) { return field.required; }) - .pluck('name') + .pluck('id') .value(); - convert_fields(self); - self.all_fields.sort(); + dataset.default_get(self.required_default_fields || [], function(res) { + self.required_fields = _(self.fields).chain() + .filter(function (field) {if( !(field.id in res)) return field.required; }) + .pluck('name') + .value(); + convert_fields(self); + self.all_fields.sort(); + }); }); }, start: function() {