tryton-sao 7.8.8 → 7.8.9

package/CHANGELOG

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

package/dist/tryton-sao.css

@@ -9758,12 +9758,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;
@@ -9774,10 +9772,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.8.8',
+    __version__: '7.8.9',
 };
 /* eslint-enable no-redeclare */
 
@@ -8796,7 +8796,7 @@ var Sao = {
                                 this.group.parent.id;
                         }
                     }
-                    return this.set_default(values);
+                    return this.set_default(values).then(() => values);
                 });
             }
             return jQuery.when();
@@ -10833,6 +10833,7 @@ var Sao = {
                 'class': 'hidden-lg',
             }).appendTo(this.name_el);
             this.view_prm = jQuery.when();
+            this._action_running = false;
         },
         menu_def: function() {
             return [
@@ -10982,8 +10983,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) {
@@ -11087,17 +11093,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;
                         });
                 });
             };
@@ -11724,7 +11727,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();
@@ -11826,18 +11829,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() {
@@ -15540,7 +15543,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];
@@ -15552,10 +15556,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
@@ -15570,7 +15576,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();
                         }
@@ -17407,11 +17413,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();
@@ -17512,11 +17520,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
@@ -18779,7 +18790,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;
@@ -18789,9 +18800,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);
@@ -18820,13 +18830,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() {
@@ -19126,9 +19136,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) {
@@ -19362,7 +19370,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;
 
@@ -19370,21 +19378,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() {
@@ -23843,6 +23850,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.8.8",
+  "version": "7.8.9",
   "homepage": "https://www.tryton.org/",
   "author": {
     "name": "Tryton"

package/src/model.js

@@ -990,7 +990,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.8.8',
+    __version__: '7.8.9',
 };
 /* eslint-enable no-redeclare */
 

package/src/sao.less

@@ -607,14 +607,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;
@@ -628,11 +625,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();
                         }
@@ -2249,11 +2252,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();
@@ -2354,11 +2359,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
@@ -3621,7 +3629,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;
@@ -3631,9 +3639,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);
@@ -3662,13 +3669,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() {
@@ -3968,9 +3975,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) {
@@ -4204,7 +4209,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;
 
@@ -4212,21 +4217,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

@@ -2762,6 +2762,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();
             }