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

[Cédric Krier]

changeset 7c9da8ff4d4f in sao:default
details: https://hg.tryton.org/sao?cmd=changeset;node=7c9da8ff4d4f
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
diffstat:

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

diffs (18 lines):

diff -r 3cdde3cf9153 -r 7c9da8ff4d4f src/view/tree.js
--- a/src/view/tree.js  Wed Jun 12 19:04:42 2019 +0200
+++ b/src/view/tree.js  Wed Jun 12 19:06:19 2019 +0200
@@ -1335,7 +1335,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;
+                    }
+                }
 
                 var EditableBuilder = Sao.View.EditableTree.WIDGETS[
                     col.attributes.widget];