tryton-sao 7.4.7 → 7.4.9

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+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)
+
+
 Version 7.4.7 - 2025-03-15
 --------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.css

@@ -10547,6 +10547,9 @@ input.column-boolean {
   float: none;
 }
 @media screen and (max-width: 767px) {
+  .form-dict-container {
+    grid-template-columns: 100%;
+  }
   .dict-row {
     grid-column: 1;
   }

package/dist/tryton-sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.4.7',
+    __version__: '7.4.9',
 };
 /* 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) {
@@ -7703,7 +7703,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 +8285,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 +8299,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 +8329,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) {
@@ -14015,7 +14012,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 +14022,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 +14693,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) {
@@ -20518,7 +20516,6 @@ function eval_pyson(value){
         _parse: Sao.common.parse_date,
         create_widget: function() {
             Sao.View.Form.Dict.Date._super.create_widget.call(this);
-            var group = this.input.parent().find('.input-group-btn');
             this.input_date = jQuery('<input/>', {
                 'type': this._input,
                 'role': 'button',
@@ -20539,12 +20536,15 @@ function eval_pyson(value){
                 }
             });
             if (this.input_date[0].type == this._input) {
-                var icon = jQuery('<button/>', {
-                    'class': 'btn btn-default',
-                    'type': 'button',
+                var group = jQuery('<div/>', {
+                    'class': 'input-icon input-icon-secondary',
+                }).prependTo(this.input.parent());
+                this.input.appendTo(group);
+                var icon = jQuery('<div/>', {
+                    'class': 'icon-input icon-secondary',
                     'aria-label': Sao.i18n.gettext("Open the calendar"),
                     'title': Sao.i18n.gettext("Open the calendar"),
-                }).prependTo(group);
+                }).appendTo(group);
                 this.input_date.appendTo(icon);
                 Sao.common.ICONFACTORY.get_icon_img('tryton-date')
                     .appendTo(icon);
@@ -20574,8 +20574,7 @@ function eval_pyson(value){
             return this._parse(this.format, this.input.val());
         },
         set_value: function(value) {
-            if ((value instanceof Sao.DateTime) ||
-                (value instanceof Sao.Date)) {
+            if (value && (value.isDate || value.isDateTime)) {
                 value = this._format(this.format, value);
             } else {
                 value = '';

package/package.json

@@ -2,7 +2,7 @@
   "name": "tryton-sao",
   "title": "sao",
   "description": "Tryton webclient",
-  "version": "7.4.7",
+  "version": "7.4.9",
   "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) {

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) {

package/src/sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.4.7',
+    __version__: '7.4.9',
 };
 /* eslint-enable no-redeclare */
 

package/src/sao.less

@@ -1194,6 +1194,9 @@ input.column-boolean {
 }
 
 @media screen and (max-width: @screen-xs-max) {
+    .form-dict-container {
+        grid-template-columns: 100%;
+    }
     .dict-row {
         grid-column: 1;
     }

package/src/screen.js

@@ -1606,7 +1606,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 +1616,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 +2287,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/view/form.js

@@ -5631,7 +5631,6 @@ function eval_pyson(value){
         _parse: Sao.common.parse_date,
         create_widget: function() {
             Sao.View.Form.Dict.Date._super.create_widget.call(this);
-            var group = this.input.parent().find('.input-group-btn');
             this.input_date = jQuery('<input/>', {
                 'type': this._input,
                 'role': 'button',
@@ -5652,12 +5651,15 @@ function eval_pyson(value){
                 }
             });
             if (this.input_date[0].type == this._input) {
-                var icon = jQuery('<button/>', {
-                    'class': 'btn btn-default',
-                    'type': 'button',
+                var group = jQuery('<div/>', {
+                    'class': 'input-icon input-icon-secondary',
+                }).prependTo(this.input.parent());
+                this.input.appendTo(group);
+                var icon = jQuery('<div/>', {
+                    'class': 'icon-input icon-secondary',
                     'aria-label': Sao.i18n.gettext("Open the calendar"),
                     'title': Sao.i18n.gettext("Open the calendar"),
-                }).prependTo(group);
+                }).appendTo(group);
                 this.input_date.appendTo(icon);
                 Sao.common.ICONFACTORY.get_icon_img('tryton-date')
                     .appendTo(icon);
@@ -5687,8 +5689,7 @@ function eval_pyson(value){
             return this._parse(this.format, this.input.val());
         },
         set_value: function(value) {
-            if ((value instanceof Sao.DateTime) ||
-                (value instanceof Sao.Date)) {
+            if (value && (value.isDate || value.isDateTime)) {
                 value = this._format(this.format, value);
             } else {
                 value = '';