[tryton-commits] changeset in sao:5.0 Test if attribute readonly is undefined bef...

[Cédric Krier]

changeset 36bee229469e in sao:5.0
details: https://hg.tryton.org/sao?cmd=changeset;node=36bee229469e
description:
        Test if attribute readonly is undefined before using states value

        This is the general behavior for readonly attribute so
        RowEditable.set_editable must follow it to avoid incoherent behavior.

        issue8390
        review249401002
        (grafted from 7c9da8ff4d4fb7de56f005d2f276ad5ad56c8d41)
diffstat:

 src/view/tree.js |  8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diffs (18 lines):

diff -r 725b5a21ea10 -r 36bee229469e src/view/tree.js
--- a/src/view/tree.js  Tue May 28 09:53:36 2019 +0200
+++ b/src/view/tree.js  Wed Jun 12 19:06:19 2019 +0200
@@ -1305,7 +1305,13 @@
                     continue;
                 }
                 var state_attrs = col.field.get_state_attrs(this.record);
-                var readonly = col.attributes.readonly || state_attrs.readonly;
+                var readonly = col.attributes.readonly;
+                if (readonly === undefined) {
+                    readonly = state_attrs.readonly;
+                    if (readonly === undefined) {
+                        readonly = false;
+                    }
+                }
 
                 if (!readonly) {
                     var EditableBuilder = Sao.View.editabletree_widget_get(