From ce36786038b7b9bfa3e292ce40207d1ea46f3c92 Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Wed, 15 Mar 2017 12:06:29 +0100 Subject: [FIX] web: _buildOnchangeSpecs must return only the present fields Before: the method '_buildOnchangeSpecs' returned all fields, but some models like 'res.users' have redefined the fields_get and add some fake fields which cause a crash. --- addons/web/static/src/js/views/basic/basic_model.js | 4 ++-- 1 file changed, 2 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 8ae01e87bbc..7d33d51257f 100644 --- a/addons/web/static/src/js/views/basic/basic_model.js +++ b/addons/web/static/src/js/views/basic/basic_model.js @@ -1239,8 +1239,8 @@ var BasicModel = AbstractModel.extend({ var field = record.fields[name]; specs[name] = (field.onChange) || ""; _.each(field.views, function (view) { - _.each(view.fields, function (field, subname) { - specs[name + '.' + subname] = (field.onChange) || ""; + _.each(view.fieldAttrs, function (field, subname) { + specs[name + '.' + subname] = (view.fields[subname].onChange) || ""; }); }); }); -- cgit v1.2.3