tryton-sao 7.6.0 → 7.6.2

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 7.6.2 - 2025-06-04
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 7.6.1 - 2025-05-15
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.6.0 - 2025-04-28
 --------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.css

@@ -9951,6 +9951,7 @@ img.icon {
   text-align: end;
 }
 .treeview > table.tree > thead > tr:first-child {
+  background-color: #fff;
   position: sticky;
   top: 0;
   z-index: 100;
@@ -9958,20 +9959,17 @@ img.icon {
 .treeview > table.tree > thead > tr:first-child th {
   border: 1px solid transparent;
   border-bottom: 2px solid #ddd;
-  background-color: #fff;
 }
 .treeview > table.tree > thead > tr:first-child:hover th {
   border: 1px solid #ddd;
   border-bottom-width: 2px;
 }
 .treeview > table.tree > tfoot > tr {
+  background-color: #fff;
   bottom: 0;
   position: sticky;
   z-index: 100;
 }
-.treeview > table.tree > tfoot > tr th {
-  background-color: #fff;
-}
 .treeview > table.tree > tfoot > tr th > label {
   display: none;
 }
@@ -10408,12 +10406,14 @@ img.icon {
 .form .form-richtext,
 .board .form-richtext {
   display: flex;
+  flex-direction: column;
 }
 .form .form-text .input-group,
 .board .form-text .input-group,
 .form .form-richtext .input-group,
 .board .form-richtext .input-group {
   display: flex;
+  height: 100%;
   width: 100%;
 }
 .form .form-text .input-group textarea,
@@ -10429,6 +10429,7 @@ img.icon {
   min-height: 12.5ex;
   overflow: auto;
   resize: vertical;
+  width: 100%;
 }
 .form .form-text .input-group > span,
 .board .form-text .input-group > span,
@@ -10440,6 +10441,10 @@ img.icon {
 .board .form-richtext > .btn-toolbar {
   min-width: 450px;
 }
+.form .form-richtext > .panel-body,
+.board .form-richtext > .panel-body {
+  flex-grow: 2;
+}
 .form .form-separator label,
 .board .form-separator label {
   margin-top: 10px;
@@ -10464,6 +10469,7 @@ img.icon {
   max-width: 100%;
   min-height: 50vh;
   height: 100%;
+  width: 100%;
 }
 @media screen and (max-width: 767px) {
   .form .form-document object,

package/dist/tryton-sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.6.0',
+    __version__: '7.6.2',
 };
 /* eslint-enable no-redeclare */
 
@@ -675,6 +675,7 @@ var Sao = {
                                 "Incompatible version of the server."),
                             Sao.i18n.gettext("Version mismatch"));
                     } else {
+                        let url = window.location.hash.substr(1);
                         Sao.Session.get_credentials()
                             .then(function(session) {
                                 Sao.Session.current_session = session;
@@ -684,7 +685,7 @@ var Sao = {
                             .then(function(preferences) {
                                 Sao.menu(preferences);
                                 Sao.user_menu(preferences);
-                                Sao.open_url();
+                                Sao.open_url(url);
                                 Sao.Bus.listen();
                             });
                     }
@@ -7721,16 +7722,19 @@ var Sao = {
                 'data:image/gif;base64,R0lGODlhAQABAAD/ACwAAAAAAQABAAACADs%3D');
         }
         var type = '';
-        try {
-            var xml = data;
-            if (xml instanceof Uint8Array) {
-                xml = new TextDecoder().decode(data);
-            }
-            if (jQuery.parseXML(xml)) {
+        var xml = data;
+        if (xml instanceof Uint8Array) {
+            xml = new TextDecoder().decode(data);
+        }
+        // simple test to avoid logging of parsing error
+        if (/^\s*<[\s\S]+>\s*$/.test(xml.trim())) {
+            let parser = new DOMParser();
+            let doc = parser.parseFromString(xml, 'image/svg+xml');
+            if (!doc.querySelector('parsererror')
+                && (doc.documentElement.tagName.toLowerCase() === 'svg' ||
+                    doc.getElementsByTagName('svg').length > 0)) {
                 type = 'image/svg+xml';
             }
-        } catch (e) {
-            // continue
         }
         var blob = new Blob([data], {type: type});
         return window.URL.createObjectURL(blob);
@@ -9125,6 +9129,9 @@ var Sao = {
             }
         },
         get_loaded: function(fields) {
+            if (this.id < 0) {
+                return true;
+            }
             if (!fields) {
                 fields = Object.keys(this.model.fields);
             }
@@ -9526,7 +9533,6 @@ var Sao = {
                         // XXX to remove once server domains are fixed
                         value = null;
                     }
-                    var setdefault = true;
                     var original_domain;
                     if (!jQuery.isEmptyObject(record.group.domain)) {
                         original_domain = inversion.merge(record.group.domain);
@@ -9534,20 +9540,12 @@ var Sao = {
                         original_domain = inversion.merge(domain);
                     }
                     var domain_readonly = original_domain[0] == 'AND';
+                    let setdefault;
                     if (leftpart.contains('.')) {
-                        var recordpart = leftpart.split('.', 1)[0];
-                        var localpart = leftpart.split('.', 1)[1];
-                        var constraintfields = [];
-                        if (domain_readonly) {
-                            for (const leaf of inversion.localize_domain(
-                                original_domain.slice(1))) {
-                                constraintfields.push(leaf);
-                            }
-                        }
-                        if ((localpart != 'id') ||
-                                !~constraintfields.indexOf(recordpart)) {
-                            setdefault = false;
-                        }
+                        let localpart = leftpart.split('.').slice(1).join('.');
+                        setdefault = localpart == 'id';
+                    } else {
+                        setdefault = true;
                     }
                     if (setdefault && jQuery.isEmptyObject(pre_validate)) {
                         this.set_client(record, value);
@@ -9817,12 +9815,23 @@ var Sao = {
         },
         apply_factor: function(record, value, factor) {
             if (value !== null) {
+                // The default precision is the one used by value (before
+                // applying the factor), per the ecmascript specification
+                // it's the shortest representation of said value.
+                // Once the factor is applied the number might become even
+                // more inexact thus we should rely on the initial
+                // precision + the effect factor will have
+                // https://tc39.es/ecma262/multipage/ecmascript-data-types-and-values.html#sec-numeric-types-number-tostring
+                let default_precision = (value.toString().split('.')[1] || '').length;
+                default_precision += Math.ceil(Math.log10(factor));
                 value /= factor;
                 var digits = this.digits(record);
                 if (digits && (digits[1] !== null)) {
                     // Round to avoid float precision error
                     // after the division by factor
                     value = value.toFixed(digits[1]);
+                } else {
+                    value = value.toFixed(default_precision);
                 }
                 value = this.convert(value);
             }
@@ -10272,6 +10281,7 @@ var Sao = {
             }
 
             if (value && (value.add || value.update)) {
+                let vals_to_set = {};
                 // First set already added fields to prevent triggering a
                 // second on_change call
                 if (value.update) {
@@ -10281,9 +10291,9 @@ var Sao = {
                         }
                         const record2 = group.get(vals.id);
                         if (record2) {
-                            var vals_to_set = {};
                             for (var key in vals) {
-                                if (!Object.prototype.hasOwnProperty.call(new_field_names, key)) {
+                                if (!Object.prototype.hasOwnProperty.call(
+                                    new_field_names, key)) {
                                     vals_to_set[key] = vals[key];
                                 }
                             }
@@ -10319,7 +10329,10 @@ var Sao = {
                         if (record2) {
                             let to_update = Object.fromEntries(
                                 Object.entries(vals).filter(
-                                    ([k, v]) => !Object.prototype.hasOwnProperty.call(vals_to_set, k)
+                                    ([k, v]) => {
+                                        !Object.prototype.hasOwnProperty.call(
+                                            vals_to_set, k)
+                                    }
                                 ));
                             record2.set_on_change(to_update);
                         }
@@ -11623,15 +11636,20 @@ var Sao = {
                             case 'ok':
                                 return this.save();
                             case 'ko':
-                                var record_id = this.screen.current_record.id;
+                                var record_id = null;
+                                if (this.screen.current_record) {
+                                    record_id = this.screen.current_record.id;
+                                }
                                 return this.reload(false).then(() => {
-                                    if (record_id < 0) {
-                                        return jQuery.Deferred().reject(true);
-                                    }
-                                    else if (this.screen.current_record) {
-                                        if (record_id !=
-                                            this.screen.current_record.id) {
-                                            return jQuery.Deferred().reject();
+                                    if (record_id !== null) {
+                                        if (record_id < 0) {
+                                            return jQuery.Deferred().reject(true);
+                                        }
+                                        else if (this.screen.current_record) {
+                                            if (record_id !=
+                                                this.screen.current_record.id) {
+                                                return jQuery.Deferred().reject();
+                                            }
                                         }
                                     }
                                 });
@@ -16488,7 +16506,8 @@ function eval_pyson(value){
         class_: 'form',
         init: function(languages, widget) {
             var dialog = new Sao.Dialog(
-                Sao.i18n.gettext('Translate'), this.class_, 'md');
+                Sao.i18n.gettext('Translate'), this.class_,
+                widget.expand? 'lg' : 'md');
             this.languages = languages;
             this.read(widget, dialog);
             jQuery('<button/>', {
@@ -16579,10 +16598,10 @@ function eval_pyson(value){
                 input.uniqueId();
                 row.append(jQuery('<label/>', {
                     'for': input.attr('id'),
-                    'class': 'col-sm-3 control-label',
+                    'class': 'col-sm-2 control-label',
                 }).append(' ' + lang.name));
                 row.append(jQuery('<div/>', {
-                    'class': 'col-sm-9 input-group',
+                    'class': 'col-sm-10 input-group',
                 }).append(input)
                     .append(jQuery('<span/>', {
                         'class': 'input-group-addon',
@@ -17654,9 +17673,11 @@ function eval_pyson(value){
             jQuery('<div/>', {
                 'class': 'richtext mousetrap',
                 'contenteditable': true
+            }).appendTo(jQuery('<div/>', {
+                'class': 'input-group',
             }).appendTo(jQuery('<div/>', {
                 'class': 'panel-body'
-            }).appendTo(widget));
+            }).appendTo(widget)));
             return widget;
         },
         translate_widget_set_readonly: function(el, value) {
@@ -22020,6 +22041,8 @@ function eval_pyson(value){
                     column.col.show();
                 }
             }
+            this.table.find('thead > tr > th .resizer').show();
+            this.table.find('thead > tr > th:visible:last .resizer').hide();
             if (this.children_field) {
                 this.columns.every(column => {
                     if (column.col.hasClass('draggable-handle') ||
@@ -22075,7 +22098,8 @@ function eval_pyson(value){
                         'type': 'button',
                         'title': Sao.i18n.gettext("More"),
                     }).text(Sao.i18n.gettext('More')
-                    ).click(() => {
+                    ).one('click', () => {
+                        this.tbody.find('tr.more-row').remove();
                         var height = this.table.height();
                         this.display_size += Sao.config.display_size;
                         this.display();
@@ -25980,7 +26004,9 @@ function eval_pyson(value){
                 }
             }
             let message = name + '/' + Sao.common.humanize(size);
-            this.label.text(message).attr('title', message);
+            if (this.label) {
+                this.label.text(message).attr('title', message);
+            }
             this._set_button_sensitive();
         },
         record_modified: function() {

package/package.json

@@ -2,7 +2,7 @@
   "name": "tryton-sao",
   "title": "sao",
   "description": "Tryton webclient",
-  "version": "7.6.0",
+  "version": "7.6.2",
   "homepage": "https://www.tryton.org/",
   "author": {
     "name": "Tryton"

package/src/common.js

@@ -4389,16 +4389,19 @@
                 'data:image/gif;base64,R0lGODlhAQABAAD/ACwAAAAAAQABAAACADs%3D');
         }
         var type = '';
-        try {
-            var xml = data;
-            if (xml instanceof Uint8Array) {
-                xml = new TextDecoder().decode(data);
-            }
-            if (jQuery.parseXML(xml)) {
+        var xml = data;
+        if (xml instanceof Uint8Array) {
+            xml = new TextDecoder().decode(data);
+        }
+        // simple test to avoid logging of parsing error
+        if (/^\s*<[\s\S]+>\s*$/.test(xml.trim())) {
+            let parser = new DOMParser();
+            let doc = parser.parseFromString(xml, 'image/svg+xml');
+            if (!doc.querySelector('parsererror')
+                && (doc.documentElement.tagName.toLowerCase() === 'svg' ||
+                    doc.getElementsByTagName('svg').length > 0)) {
                 type = 'image/svg+xml';
             }
-        } catch (e) {
-            // continue
         }
         var blob = new Blob([data], {type: type});
         return window.URL.createObjectURL(blob);

package/src/model.js

@@ -1367,6 +1367,9 @@
             }
         },
         get_loaded: function(fields) {
+            if (this.id < 0) {
+                return true;
+            }
             if (!fields) {
                 fields = Object.keys(this.model.fields);
             }
@@ -1768,7 +1771,6 @@
                         // XXX to remove once server domains are fixed
                         value = null;
                     }
-                    var setdefault = true;
                     var original_domain;
                     if (!jQuery.isEmptyObject(record.group.domain)) {
                         original_domain = inversion.merge(record.group.domain);
@@ -1776,20 +1778,12 @@
                         original_domain = inversion.merge(domain);
                     }
                     var domain_readonly = original_domain[0] == 'AND';
+                    let setdefault;
                     if (leftpart.contains('.')) {
-                        var recordpart = leftpart.split('.', 1)[0];
-                        var localpart = leftpart.split('.', 1)[1];
-                        var constraintfields = [];
-                        if (domain_readonly) {
-                            for (const leaf of inversion.localize_domain(
-                                original_domain.slice(1))) {
-                                constraintfields.push(leaf);
-                            }
-                        }
-                        if ((localpart != 'id') ||
-                                !~constraintfields.indexOf(recordpart)) {
-                            setdefault = false;
-                        }
+                        let localpart = leftpart.split('.').slice(1).join('.');
+                        setdefault = localpart == 'id';
+                    } else {
+                        setdefault = true;
                     }
                     if (setdefault && jQuery.isEmptyObject(pre_validate)) {
                         this.set_client(record, value);
@@ -2059,12 +2053,23 @@
         },
         apply_factor: function(record, value, factor) {
             if (value !== null) {
+                // The default precision is the one used by value (before
+                // applying the factor), per the ecmascript specification
+                // it's the shortest representation of said value.
+                // Once the factor is applied the number might become even
+                // more inexact thus we should rely on the initial
+                // precision + the effect factor will have
+                // https://tc39.es/ecma262/multipage/ecmascript-data-types-and-values.html#sec-numeric-types-number-tostring
+                let default_precision = (value.toString().split('.')[1] || '').length;
+                default_precision += Math.ceil(Math.log10(factor));
                 value /= factor;
                 var digits = this.digits(record);
                 if (digits && (digits[1] !== null)) {
                     // Round to avoid float precision error
                     // after the division by factor
                     value = value.toFixed(digits[1]);
+                } else {
+                    value = value.toFixed(default_precision);
                 }
                 value = this.convert(value);
             }
@@ -2514,6 +2519,7 @@
             }
 
             if (value && (value.add || value.update)) {
+                let vals_to_set = {};
                 // First set already added fields to prevent triggering a
                 // second on_change call
                 if (value.update) {
@@ -2523,9 +2529,9 @@
                         }
                         const record2 = group.get(vals.id);
                         if (record2) {
-                            var vals_to_set = {};
                             for (var key in vals) {
-                                if (!Object.prototype.hasOwnProperty.call(new_field_names, key)) {
+                                if (!Object.prototype.hasOwnProperty.call(
+                                    new_field_names, key)) {
                                     vals_to_set[key] = vals[key];
                                 }
                             }
@@ -2561,7 +2567,10 @@
                         if (record2) {
                             let to_update = Object.fromEntries(
                                 Object.entries(vals).filter(
-                                    ([k, v]) => !Object.prototype.hasOwnProperty.call(vals_to_set, k)
+                                    ([k, v]) => {
+                                        !Object.prototype.hasOwnProperty.call(
+                                            vals_to_set, k)
+                                    }
                                 ));
                             record2.set_on_change(to_update);
                         }

package/src/sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.6.0',
+    __version__: '7.6.2',
 };
 /* eslint-enable no-redeclare */
 
@@ -675,6 +675,7 @@ var Sao = {
                                 "Incompatible version of the server."),
                             Sao.i18n.gettext("Version mismatch"));
                     } else {
+                        let url = window.location.hash.substr(1);
                         Sao.Session.get_credentials()
                             .then(function(session) {
                                 Sao.Session.current_session = session;
@@ -684,7 +685,7 @@ var Sao = {
                             .then(function(preferences) {
                                 Sao.menu(preferences);
                                 Sao.user_menu(preferences);
-                                Sao.open_url();
+                                Sao.open_url(url);
                                 Sao.Bus.listen();
                             });
                     }

package/src/sao.less

@@ -814,6 +814,7 @@ img.icon {
             }
         }
         > thead > tr:first-child {
+            background-color: @body-bg;
             position: sticky;
             top: 0;
             z-index: 100;
@@ -821,7 +822,6 @@ img.icon {
             th {
                 border: 1px solid transparent;
                 border-bottom: 2px solid @table-border-color;
-                background-color: @body-bg;
             }
 
             &:hover th {
@@ -830,12 +830,12 @@ img.icon {
             }
         }
         > tfoot > tr {
+            background-color: @body-bg;
             bottom: 0;
             position: sticky;
             z-index: 100;
 
             th {
-                background-color: @body-bg;
                 & > label {
                     display: none;
                 }
@@ -1131,9 +1131,11 @@ img.icon {
     }
     .form-text, .form-richtext {
         display: flex;
+        flex-direction: column;
 
         .input-group {
             display: flex;
+            height: 100%;
             width: 100%;
 
             textarea, .richtext {
@@ -1142,6 +1144,7 @@ img.icon {
                 min-height: 12.5ex;
                 overflow: auto;
                 resize: vertical;
+                width: 100%;
             }
 
             > span {
@@ -1153,6 +1156,9 @@ img.icon {
         > .btn-toolbar {
             min-width: 450px;
         }
+        > .panel-body {
+            flex-grow: 2;
+        }
     }
     .form-separator {
         label {
@@ -1177,6 +1183,7 @@ img.icon {
             max-width: 100%;
             min-height: 50vh;
             height: 100%;
+            width: 100%;
             @media screen and (max-width: @screen-xs-max) {
                 min-height: 25vh;
             }

package/src/tab.js

@@ -861,15 +861,20 @@
                             case 'ok':
                                 return this.save();
                             case 'ko':
-                                var record_id = this.screen.current_record.id;
+                                var record_id = null;
+                                if (this.screen.current_record) {
+                                    record_id = this.screen.current_record.id;
+                                }
                                 return this.reload(false).then(() => {
-                                    if (record_id < 0) {
-                                        return jQuery.Deferred().reject(true);
-                                    }
-                                    else if (this.screen.current_record) {
-                                        if (record_id !=
-                                            this.screen.current_record.id) {
-                                            return jQuery.Deferred().reject();
+                                    if (record_id !== null) {
+                                        if (record_id < 0) {
+                                            return jQuery.Deferred().reject(true);
+                                        }
+                                        else if (this.screen.current_record) {
+                                            if (record_id !=
+                                                this.screen.current_record.id) {
+                                                return jQuery.Deferred().reject();
+                                            }
                                         }
                                     }
                                 });

package/src/view/form.js

@@ -1423,7 +1423,8 @@ function eval_pyson(value){
         class_: 'form',
         init: function(languages, widget) {
             var dialog = new Sao.Dialog(
-                Sao.i18n.gettext('Translate'), this.class_, 'md');
+                Sao.i18n.gettext('Translate'), this.class_,
+                widget.expand? 'lg' : 'md');
             this.languages = languages;
             this.read(widget, dialog);
             jQuery('<button/>', {
@@ -1514,10 +1515,10 @@ function eval_pyson(value){
                 input.uniqueId();
                 row.append(jQuery('<label/>', {
                     'for': input.attr('id'),
-                    'class': 'col-sm-3 control-label',
+                    'class': 'col-sm-2 control-label',
                 }).append(' ' + lang.name));
                 row.append(jQuery('<div/>', {
-                    'class': 'col-sm-9 input-group',
+                    'class': 'col-sm-10 input-group',
                 }).append(input)
                     .append(jQuery('<span/>', {
                         'class': 'input-group-addon',
@@ -2589,9 +2590,11 @@ function eval_pyson(value){
             jQuery('<div/>', {
                 'class': 'richtext mousetrap',
                 'contenteditable': true
+            }).appendTo(jQuery('<div/>', {
+                'class': 'input-group',
             }).appendTo(jQuery('<div/>', {
                 'class': 'panel-body'
-            }).appendTo(widget));
+            }).appendTo(widget)));
             return widget;
         },
         translate_widget_set_readonly: function(el, value) {

package/src/view/tree.js

@@ -1063,6 +1063,8 @@
                     column.col.show();
                 }
             }
+            this.table.find('thead > tr > th .resizer').show();
+            this.table.find('thead > tr > th:visible:last .resizer').hide();
             if (this.children_field) {
                 this.columns.every(column => {
                     if (column.col.hasClass('draggable-handle') ||
@@ -1118,7 +1120,8 @@
                         'type': 'button',
                         'title': Sao.i18n.gettext("More"),
                     }).text(Sao.i18n.gettext('More')
-                    ).click(() => {
+                    ).one('click', () => {
+                        this.tbody.find('tr.more-row').remove();
                         var height = this.table.height();
                         this.display_size += Sao.config.display_size;
                         this.display();

package/src/window.js

@@ -416,7 +416,9 @@
                 }
             }
             let message = name + '/' + Sao.common.humanize(size);
-            this.label.text(message).attr('title', message);
+            if (this.label) {
+                this.label.text(message).attr('title', message);
+            }
             this._set_button_sensitive();
         },
         record_modified: function() {