From c1256d1453078d9ceb64ea308efc39acb6cd4465 Mon Sep 17 00:00:00 2001 From: qsm-odoo Date: Mon, 13 Mar 2017 11:48:03 +0100 Subject: [FIX] web: record with field status with domain If a m2o field status has an associated domain, it is used to match the records to display as status choices. This domain is extended so that it always matches the current status value. The code which was in charge of that was buggy when the status had no value (in that case the domain should not be extended). --- addons/web/static/src/js/views/basic/basic_model.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'addons/web/static/src/js/views/basic') diff --git a/addons/web/static/src/js/views/basic/basic_model.js b/addons/web/static/src/js/views/basic/basic_model.js index 2cfa96a3015..8e534d4eda1 100644 --- a/addons/web/static/src/js/views/basic/basic_model.js +++ b/addons/web/static/src/js/views/basic/basic_model.js @@ -1414,8 +1414,10 @@ var BasicModel = AbstractModel.extend({ var domain = record.getDomain({fieldName: fieldName}); if (domain.length) { var localID = record.data[fieldName]; - var element = this.localData[localID]; - domain = ["|", ["id", "=", element.data.id]].concat(domain); + if (localID) { + var element = this.localData[localID]; + domain = ["|", ["id", "=", element.data.id]].concat(domain); + } } // avoid rpc if not necessary -- cgit v1.2.3