[tryton-commits] changeset in sao:default Do not rely on depends to select the fi...

[Nicolas Évrard]

changeset e674453cb747 in sao:default
details: https://hg.tryton.org/sao?cmd=changeset&node=e674453cb747
description:
        Do not rely on depends to select the fields to read in form view

        issue11134
        review389681019
diffstat:

 src/view/form.js |  17 ++++++-----------
 1 files changed, 6 insertions(+), 11 deletions(-)

diffs (29 lines):

diff -r fb62f70ec995 -r e674453cb747 src/view/form.js
--- a/src/view/form.js  Sun Jan 30 13:10:57 2022 +0100
+++ b/src/view/form.js  Mon Jan 31 22:25:58 2022 +0100
@@ -313,19 +313,14 @@
             if (record) {
                 // Force to set fields in record
                 // Get first the lazy one from the view to reduce number of 
requests
-                var field_names = new Set();
-                for (name in this.widgets) {
-                    depends = (
-                        record.model.fields[name].description.depends || []);
-                    field_names.add(name);
-                    for (var i = 0; i < depends.length; i++) {
-                        var depend = depends[i];
-                        if (!depend.startsWith('_parent') &&
-                            depend in record.model.fields) {
-                            field_names.add(depend);
-                        }
+                var field_names = new Set(this.get_fields());
+                for (name in record.model.fields) {
+                    field = record.model.fields[name];
+                    if (~field.views.indexOf(this.view_id)) {
+                        field_names.add(name);
                     }
                 }
+
                 var fields = [];
                 field_names.forEach(function(fname) {
                     field = record.model.fields[fname];