tryton-sao 7.6.19 → 7.6.20

package/CHANGELOG

@@ -1,4 +1,9 @@
 
+Version 7.6.20 - 2026-05-02
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.6.19 - 2026-04-16
 ---------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.css

@@ -9687,12 +9687,10 @@ html.accesskey *[accesskey]:after {
   padding: 0 2px;
   position: absolute;
   text-transform: uppercase;
-  margin: 0 -1em;
+  left: -1em;
+  right: 0;
   z-index: 5;
 }
-html.accesskey input[accesskey][type="checkbox"]:after {
-  background-color: initial;
-}
 html.accesskey *[accesskey] ~ span[data-accesskey] {
   float: right;
   position: relative;
@@ -9703,10 +9701,13 @@ html.accesskey *[accesskey] ~ span[data-accesskey]:after {
 html.accesskey *[accesskey]:after {
   content: attr(accesskey);
 }
-html.accesskey input[accesskey][type="checkbox"] ~ span[data-accesskey],
-html.accesskey input[accesskey][type="checkbox"] ~ span[data-accesskey],
-html.accesskey select[accesskey] ~ span[data-accesskey] {
-  bottom: 1em;
+html.accesskey[dir="rtl"] *[accesskey] ~ span[data-accesskey]:after,
+html.accesskey[dir="rtl"] *[accesskey]:after {
+  left: 0;
+  right: -1em;
+}
+html.accesskey[dir="rtl"] *[accesskey] ~ span[data-accesskey] {
+  float: left;
 }
 .tab-form,
 .tab-board {

package/dist/tryton-sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.6.19',
+    __version__: '7.6.20',
 };
 /* eslint-enable no-redeclare */
 
@@ -8829,7 +8829,7 @@ var Sao = {
                                 this.group.parent.id;
                         }
                     }
-                    return this.set_default(values);
+                    return this.set_default(values).then(() => values);
                 });
             }
             return jQuery.when();
@@ -10866,6 +10866,7 @@ var Sao = {
                 'class': 'hidden-lg',
             }).appendTo(this.name_el);
             this.view_prm = jQuery.when();
+            this._action_running = false;
         },
         menu_def: function() {
             return [
@@ -11015,8 +11016,13 @@ var Sao = {
                     this.menu_buttons[item.id] = menuitem;
                     link.click(evt => {
                         evt.preventDefault();
-                        if (!menuitem.hasClass('disabled')) {
-                            this[item.id]();
+                        if (!menuitem.hasClass('disabled')
+                            && !this._action_running) {
+                            this._action_running = true;
+                            (this[item.id]() || jQuery.when())
+                                .always(() => {
+                                    this._action_running = false;
+                                });
                         }
                     });
                 } else if (!item && previous) {
@@ -11120,17 +11126,14 @@ var Sao = {
                 }
                 this.buttons[item.id].click(item, event => {
                     var item = event.data;
-                    var button = this.buttons[item.id];
-                    // Use data instead of disabled prop because the action may
-                    // actually disable the button.
-                    if (button.data('disabled')) {
+                    if (this._action_running) {
                         event.preventDefault();
                         return;
                     }
-                    button.data('disabled', true);
+                    this._action_running = true;
                     (this[item.id](this) || jQuery.when())
-                        .always(function() {
-                            button.data('disabled', false);
+                        .always(() => {
+                            this._action_running = false;
                         });
                 });
             };
@@ -11757,7 +11760,7 @@ var Sao = {
             } else {
                 prm = jQuery.when();
             }
-            prm.then(() => {
+            return prm.then(() => {
                 var access = Sao.common.MODELACCESS.get(this.screen.model_name);
                 if (this.screen.readonly || !(access.write || access.create)) {
                     return jQuery.Deferred().reject();
@@ -11859,18 +11862,18 @@ var Sao = {
         },
         previous: function() {
             return this.modified_save().then(() => {
-                var prm = this.screen.display_previous();
-                this.info_bar.clear();
-                this.set_buttons_sensitive();
-                return prm;
+                return this.screen.display_previous().then(() => {
+                    this.info_bar.clear();
+                    this.set_buttons_sensitive();
+                });
             });
         },
         next: function() {
             return this.modified_save().then(() => {
-                var prm = this.screen.display_next();
-                this.info_bar.clear();
-                this.set_buttons_sensitive();
-                return prm;
+                return this.screen.display_next().then(() => {
+                    this.info_bar.clear();
+                    this.set_buttons_sensitive();
+                });
             });
         },
         search: function() {
@@ -15567,7 +15570,8 @@ function eval_pyson(value){
                 }
             }
             return jQuery.when.apply(jQuery,promesses)
-                .done(() => {
+                .then(() => {
+                    let promesses = [];
                     var record = this.record;
                     for (const name in this.widgets) {
                         var widgets = this.widgets[name];
@@ -15579,10 +15583,12 @@ function eval_pyson(value){
                             field.set_state(record);
                         }
                         for (const widget of widgets) {
-                            widget.display();
+                            let prm = widget.display();
+                            if (prm) {
+                                promesses.push(prm);
+                            }
                         }
                     }
-                    var promesses = [];
                     // We iterate in the reverse order so that the most nested
                     // widgets are computed first and set_state methods can rely
                     // on their children having their state set
@@ -15597,7 +15603,7 @@ function eval_pyson(value){
                     }
                     // re-set the grid templates for the StateWidget that are
                     // asynchronous
-                    jQuery.when.apply(jQuery, promesses).done(() => {
+                    return jQuery.when.apply(jQuery, promesses).then(() => {
                         for (const container of this.containers) {
                             container.set_grid_template();
                         }
@@ -17432,11 +17438,13 @@ function eval_pyson(value){
             this.el = jQuery('<div/>', {
                 'class': this.class_
             });
+            this.group = jQuery('<div/>', {
+                'class': 'input-group input-group-sm'
+            }).css('width', '100%').appendTo(this.el);
             this.select = this.labelled = jQuery('<select/>', {
                 'class': 'form-control input-sm mousetrap',
                 'name': attributes.name,
-            });
-            this.el.append(this.select);
+            }).appendTo(this.group);
             this.select.change(this.focus_out.bind(this));
             Sao.common.selection_mixin.init.call(this);
             this.init_selection();
@@ -17537,11 +17545,14 @@ function eval_pyson(value){
             this.el = jQuery('<div/>', {
                 'class': this.class_
             });
+            this.group = jQuery('<div/>', {
+                'class': 'input-group input-group-sm'
+            }).css('width', '100%').appendTo(this.el);
             this.input = this.labelled = jQuery('<input/>', {
                 'type': 'checkbox',
                 'class': 'form-control input-sm mousetrap',
                 'name': attributes.name,
-            }).appendTo(this.el);
+            }).appendTo(this.group);
             this.input.change(this.focus_out.bind(this));
             this.input.click(function() {
                 // Dont trigger click if field is readonly as readonly has no
@@ -18804,7 +18815,7 @@ function eval_pyson(value){
         display: function() {
             Sao.View.Form.One2Many._super.display.call(this);
 
-            this.prm.done(() => {
+            return this.prm.then(() => {
                 this._set_button_sensitive();
 
                 var record = this.record;
@@ -18814,9 +18825,8 @@ function eval_pyson(value){
                     this.screen.new_group();
                     this.screen.current_record = null;
                     this.screen.group.parent = null;
-                    this.screen.display();
                     this.screen.screen_container.hide_filter();
-                    return;
+                    return this.screen.display();
                 }
 
                 var new_group = record.field_get_client(this.field_name);
@@ -18845,13 +18855,13 @@ function eval_pyson(value){
                     this.screen.domain = domain;
                 }
                 this.screen.size_limit = size_limit;
-                this.screen.display();
                 if (this.attributes.height !== undefined) {
                     this.content
                         .find('.treeview,.list-form').first()
                         .css('min-height', this.attributes.height + 'px')
                         .css('max-height', this.attributes.height + 'px');
                 }
+                return this.screen.display();
             });
         },
         focus: function() {
@@ -19151,9 +19161,7 @@ function eval_pyson(value){
             return prm;
         },
         set_value: function() {
-            if (this.screen.modified()) {  // TODO check if required
-                this.view.screen.record_modified(false);
-            }
+            this.screen.current_view.set_value();
         },
         _update_completion: function(text) {
             if (!this.record) {
@@ -19387,7 +19395,7 @@ function eval_pyson(value){
         display: function() {
             Sao.View.Form.Many2Many._super.display.call(this);
 
-            this.prm.done(() => {
+            return this.prm.then(() => {
                 var record = this.record;
                 var field = this.field;
 
@@ -19395,21 +19403,20 @@ function eval_pyson(value){
                     this.screen.new_group();
                     this.screen.current_record = null;
                     this.screen.group.parent = null;
-                    this.screen.display();
                     this.screen.screen_container.hide_filter();
-                    return;
+                    return this.screen.display();
                 }
                 var new_group = record.field_get_client(this.field_name);
                 if (new_group != this.screen.group) {
                     this.screen.set_group(new_group);
                 }
-                this.screen.display();
                 if (this.attributes.height !== undefined) {
                     this.content
                         .find('.treeview,.list-form').first()
                         .css('min-height', this.attributes.height + 'px')
                         .css('max-height', this.attributes.height + 'px');
                 }
+                return this.screen.display();
             });
         },
         focus: function() {
@@ -23847,6 +23854,10 @@ function eval_pyson(value){
                 fields, false, false)) {
                 var value = cell.prop('checked');
                 this.field.set_client(record, value);
+                if (record !== current_record) {
+                    // we can not rely on editable tree handler to save the row
+                    record.save();
+                }
             } else {
                 evt.preventDefault();
             }

package/package.json

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

package/src/model.js

@@ -989,7 +989,7 @@
                                 this.group.parent.id;
                         }
                     }
-                    return this.set_default(values);
+                    return this.set_default(values).then(() => values);
                 });
             }
             return jQuery.when();

package/src/sao.js

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

package/src/sao.less

@@ -515,14 +515,11 @@ html.accesskey {
         padding: 0 2px;
         position: absolute;
         text-transform: uppercase;
-        margin: 0 -1em;
+        left: -1em;
+        right: 0;
         z-index: 5;
     }
 
-    input[accesskey][type="checkbox"]:after {
-        background-color: initial;
-    }
-
     *[accesskey] ~ span[data-accesskey] {
         float: right;
         position: relative;
@@ -536,11 +533,15 @@ html.accesskey {
         content: attr(accesskey);
     }
 
-    input[accesskey][type="checkbox"],
-    input[accesskey][type="checkbox"],
-    select[accesskey] {
-        & ~ span[data-accesskey] {
-            bottom: 1em;
+    &[dir="rtl"] {
+        *[accesskey] ~ span[data-accesskey]:after,
+        *[accesskey]:after {
+            left: 0;
+            right: -1em;
+        }
+
+        *[accesskey] ~ span[data-accesskey] {
+            float: left;
         }
     }
 }

package/src/tab.js

@@ -18,6 +18,7 @@
                 'class': 'hidden-lg',
             }).appendTo(this.name_el);
             this.view_prm = jQuery.when();
+            this._action_running = false;
         },
         menu_def: function() {
             return [
@@ -167,8 +168,13 @@
                     this.menu_buttons[item.id] = menuitem;
                     link.click(evt => {
                         evt.preventDefault();
-                        if (!menuitem.hasClass('disabled')) {
-                            this[item.id]();
+                        if (!menuitem.hasClass('disabled')
+                            && !this._action_running) {
+                            this._action_running = true;
+                            (this[item.id]() || jQuery.when())
+                                .always(() => {
+                                    this._action_running = false;
+                                });
                         }
                     });
                 } else if (!item && previous) {
@@ -272,17 +278,14 @@
                 }
                 this.buttons[item.id].click(item, event => {
                     var item = event.data;
-                    var button = this.buttons[item.id];
-                    // Use data instead of disabled prop because the action may
-                    // actually disable the button.
-                    if (button.data('disabled')) {
+                    if (this._action_running) {
                         event.preventDefault();
                         return;
                     }
-                    button.data('disabled', true);
+                    this._action_running = true;
                     (this[item.id](this) || jQuery.when())
-                        .always(function() {
-                            button.data('disabled', false);
+                        .always(() => {
+                            this._action_running = false;
                         });
                 });
             };
@@ -909,7 +912,7 @@
             } else {
                 prm = jQuery.when();
             }
-            prm.then(() => {
+            return prm.then(() => {
                 var access = Sao.common.MODELACCESS.get(this.screen.model_name);
                 if (this.screen.readonly || !(access.write || access.create)) {
                     return jQuery.Deferred().reject();
@@ -1011,18 +1014,18 @@
         },
         previous: function() {
             return this.modified_save().then(() => {
-                var prm = this.screen.display_previous();
-                this.info_bar.clear();
-                this.set_buttons_sensitive();
-                return prm;
+                return this.screen.display_previous().then(() => {
+                    this.info_bar.clear();
+                    this.set_buttons_sensitive();
+                });
             });
         },
         next: function() {
             return this.modified_save().then(() => {
-                var prm = this.screen.display_next();
-                this.info_bar.clear();
-                this.set_buttons_sensitive();
-                return prm;
+                return this.screen.display_next().then(() => {
+                    this.info_bar.clear();
+                    this.set_buttons_sensitive();
+                });
             });
         },
         search: function() {

package/src/view/form.js

@@ -382,7 +382,8 @@ function eval_pyson(value){
                 }
             }
             return jQuery.when.apply(jQuery,promesses)
-                .done(() => {
+                .then(() => {
+                    let promesses = [];
                     var record = this.record;
                     for (const name in this.widgets) {
                         var widgets = this.widgets[name];
@@ -394,10 +395,12 @@ function eval_pyson(value){
                             field.set_state(record);
                         }
                         for (const widget of widgets) {
-                            widget.display();
+                            let prm = widget.display();
+                            if (prm) {
+                                promesses.push(prm);
+                            }
                         }
                     }
-                    var promesses = [];
                     // We iterate in the reverse order so that the most nested
                     // widgets are computed first and set_state methods can rely
                     // on their children having their state set
@@ -412,7 +415,7 @@ function eval_pyson(value){
                     }
                     // re-set the grid templates for the StateWidget that are
                     // asynchronous
-                    jQuery.when.apply(jQuery, promesses).done(() => {
+                    return jQuery.when.apply(jQuery, promesses).then(() => {
                         for (const container of this.containers) {
                             container.set_grid_template();
                         }
@@ -2247,11 +2250,13 @@ function eval_pyson(value){
             this.el = jQuery('<div/>', {
                 'class': this.class_
             });
+            this.group = jQuery('<div/>', {
+                'class': 'input-group input-group-sm'
+            }).css('width', '100%').appendTo(this.el);
             this.select = this.labelled = jQuery('<select/>', {
                 'class': 'form-control input-sm mousetrap',
                 'name': attributes.name,
-            });
-            this.el.append(this.select);
+            }).appendTo(this.group);
             this.select.change(this.focus_out.bind(this));
             Sao.common.selection_mixin.init.call(this);
             this.init_selection();
@@ -2352,11 +2357,14 @@ function eval_pyson(value){
             this.el = jQuery('<div/>', {
                 'class': this.class_
             });
+            this.group = jQuery('<div/>', {
+                'class': 'input-group input-group-sm'
+            }).css('width', '100%').appendTo(this.el);
             this.input = this.labelled = jQuery('<input/>', {
                 'type': 'checkbox',
                 'class': 'form-control input-sm mousetrap',
                 'name': attributes.name,
-            }).appendTo(this.el);
+            }).appendTo(this.group);
             this.input.change(this.focus_out.bind(this));
             this.input.click(function() {
                 // Dont trigger click if field is readonly as readonly has no
@@ -3619,7 +3627,7 @@ function eval_pyson(value){
         display: function() {
             Sao.View.Form.One2Many._super.display.call(this);
 
-            this.prm.done(() => {
+            return this.prm.then(() => {
                 this._set_button_sensitive();
 
                 var record = this.record;
@@ -3629,9 +3637,8 @@ function eval_pyson(value){
                     this.screen.new_group();
                     this.screen.current_record = null;
                     this.screen.group.parent = null;
-                    this.screen.display();
                     this.screen.screen_container.hide_filter();
-                    return;
+                    return this.screen.display();
                 }
 
                 var new_group = record.field_get_client(this.field_name);
@@ -3660,13 +3667,13 @@ function eval_pyson(value){
                     this.screen.domain = domain;
                 }
                 this.screen.size_limit = size_limit;
-                this.screen.display();
                 if (this.attributes.height !== undefined) {
                     this.content
                         .find('.treeview,.list-form').first()
                         .css('min-height', this.attributes.height + 'px')
                         .css('max-height', this.attributes.height + 'px');
                 }
+                return this.screen.display();
             });
         },
         focus: function() {
@@ -3966,9 +3973,7 @@ function eval_pyson(value){
             return prm;
         },
         set_value: function() {
-            if (this.screen.modified()) {  // TODO check if required
-                this.view.screen.record_modified(false);
-            }
+            this.screen.current_view.set_value();
         },
         _update_completion: function(text) {
             if (!this.record) {
@@ -4202,7 +4207,7 @@ function eval_pyson(value){
         display: function() {
             Sao.View.Form.Many2Many._super.display.call(this);
 
-            this.prm.done(() => {
+            return this.prm.then(() => {
                 var record = this.record;
                 var field = this.field;
 
@@ -4210,21 +4215,20 @@ function eval_pyson(value){
                     this.screen.new_group();
                     this.screen.current_record = null;
                     this.screen.group.parent = null;
-                    this.screen.display();
                     this.screen.screen_container.hide_filter();
-                    return;
+                    return this.screen.display();
                 }
                 var new_group = record.field_get_client(this.field_name);
                 if (new_group != this.screen.group) {
                     this.screen.set_group(new_group);
                 }
-                this.screen.display();
                 if (this.attributes.height !== undefined) {
                     this.content
                         .find('.treeview,.list-form').first()
                         .css('min-height', this.attributes.height + 'px')
                         .css('max-height', this.attributes.height + 'px');
                 }
+                return this.screen.display();
             });
         },
         focus: function() {

package/src/view/tree.js

@@ -2744,6 +2744,10 @@
                 fields, false, false)) {
                 var value = cell.prop('checked');
                 this.field.set_client(record, value);
+                if (record !== current_record) {
+                    // we can not rely on editable tree handler to save the row
+                    record.save();
+                }
             } else {
                 evt.preventDefault();
             }