tryton-sao 7.4.8 → 7.4.10

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 7.4.10 - 2025-05-02
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 7.4.9 - 2025-04-26
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.4.8 - 2025-04-02
 --------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.4.8',
+    __version__: '7.4.10',
 };
 /* eslint-enable no-redeclare */
 
@@ -4165,7 +4165,7 @@ var Sao = {
                 });
             }
         };
-        this._selection_prm.done(_update_selection);
+        this._selection_prm.always(_update_selection);
     };
     Sao.common.selection_mixin.filter_selection = function(
             domain, record, field) {
@@ -5345,6 +5345,15 @@ var Sao = {
             if (!context) {
                 context = {};
             }
+            function atof(string) {
+                if (!string) {
+                    throw("empty string");
+                }
+                let { format } = new Intl.NumberFormat(
+                    Sao.i18n.BC47(Sao.i18n.getlang()));
+                let [, decimalSign] = /^0(.)1$/.exec(format(0.1));
+                return Number(string.replace(decimalSign, '.'));
+            }
             var convert_selection = function() {
                 if (typeof value == 'string') {
                     for (var i = 0; i < field.selection.length; i++) {
@@ -5376,8 +5385,12 @@ var Sao = {
                 },
                 'float': function() {
                     var factor = Number(field.factor || 1);
-                    var result = Number(value);
-                    if (isNaN(result) || value === '' || value === null) {
+                    try {
+                        var result = atof(value);
+                    } catch (e) {
+                        return null;
+                    }
+                    if (isNaN(result)) {
                         return null;
                     } else {
                         return result / factor;
@@ -5385,18 +5398,25 @@ var Sao = {
                 },
                 'integer': function() {
                     var factor = Number(field.factor || 1, 10);
-                    var result = parseInt(value, 10);
+                    try {
+                        var result = atof(value);
+                    } catch (e) {
+                        return null;
+                    }
                     if (isNaN(result)) {
                         return null;
                     } else {
-                        return result / factor;
+                        return parseInt(result / factor, 10);
                     }
                 },
                 'numeric': function() {
                     var factor = Number(field.factor || 1);
-                    var result = Number(value);
-                    if (isNaN(result.valueOf()) ||
-                            value === '' || value === null) {
+                    try {
+                        var result = atof(value);
+                    } catch (e) {
+                        return null;
+                    }
+                    if (isNaN(result)) {
                         return null;
                     } else {
                         return new Sao.Decimal(result / factor);
@@ -5451,8 +5471,7 @@ var Sao = {
                     return '';
                 }
                 var digit = 0;
-                var factor = Number(field.factor || 1);
-                var string = String(value * factor);
+                var string = String(value);
                 if (string.contains('e')) {
                     var exp = string.split('e')[1];
                     string = string.split('e')[0];
@@ -5461,7 +5480,14 @@ var Sao = {
                 if (string.contains('.')) {
                     digit += string.replace(/0+$/, '').split('.')[1].length;
                 }
-                return (value * factor).toFixed(digit);
+                var factor = Number(field.factor || 1);
+                digit -= Math.round(Math.log10(factor));
+                return (value * factor).toLocaleString(
+                    Sao.i18n.BC47(Sao.i18n.getlang()), {
+                        useGrouping: true,
+                        minimumFractionDigits: digit,
+                        maximumFractionDigits: digit,
+                    });
             };
             var format_selection = function() {
                 if (field.selection instanceof Array) {
@@ -7703,7 +7729,7 @@ var Sao = {
     'use strict';
 
     function get_x2m_sub_fields(f_attrs, prefix) {
-        if (f_attrs.visible && f_attrs.views) {
+        if (f_attrs.visible && !jQuery.isEmptyObject(f_attrs.views)) {
             // There's only one key but we don't know its value
             const [[, view],] = Object.entries(f_attrs.views);
 
@@ -8285,6 +8311,7 @@ var Sao = {
             this.autocompletion = {};
             this.exception = false;
             this.destroyed = false;
+            this._save_prm = jQuery.when();
         },
         get modified() {
             if (!jQuery.isEmptyObject(this.modified_fields)) {
@@ -8298,33 +8325,28 @@ var Sao = {
         },
         save: function(force_reload=false) {
             var context = this.get_context();
+            if (this._save_prm.state() == 'pending') {
+                return this._save_prm.then(() => this.save(force_reload));
+            }
             var prm = jQuery.when();
             if ((this.id < 0) || this.modified) {
                 var values = this.get();
-                try {
-                    // synchronous call to avoid multiple creation or save
-                    if (this.id < 0) {
-                        this.id = this.model.execute(
-                            'create', [[values]], context,  false)[0];
-
-                    } else {
-                        if (!jQuery.isEmptyObject(values)) {
-                            context._timestamp = this.get_timestamp();
-                            this.model.execute(
-                                'write', [[this.id], values], context, false);
-                        }
-                    }
-                } catch (e) {
-                    if (e.promise) {
-                        return e.then(() => this.save(force_reload));
-                    } else {
-                        return jQuery.Deferred().reject();
+                if (this.id < 0) {
+                    prm = this.model.execute('create', [[values]], context)
+                        .then(ids => this.id = ids[0]);
+                } else {
+                    if (!jQuery.isEmptyObject(values)) {
+                        context._timestamp = this.get_timestamp();
+                        prm = this.model.execute(
+                            'write', [[this.id], values], context);
                     }
                 }
-                this.cancel();
-                if (force_reload) {
-                    return this.reload();
-                }
+                prm = prm.then(() => {
+                    this.cancel();
+                    if (force_reload) {
+                        return this.reload();
+                    }
+                });
                 if (this.group) {
                     prm = prm.then(() => this.group.written(this.id));
                 }
@@ -8333,6 +8355,7 @@ var Sao = {
                 delete this.group.parent.modified_fields[this.group.child_name];
                 prm = prm.then(() => this.group.parent.save(force_reload));
             }
+            this._save_prm = prm;
             return prm;
         },
         reload: function(fields, async=true) {
@@ -10195,6 +10218,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) {
@@ -10204,9 +10228,9 @@ var Sao = {
                         }
                         const record2 = group.get(vals.id);
                         if (record2) {
-                            var vals_to_set = {};
                             for (var key in vals) {
-                                if (!(key in new_field_names)) {
+                                if (!Object.prototype.hasOwnProperty.call(
+                                    new_field_names, key)) {
                                     vals_to_set[key] = vals[key];
                                 }
                             }
@@ -10240,7 +10264,14 @@ var Sao = {
                         }
                         const record2 = group.get(vals.id);
                         if (record2) {
-                            record2.set_on_change(vals);
+                            let to_update = Object.fromEntries(
+                                Object.entries(vals).filter(
+                                    ([k, v]) => {
+                                        !Object.prototype.hasOwnProperty.call(
+                                            vals_to_set, k)
+                                    }
+                                ));
+                            record2.set_on_change(to_update);
                         }
                     }
                 }
@@ -13034,7 +13065,7 @@ var Sao = {
             Sao.ScreenContainer.BetweenDates._super.init.call(this, id);
             this.from.change(this._from_changed.bind(this));
         },
-        _get_value: function(entry, value) {
+        _get_value: function(entry) {
             return entry.find('input[type=text]').val();
         },
         _set_value: function(entry, value) {
@@ -13147,11 +13178,17 @@ var Sao = {
                 'class': 'form-control input-sm',
                 'type': 'number',
                 'step': 'any',
+                'lang': Sao.i18n.getlang(),
             }).appendTo(el);
             return entry;
         },
-        _get_value: function(entry, value) {
-            return entry.val();
+        _get_value: function(entry) {
+            let value = entry.val();
+            if (value) {
+                value = Number(value).toLocaleString(
+                    Sao.i18n.BC47(Sao.i18n.getlang()))
+            }
+            return value;
         },
         _set_value: function(entry, value) {
             return entry.val(value);
@@ -13705,9 +13742,6 @@ var Sao = {
             return this.__current_record;
         },
         set current_record(record) {
-            if ((this.__current_record === record) && record) {
-                return;
-            }
             this.__current_record = record;
             var pos = null;
             var record_id = null;
@@ -14015,7 +14049,6 @@ var Sao = {
         },
         save_current: function() {
             var current_record = this.current_record;
-            let new_record = current_record.id < 0;
             if (!current_record) {
                 if (this.current_view &&
                     (this.current_view.view_type == 'tree') &&
@@ -14026,6 +14059,7 @@ var Sao = {
                     return jQuery.when();
                 }
             }
+            let new_record = current_record.id < 0;
             if (this.current_view) {
                 this.current_view.set_value();
                 var fields = this.current_view.get_fields();
@@ -14696,7 +14730,8 @@ var Sao = {
                     return view.display(selected_nodes);
                 } else {
                     var record;
-                    if (!jQuery.isEmptyObject(selected_nodes)) {
+                    if (!jQuery.isEmptyObject(selected_nodes) &&
+                        !this.current_record) {
                         for (const id of selected_nodes[0]) {
                             const new_record = this.group.get(id);
                             if (!new_record) {
@@ -25227,9 +25262,10 @@ function eval_pyson(value){
             });
         },
         clear: function() {
-            var kinds = this.el.children().each(
-                (i, el) => jQuery(el).data('kind'));
-            new Set(kinds).forEach(kind => {
+            let kinds = new Set();
+            this.el.children().each(
+                (i, el) => kinds.add(jQuery(el).data('kind')));
+            kinds.forEach(kind => {
                 this.refresh(kind);
             });
             this.__messages.clear();

package/package.json

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

package/src/common.js

@@ -833,7 +833,7 @@
                 });
             }
         };
-        this._selection_prm.done(_update_selection);
+        this._selection_prm.always(_update_selection);
     };
     Sao.common.selection_mixin.filter_selection = function(
             domain, record, field) {
@@ -2013,6 +2013,15 @@
             if (!context) {
                 context = {};
             }
+            function atof(string) {
+                if (!string) {
+                    throw("empty string");
+                }
+                let { format } = new Intl.NumberFormat(
+                    Sao.i18n.BC47(Sao.i18n.getlang()));
+                let [, decimalSign] = /^0(.)1$/.exec(format(0.1));
+                return Number(string.replace(decimalSign, '.'));
+            }
             var convert_selection = function() {
                 if (typeof value == 'string') {
                     for (var i = 0; i < field.selection.length; i++) {
@@ -2044,8 +2053,12 @@
                 },
                 'float': function() {
                     var factor = Number(field.factor || 1);
-                    var result = Number(value);
-                    if (isNaN(result) || value === '' || value === null) {
+                    try {
+                        var result = atof(value);
+                    } catch (e) {
+                        return null;
+                    }
+                    if (isNaN(result)) {
                         return null;
                     } else {
                         return result / factor;
@@ -2053,18 +2066,25 @@
                 },
                 'integer': function() {
                     var factor = Number(field.factor || 1, 10);
-                    var result = parseInt(value, 10);
+                    try {
+                        var result = atof(value);
+                    } catch (e) {
+                        return null;
+                    }
                     if (isNaN(result)) {
                         return null;
                     } else {
-                        return result / factor;
+                        return parseInt(result / factor, 10);
                     }
                 },
                 'numeric': function() {
                     var factor = Number(field.factor || 1);
-                    var result = Number(value);
-                    if (isNaN(result.valueOf()) ||
-                            value === '' || value === null) {
+                    try {
+                        var result = atof(value);
+                    } catch (e) {
+                        return null;
+                    }
+                    if (isNaN(result)) {
                         return null;
                     } else {
                         return new Sao.Decimal(result / factor);
@@ -2119,8 +2139,7 @@
                     return '';
                 }
                 var digit = 0;
-                var factor = Number(field.factor || 1);
-                var string = String(value * factor);
+                var string = String(value);
                 if (string.contains('e')) {
                     var exp = string.split('e')[1];
                     string = string.split('e')[0];
@@ -2129,7 +2148,14 @@
                 if (string.contains('.')) {
                     digit += string.replace(/0+$/, '').split('.')[1].length;
                 }
-                return (value * factor).toFixed(digit);
+                var factor = Number(field.factor || 1);
+                digit -= Math.round(Math.log10(factor));
+                return (value * factor).toLocaleString(
+                    Sao.i18n.BC47(Sao.i18n.getlang()), {
+                        useGrouping: true,
+                        minimumFractionDigits: digit,
+                        maximumFractionDigits: digit,
+                    });
             };
             var format_selection = function() {
                 if (field.selection instanceof Array) {

package/src/model.js

@@ -4,7 +4,7 @@
     'use strict';
 
     function get_x2m_sub_fields(f_attrs, prefix) {
-        if (f_attrs.visible && f_attrs.views) {
+        if (f_attrs.visible && !jQuery.isEmptyObject(f_attrs.views)) {
             // There's only one key but we don't know its value
             const [[, view],] = Object.entries(f_attrs.views);
 
@@ -586,6 +586,7 @@
             this.autocompletion = {};
             this.exception = false;
             this.destroyed = false;
+            this._save_prm = jQuery.when();
         },
         get modified() {
             if (!jQuery.isEmptyObject(this.modified_fields)) {
@@ -599,33 +600,28 @@
         },
         save: function(force_reload=false) {
             var context = this.get_context();
+            if (this._save_prm.state() == 'pending') {
+                return this._save_prm.then(() => this.save(force_reload));
+            }
             var prm = jQuery.when();
             if ((this.id < 0) || this.modified) {
                 var values = this.get();
-                try {
-                    // synchronous call to avoid multiple creation or save
-                    if (this.id < 0) {
-                        this.id = this.model.execute(
-                            'create', [[values]], context,  false)[0];
-
-                    } else {
-                        if (!jQuery.isEmptyObject(values)) {
-                            context._timestamp = this.get_timestamp();
-                            this.model.execute(
-                                'write', [[this.id], values], context, false);
-                        }
-                    }
-                } catch (e) {
-                    if (e.promise) {
-                        return e.then(() => this.save(force_reload));
-                    } else {
-                        return jQuery.Deferred().reject();
+                if (this.id < 0) {
+                    prm = this.model.execute('create', [[values]], context)
+                        .then(ids => this.id = ids[0]);
+                } else {
+                    if (!jQuery.isEmptyObject(values)) {
+                        context._timestamp = this.get_timestamp();
+                        prm = this.model.execute(
+                            'write', [[this.id], values], context);
                     }
                 }
-                this.cancel();
-                if (force_reload) {
-                    return this.reload();
-                }
+                prm = prm.then(() => {
+                    this.cancel();
+                    if (force_reload) {
+                        return this.reload();
+                    }
+                });
                 if (this.group) {
                     prm = prm.then(() => this.group.written(this.id));
                 }
@@ -634,6 +630,7 @@
                 delete this.group.parent.modified_fields[this.group.child_name];
                 prm = prm.then(() => this.group.parent.save(force_reload));
             }
+            this._save_prm = prm;
             return prm;
         },
         reload: function(fields, async=true) {
@@ -2496,6 +2493,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) {
@@ -2505,9 +2503,9 @@
                         }
                         const record2 = group.get(vals.id);
                         if (record2) {
-                            var vals_to_set = {};
                             for (var key in vals) {
-                                if (!(key in new_field_names)) {
+                                if (!Object.prototype.hasOwnProperty.call(
+                                    new_field_names, key)) {
                                     vals_to_set[key] = vals[key];
                                 }
                             }
@@ -2541,7 +2539,14 @@
                         }
                         const record2 = group.get(vals.id);
                         if (record2) {
-                            record2.set_on_change(vals);
+                            let to_update = Object.fromEntries(
+                                Object.entries(vals).filter(
+                                    ([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.4.8',
+    __version__: '7.4.10',
 };
 /* eslint-enable no-redeclare */
 

package/src/screen.js

@@ -625,7 +625,7 @@
             Sao.ScreenContainer.BetweenDates._super.init.call(this, id);
             this.from.change(this._from_changed.bind(this));
         },
-        _get_value: function(entry, value) {
+        _get_value: function(entry) {
             return entry.find('input[type=text]').val();
         },
         _set_value: function(entry, value) {
@@ -738,11 +738,17 @@
                 'class': 'form-control input-sm',
                 'type': 'number',
                 'step': 'any',
+                'lang': Sao.i18n.getlang(),
             }).appendTo(el);
             return entry;
         },
-        _get_value: function(entry, value) {
-            return entry.val();
+        _get_value: function(entry) {
+            let value = entry.val();
+            if (value) {
+                value = Number(value).toLocaleString(
+                    Sao.i18n.BC47(Sao.i18n.getlang()))
+            }
+            return value;
         },
         _set_value: function(entry, value) {
             return entry.val(value);
@@ -1296,9 +1302,6 @@
             return this.__current_record;
         },
         set current_record(record) {
-            if ((this.__current_record === record) && record) {
-                return;
-            }
             this.__current_record = record;
             var pos = null;
             var record_id = null;
@@ -1606,7 +1609,6 @@
         },
         save_current: function() {
             var current_record = this.current_record;
-            let new_record = current_record.id < 0;
             if (!current_record) {
                 if (this.current_view &&
                     (this.current_view.view_type == 'tree') &&
@@ -1617,6 +1619,7 @@
                     return jQuery.when();
                 }
             }
+            let new_record = current_record.id < 0;
             if (this.current_view) {
                 this.current_view.set_value();
                 var fields = this.current_view.get_fields();
@@ -2287,7 +2290,8 @@
                     return view.display(selected_nodes);
                 } else {
                     var record;
-                    if (!jQuery.isEmptyObject(selected_nodes)) {
+                    if (!jQuery.isEmptyObject(selected_nodes) &&
+                        !this.current_record) {
                         for (const id of selected_nodes[0]) {
                             const new_record = this.group.get(id);
                             if (!new_record) {

package/src/window.js

@@ -101,9 +101,10 @@
             });
         },
         clear: function() {
-            var kinds = this.el.children().each(
-                (i, el) => jQuery(el).data('kind'));
-            new Set(kinds).forEach(kind => {
+            let kinds = new Set();
+            this.el.children().each(
+                (i, el) => kinds.add(jQuery(el).data('kind')));
+            kinds.forEach(kind => {
                 this.refresh(kind);
             });
             this.__messages.clear();