tryton-sao 7.0.2 → 7.0.3

package/CHANGELOG

@@ -1,4 +1,9 @@
 
+Version 7.0.3 - 2023-12-16
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.0.2 - 2023-12-01
 --------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.js

@@ -1,7 +1,7 @@
 /* This file is part of Tryton.  The COPYRIGHT file at the top level of
    this repository contains the full copyright notices and license terms. */
 var Sao = {
-    __version__: '7.0.2',
+    __version__: '7.0.3',
 };
 
 (function() {
@@ -3256,12 +3256,14 @@ var Sao = {
 
     Sao.common.scrollIntoViewIfNeeded = function(element) {
         element = element[0];
-        var rect = element.getBoundingClientRect();
-        if (rect.bottom > window.innerHeight) {
-            element.scrollIntoView(false);
-        }
-        if (rect.top < 0) {
-            element.scrollIntoView();
+        if (element) {
+            var rect = element.getBoundingClientRect();
+            if (rect.bottom > window.innerHeight) {
+                element.scrollIntoView(false);
+            }
+            if (rect.top < 0) {
+                element.scrollIntoView();
+            }
         }
     };
 
@@ -4106,9 +4108,7 @@ var Sao = {
             } else {
                 this.el.show();
             }
-            this.el.prop(
-                'disabled',
-                (record? record.readonly : false) || Boolean(states.readonly));
+            this.el.prop('disabled', Boolean(states.readonly));
             this.set_icon(states.icon || this.attributes.icon);
 
             if (this.attributes.rule) {
@@ -5927,12 +5927,12 @@ var Sao = {
             }
             return this.simplify(this.merge(result));
         },
-        unique_value: function(domain) {
+        unique_value: function(domain, single_value=true) {
             if ((domain instanceof Array) &&
                     (domain.length == 1)) {
                 let [name, operator, value, ...model] = domain[0];
                 if (operator == '=' ||
-                    (operator == 'in' && value.length == 1)) {
+                    (single_value && operator == 'in' && value.length == 1)) {
                     value = operator == '=' ? value : value[0];
                     var count = 0;
                     if (model.length && name.endsWith('.id')) {
@@ -8375,6 +8375,7 @@ var Sao = {
                 }
                 if (this.model.fields[name] instanceof Sao.field.One2Many) {
                     later[name] = value;
+                    continue;
                 }
                 const field = this.model.fields[name];
                 var related;
@@ -8399,6 +8400,7 @@ var Sao = {
                 value = later[name];
                 this.model.fields[name].set(this, value);
                 this._loaded[name] = true;
+                fieldnames.push(name);
             }
             if (validate) {
                 this.validate(fieldnames, true, false, false);
@@ -9095,6 +9097,7 @@ var Sao = {
 
     Sao.field.Field = Sao.class_(Object, {
         _default: null,
+        _single_value: true,
         init: function(description) {
             this.description = description;
             this.name = description.name;
@@ -9267,12 +9270,13 @@ var Sao = {
             } else {
                 let [screen_domain, _] = this.get_domains(
                     record, pre_validate);
-                var uniques = inversion.unique_value(domain);
+                var uniques = inversion.unique_value(
+                    domain, this._single_value);
                 var unique = uniques[0];
                 var leftpart = uniques[1];
                 var value = uniques[2];
                 let unique_from_screen = inversion.unique_value(
-                    screen_domain)[0];
+                    screen_domain, this._single_value)[0];
                 if (this._is_empty(record) &&
                     !is_required &&
                     !is_invisible &&
@@ -9360,6 +9364,7 @@ var Sao = {
 
     Sao.field.MultiSelection = Sao.class_(Sao.field.Selection, {
         _default: null,
+        _single_value: false,
         get: function(record) {
             var value = Sao.field.MultiSelection._super.get.call(this, record);
             if (jQuery.isEmptyObject(value)) {
@@ -9742,6 +9747,7 @@ var Sao = {
             Sao.field.One2Many._super.init.call(this, description);
         },
         _default: null,
+        _single_value: false,
         _set_value: function(record, value, default_, modified) {
             this._set_default_value(record);
             var group = record._values[this.name];
@@ -10352,6 +10358,7 @@ var Sao = {
 
     Sao.field.Dict = Sao.class_(Sao.field.Field, {
         _default: {},
+        _single_value: false,
         init: function(description) {
             Sao.field.Dict._super.init.call(this, description);
             this.schema_model = new Sao.Model(description.schema_model);
@@ -10763,8 +10770,9 @@ var Sao = {
             return toolbar;
         },
         show: function() {
-            jQuery('#tablist').find('a[href="#' + this.id + '"]')
-                .tab('show')[0].scrollIntoView();
+            Sao.common.scrollIntoViewIfNeeded(
+                jQuery('#tablist').find('a[href="#' + this.id + '"]')
+                .tab('show'));
         },
         close: function() {
             var tabs = jQuery('#tabs');
@@ -10849,8 +10857,8 @@ var Sao = {
         }
         for (const other of Sao.Tab.tabs) {
             if (other.compare(attributes)) {
-                tablist.find('a[href="#' + other.id + '"]')
-                    .tab('show')[0].scrollIntoView();
+                Sao.common.scrollIntoViewIfNeeded(
+                    tablist.find('a[href="#' + other.id + '"]').tab('show'));
                 return;
             }
         }
@@ -10907,7 +10915,7 @@ var Sao = {
         tab_link.on('shown.bs.tab', function(evt) {
             tabs.scrollTop(jQuery(evt.target).data('scrollTop') || 0);
         });
-        tab_link.tab('show')[0].scrollIntoView();
+        Sao.common.scrollIntoViewIfNeeded(tab_link.tab('show'));
         tabs.trigger('ready');
     };
 
@@ -11507,7 +11515,6 @@ var Sao = {
                     ['delete_', access.delete],
                     ['copy', access.create],
                     ['import', access.create],
-                    ['action', access.write && !this.screen.readonly],
                 ]);
                 for (const [name, access] of accesses) {
                     if (this.buttons[name]) {
@@ -11520,7 +11527,7 @@ var Sao = {
                 }
             } else {
                 for (const name of [
-                    'new_', 'save', 'delete_', 'copy', 'import', 'action']) {
+                    'new_', 'save', 'delete_', 'copy', 'import']) {
                     if (this.buttons[name]) {
                         this.buttons[name].prop('disabled', true);
                     }
@@ -11754,8 +11761,8 @@ var Sao = {
             preview.record_message = function(position, length) {
                 var text = (position || '_') + '/' + length;
                 label.text(text).attr('title', text);
-                but_prev.prop('disabled', !position || position <= 1);
-                but_next.prop('disabled', !position || position >= length);
+                but_prev.prop('disabled', !screen.has_previous());
+                but_next.prop('disabled', !screen.has_next());
             };
             screen.windows.push(preview);
 
@@ -11780,7 +11787,13 @@ var Sao = {
             if (!Sao.common.compare(this.attachment_screen.domain, domain) ||
                 force) {
                 this.attachment_screen.domain = domain;
-                this.attachment_screen.search_filter();
+                this.attachment_screen.search_filter().then(() => {
+                    const group = this.attachment_screen.group;
+                    if (group.length) {
+                        this.attachment_screen.current_record = group[0];
+                        this.attachment_screen.display();
+                    }
+                });
             }
         },
         note: function() {
@@ -14049,8 +14062,7 @@ var Sao = {
             var buttons = this.current_view.get_buttons();
             for (const record of selected_records) {
                 buttons = buttons.filter(function(button) {
-                    if (record.readonly ||
-                        button.attributes.type === 'instance') {
+                    if (button.attributes.type === 'instance') {
                         return false;
                     }
                     var states = record.expr_eval(
@@ -19155,7 +19167,7 @@ function eval_pyson(value){
         },
         set_url: function(value) {
             this.button.attr('href', value);
-            this.button.prop('disabled', !value);
+            this.button.toggle(value);
         },
         set_invisible: function(invisible) {
             Sao.View.Form.URL._super.set_invisible.call(this, invisible);
@@ -20915,12 +20927,14 @@ function eval_pyson(value){
                 } else if (name === this.screen.exclude_field) {
                     visible_columns -= 1;
                     column.set_visible(false);
-                } else {
+                } else if (name in this.screen.model.fields) {
+                    const field = this.screen.model.fields[name];
                     var inv_domain = inversion.domain_inversion(domain, name);
                     if (typeof inv_domain != 'boolean') {
                         inv_domain = inversion.simplify(inv_domain);
                     }
-                    var unique = inversion.unique_value(inv_domain)[0];
+                    var unique = inversion.unique_value(
+                        inv_domain, field._single_value)[0];
                     if (unique && jQuery.isEmptyObject(this.children_field)) {
                         visible_columns -= 1;
                         column.set_visible(false);