From 1e30b30a8fc08d539cb68fa214226501a3fb96d6 Mon Sep 17 00:00:00 2001 From: Cedric Snauwaert Date: Mon, 3 Jul 2017 14:58:22 +0200 Subject: [FIX] web: onchange with domain should not crash if field is not in view In some cases (onchange in a purchase order), the onchange can return a domain for a field that is not in view. It should not be the case, but it is really simple to protect the web client anyway. --- addons/web/static/src/js/views/basic/basic_model.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'addons/web/static/src/js/views/basic/basic_model.js') 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 61c153e9032..2f06168fd23 100644 --- a/addons/web/static/src/js/views/basic/basic_model.js +++ b/addons/web/static/src/js/views/basic/basic_model.js @@ -2996,7 +2996,9 @@ var BasicModel = AbstractModel.extend({ if (result.domain) { var fieldsInfo = record.fieldsInfo[viewType || record.viewType]; for (var fieldName in result.domain) { - fieldsInfo[fieldName].domain = result.domain[fieldName]; + if (fieldsInfo[fieldName]) { + fieldsInfo[fieldName].domain = result.domain[fieldName]; + } } } return self._applyOnChange(result.value, record).then(function () { -- cgit v1.2.3