tryton-sao 7.2.21 → 7.2.22

package/CHANGELOG

@@ -1,4 +1,9 @@
 
+Version 7.2.22 - 2025-06-04
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.2.21 - 2025-05-15
 ---------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.2.21',
+    __version__: '7.2.22',
 };
 /* eslint-enable no-redeclare */
 
@@ -660,6 +660,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;
@@ -669,7 +670,7 @@ var Sao = {
                             .then(function(preferences) {
                                 Sao.menu(preferences);
                                 Sao.user_menu(preferences);
-                                Sao.open_url();
+                                Sao.open_url(url);
                                 Sao.Bus.listen();
                             });
                     }
@@ -7656,16 +7657,19 @@ var Sao = {
             return null;
         }
         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);
@@ -9462,7 +9466,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);
@@ -9470,20 +9473,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);
@@ -9747,12 +9742,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) {
                     // 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);
             }

package/package.json

@@ -2,7 +2,7 @@
   "name": "tryton-sao",
   "title": "sao",
   "description": "Tryton webclient",
-  "version": "7.2.21",
+  "version": "7.2.22",
   "homepage": "http://www.tryton.org/",
   "author": {
     "name": "Tryton"

package/src/common.js

@@ -4360,16 +4360,19 @@
             return null;
         }
         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

@@ -1781,7 +1781,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);
@@ -1789,20 +1788,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);
@@ -2066,12 +2057,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) {
                     // 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);
             }

package/src/sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.2.21',
+    __version__: '7.2.22',
 };
 /* eslint-enable no-redeclare */
 
@@ -660,6 +660,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;
@@ -669,7 +670,7 @@ var Sao = {
                             .then(function(preferences) {
                                 Sao.menu(preferences);
                                 Sao.user_menu(preferences);
-                                Sao.open_url();
+                                Sao.open_url(url);
                                 Sao.Bus.listen();
                             });
                     }