tryton-sao 7.0.2 → 7.0.4

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 7.0.4 - 2024-01-01
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+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.css

@@ -10229,7 +10229,8 @@ input.column-boolean {
   }
   .xalign-center,
   .xalign-end {
-    text-align: start;
+    justify-self: start;
+    justify-content: start;
   }
 }
 @media screen and (max-width: 767px) {

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.4',
 };
 
 (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();
+            }
         }
     };
 
@@ -3987,8 +3989,12 @@ var Sao = {
                     }
                 });
                 prm.fail(() => {
+                    var selection = [];
+                    if (this.nullable_widget) {
+                        selection.push([null, '']);
+                    }
                     this._last_domain = null;
-                    this.selection = [];
+                    this.selection = selection;
                     if (callback) {
                         callback(this.selection, this.help);
                     }
@@ -4034,6 +4040,9 @@ var Sao = {
         if (!this.attributes.relation) {
             return jQuery.when([]);
         }
+        if (value === null) {
+            return jQuery.when([null, '']);
+        }
         for (var i = 0, len = this.inactive_selection.length; i < len; i++) {
             if (value == this.inactive_selection[i][0]) {
                 return jQuery.when(this.inactive_selection[i]);
@@ -4106,9 +4115,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) {
@@ -5612,10 +5619,6 @@ var Sao = {
             if (!domain.length) {
                 return [0, domain];
             } else if (this.is_leaf(domain)) {
-                if (domain[2] === null) {
-                    domain = domain.slice();
-                    domain[2] = Number.NEGATIVE_INFINITY;
-                }
                 return [1, domain];
             } else if (~['AND', 'OR'].indexOf(domain)) {
                 return [0, domain];
@@ -5927,12 +5930,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 +8378,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 +8403,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);
@@ -8665,7 +8670,8 @@ var Sao = {
                 }
                 fieldnames[fieldname] = true;
                 values = jQuery.extend(values,
-                        this._get_on_change_args(on_change_with));
+                    this._get_on_change_args(
+                        on_change_with.concat([fieldname])));
                 if ((this.model.fields[fieldname] instanceof
                             Sao.field.Many2One) ||
                         (this.model.fields[fieldname] instanceof
@@ -8704,7 +8710,8 @@ var Sao = {
                         .description.on_change_with;
                     values = jQuery.extend(
                         values,
-                        this._get_on_change_args(on_change_with));
+                        this._get_on_change_args(
+                            on_change_with.concat([fieldname])));
                 }
                 fieldnames = Object.keys(later);
                 try {
@@ -9095,6 +9102,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 +9275,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 +9369,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 +9752,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 +10363,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 +10775,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 +10862,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 +10920,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 +11520,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 +11532,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 +11766,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 +11792,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 +14067,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(
@@ -18640,8 +18657,12 @@ function eval_pyson(value){
             const callback = result => {
                 if (result) {
                     screen.current_record.save().done(() => {
+                        var added = 'id' in this.screen.current_record.modified_fields;
                         // Force a reload on next display
                         this.screen.current_record.cancel();
+                        if (added) {
+                            this.screen.current_record.modified_fields.id = true;
+                        }
                     });
                 }
             };
@@ -19101,12 +19122,16 @@ function eval_pyson(value){
                     });
                     url = window.URL.createObjectURL(blob);
                 }
+                // duplicate object to force refresh on buggy browsers
+                const object = this.object.clone();
                 // set onload before data to be always called
-                this.object.get(0).onload = function() {
+                object.get(0).onload = function() {
                     this.onload = null;
                     window.URL.revokeObjectURL(url);
                 };
-                this.object.attr('data', url);
+                object.attr('data', url);
+                this.object.replaceWith(object);
+                this.object = object;
             });
         },
     });
@@ -19155,7 +19180,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 +20940,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);

package/dist/tryton-sao.min.css

@@ -10229,7 +10229,8 @@ input.column-boolean {
   }
   .xalign-center,
   .xalign-end {
-    text-align: start;
+    justify-self: start;
+    justify-content: start;
   }
 }
 @media screen and (max-width: 767px) {