From f0cfe7c174307009575661d8ebe4c2e5d84c1768 Mon Sep 17 00:00:00 2001 From: Christophe Matthieu Date: Thu, 9 Mar 2017 12:44:40 +0100 Subject: [FIX] web: load missing fields when opening form view dialogs Problem was that sub form field has usually more fields than the list view in one2many, so we have to load missing fields --- addons/web/static/src/js/views/basic/basic_renderer.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'addons/web/static/src/js/views/basic/basic_renderer.js') diff --git a/addons/web/static/src/js/views/basic/basic_renderer.js b/addons/web/static/src/js/views/basic/basic_renderer.js index e581b222417..7b57f16dd84 100644 --- a/addons/web/static/src/js/views/basic/basic_renderer.js +++ b/addons/web/static/src/js/views/basic/basic_renderer.js @@ -68,17 +68,20 @@ return AbstractRenderer.extend({ var field = record.fields[name]; var value = record.data[name]; result[name] = value; - if (field.type === 'many2one') { + if (!value) { + return; + } + else if (field.type === 'many2one') { result[name] = value.data ? value.data.id || false : false; } - if (field.type === 'one2many' || field.type === 'many2many') { + else if (field.type === 'one2many' || field.type === 'many2many') { if (value) { result[name] = _.pluck(value.data, 'res_id'); } else { result[name] = []; } } - if (field.type === 'boolean') { + else if (field.type === 'boolean') { // we want an explicit false value, not null if it is unset result[name] = value || false; } -- cgit v1.2.3