tryton-sao 7.0.31 → 7.0.33

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 7.0.33 - 2025-07-15
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 7.0.32 - 2025-07-01
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.0.31 - 2025-06-04
 ---------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.css

@@ -8777,6 +8777,11 @@ html[theme="default"] .radio input[type="radio"]:focus,
 html[theme="default"] .radio-inline input[type="radio"]:focus {
   outline: none;
 }
+html[theme="default"] input[type="radio"]:focus:after,
+html[theme="default"] .radio input[type="radio"]:focus:after,
+html[theme="default"] .radio-inline input[type="radio"]:focus:after {
+  border-color: #71bdc1 !important;
+}
 html[theme="default"] input[type="radio"]:before,
 html[theme="default"] .radio input[type="radio"]:before,
 html[theme="default"] .radio-inline input[type="radio"]:before,
@@ -8866,10 +8871,10 @@ html[theme="default"] .checkbox input[type="checkbox"]:focus,
 html[theme="default"] .checkbox-inline input[type="checkbox"]:focus {
   outline: none;
 }
-html[theme="default"] input[type="checkbox"]:focus:before,
-html[theme="default"] .checkbox input[type="checkbox"]:focus:before,
-html[theme="default"] .checkbox-inline input[type="checkbox"]:focus:before {
-  border-color: #267f82;
+html[theme="default"] input[type="checkbox"]:focus:after,
+html[theme="default"] .checkbox input[type="checkbox"]:focus:after,
+html[theme="default"] .checkbox-inline input[type="checkbox"]:focus:after {
+  border-color: #71bdc1 !important;
 }
 html[theme="default"] input[type="checkbox"]:before,
 html[theme="default"] .checkbox input[type="checkbox"]:before,
@@ -9201,7 +9206,8 @@ html[theme="default"] .carousel-caption h6 {
     float: none;
   }
 }
-.btn-primary .icon {
+.btn-primary .icon,
+.bg-primary .icon {
   filter: brightness(0) invert(1);
 }
 .panel-heading a {

package/dist/tryton-sao.js

@@ -3,7 +3,7 @@
 
 /* eslint-disable no-redeclare */
 var Sao = {
-    __version__: '7.0.31',
+    __version__: '7.0.33',
 };
 /* eslint-enable no-redeclare */
 
@@ -1136,20 +1136,26 @@ var Sao = {
                 shortcut: 'alt+shift+tab',
                 label: Sao.i18n.gettext('Previous tab'),
                 callback: function() {
-                    Sao.Tab.previous_tab();
+                    if (!jQuery('body').children('.modal').length) {
+                        Sao.Tab.previous_tab();
+                    }
                 },
             }, {
                 shortcut: 'alt+tab',
                 label: Sao.i18n.gettext('Next tab'),
                 callback: function() {
-                    Sao.Tab.next_tab();
+                    if (!jQuery('body').children('.modal').length) {
+                        Sao.Tab.next_tab();
+                    }
                 },
             }, {
                 shortcut: 'ctrl+k',
                 label: Sao.i18n.gettext('Global search'),
                 callback: function() {
-                    jQuery('#main_navbar:hidden').collapse('show');
-                    jQuery('#global-search-entry').focus();
+                    if (!jQuery('body').children('.modal').length) {
+                        jQuery('#main_navbar:hidden').collapse('show');
+                        jQuery('#global-search-entry').focus();
+                    }
                 },
             }, {
                 shortcut: 'f1',
@@ -5206,8 +5212,13 @@ var Sao = {
                 }
                 let { format } = new Intl.NumberFormat(
                     Sao.i18n.BC47(Sao.i18n.getlang()));
+                // use 10000 because some language (ex: es) add thousand
+                // separator only after 9999
+                let [, thousandSeparator] = /^10(.)?000/.exec(format(10000));
                 let [, decimalSign] = /^0(.)1$/.exec(format(0.1));
-                return Number(string.replace(decimalSign, '.'));
+                return Number(string
+                    .replace(new RegExp(thousandSeparator, 'g'), '')
+                    .replace(decimalSign, '.'));
             }
             var convert_selection = function() {
                 if (typeof value == 'string') {
@@ -11347,40 +11358,40 @@ var Sao = {
             });
         },
         modified_save: function() {
-            this.screen.save_tree_state();
-            this.screen.current_view.set_value();
-            if (this.screen.modified()) {
-                return Sao.common.sur_3b.run(
-                        Sao.i18n.gettext('This record has been modified\n' +
-                            'do you want to save it?'))
-                    .then(result => {
-                        switch(result) {
-                            case 'ok':
-                                return this.save();
-                            case 'ko':
-                                var record_id = null;
-                                if (this.screen.current_record) {
-                                    record_id = this.screen.current_record.id;
-                                }
-                                return this.reload(false).then(() => {
-                                    if (record_id !== null) {
-                                        if (record_id < 0) {
-                                            return jQuery.Deferred().reject(true);
-                                        }
-                                        else if (this.screen.current_record) {
-                                            if (record_id !=
-                                                this.screen.current_record.id) {
-                                                return jQuery.Deferred().reject();
+            return this.screen.save_tree_state().then(() => {
+                this.screen.current_view.set_value();
+                if (this.screen.modified()) {
+                    return Sao.common.sur_3b.run(
+                            Sao.i18n.gettext('This record has been modified\n' +
+                                'do you want to save it?'))
+                        .then(result => {
+                            switch(result) {
+                                case 'ok':
+                                    return this.save();
+                                case 'ko':
+                                    var record_id = null;
+                                    if (this.screen.current_record) {
+                                        record_id = this.screen.current_record.id;
+                                    }
+                                    return this.reload(false).then(() => {
+                                        if (record_id !== null) {
+                                            if (record_id < 0) {
+                                                return jQuery.Deferred().reject(true);
+                                            }
+                                            else if (this.screen.current_record) {
+                                                if (record_id !=
+                                                    this.screen.current_record.id) {
+                                                    return jQuery.Deferred().reject();
+                                                }
                                             }
                                         }
-                                    }
-                                });
-                            default:
-                                return jQuery.Deferred().reject();
-                        }
-                    });
-            }
-            return jQuery.when();
+                                    });
+                                default:
+                                    return jQuery.Deferred().reject();
+                            }
+                        });
+                }
+            });
         },
         new_: function() {
             if (!Sao.common.MODELACCESS.get(this.screen.model_name).create) {
@@ -11394,24 +11405,29 @@ var Sao = {
             });
         },
         save: function(tab) {
+            let prm;
             if (tab) {
                 // Called from button so we must save the tree state
-                this.screen.save_tree_state();
-            }
-            var access = Sao.common.MODELACCESS.get(this.screen.model_name);
-            if (this.screen.readonly || !(access.write || access.create)) {
-                return jQuery.Deferred().reject();
+                prm = this.screen.save_tree_state();
+            } else {
+                prm = jQuery.when();
             }
-            return this.screen.save_current().then(
-                () => {
-                    this.info_bar.add(
-                        Sao.i18n.gettext('Record saved.'), 'info');
-                    this.screen.count_tab_domain(true);
-                }, () => {
-                    this.info_bar.add(
-                        this.screen.invalid_message(), 'danger');
+            prm.then(() => {
+                var access = Sao.common.MODELACCESS.get(this.screen.model_name);
+                if (this.screen.readonly || !(access.write || access.create)) {
                     return jQuery.Deferred().reject();
-                });
+                }
+                return this.screen.save_current().then(
+                    () => {
+                        this.info_bar.add(
+                            Sao.i18n.gettext('Record saved.'), 'info');
+                        this.screen.count_tab_domain(true);
+                    }, () => {
+                        this.info_bar.add(
+                            this.screen.invalid_message(), 'danger');
+                        return jQuery.Deferred().reject();
+                    });
+            });
         },
         switch_: function() {
             return this.modified_save().then(() => this.screen.switch_view());
@@ -11450,8 +11466,7 @@ var Sao = {
             if (test_modified) {
                 return this.modified_save().then(reload);
             } else {
-                this.screen.save_tree_state(false);
-                return reload();
+                return this.screen.save_tree_state(false).then(reload);
             }
         },
         copy: function() {
@@ -18485,7 +18500,6 @@ function eval_pyson(value){
             return prm;
         },
         set_value: function() {
-            this.screen.save_tree_state();
             if (this.screen.modified()) {  // TODO check if required
                 this.view.screen.record_modified(false);
             }
@@ -22122,12 +22136,13 @@ function eval_pyson(value){
                     current_record = this.tree.screen.current_record;
                     this.tree.select_records(current_record, this.record);
                 } else {
+                    let selected = this.is_selected();
                     if (!(event_.ctrlKey || event_.metaKey) ||
                         this.tree.selection_mode ==
                         Sao.common.SELECTION_SINGLE) {
                         this.tree.select_records(null, null);
                     }
-                    this.set_selection(!this.is_selected());
+                    this.set_selection(!selected);
                 }
                 this.selection_changed();
                 if (current_record) {
@@ -22158,6 +22173,9 @@ function eval_pyson(value){
         },
         selection_changed: function() {
             var is_selected = this.is_selected();
+            if (this.tree.selection_mode == Sao.common.SELECTION_SINGLE) {
+                this.tree.select_records(null, null);
+            }
             this.set_selection(is_selected);
             if (is_selected) {
                 this.tree.select_changed(this.record);
@@ -22315,10 +22333,13 @@ function eval_pyson(value){
             }
             listener.on('click.sao.editabletree', handler);
 
-            Sao.View.Tree.RowEditable._super.select_row.call(this, event_);
-
-            if (!event_.shiftKey && !(event_.ctrlKey || event_.metaKey)) {
+            // do not call super when editing row because the selection must
+            // not be changed
+            if (!event_.shiftKey && !(event_.ctrlKey || event_.metaKey) &&
+                (this.record === current_record)) {
                 this.tree.edit_row(this);
+            } else {
+                Sao.View.Tree.RowEditable._super.select_row.call(this, event_);
             }
         },
         unset_editable: function() {
@@ -23291,6 +23312,7 @@ function eval_pyson(value){
             if (states.invisible || states.readonly) {
                 return;
             }
+            event.stopImmediatePropagation();
             button.el.prop('disabled', true);  // state will be reset at display
             var row = this.view.rows.find(function(row) {
                 return row.record == record;

package/dist/tryton-sao.min.css

@@ -8777,6 +8777,11 @@ html[theme="default"] .radio input[type="radio"]:focus,
 html[theme="default"] .radio-inline input[type="radio"]:focus {
   outline: none;
 }
+html[theme="default"] input[type="radio"]:focus:after,
+html[theme="default"] .radio input[type="radio"]:focus:after,
+html[theme="default"] .radio-inline input[type="radio"]:focus:after {
+  border-color: #71bdc1 !important;
+}
 html[theme="default"] input[type="radio"]:before,
 html[theme="default"] .radio input[type="radio"]:before,
 html[theme="default"] .radio-inline input[type="radio"]:before,
@@ -8866,10 +8871,10 @@ html[theme="default"] .checkbox input[type="checkbox"]:focus,
 html[theme="default"] .checkbox-inline input[type="checkbox"]:focus {
   outline: none;
 }
-html[theme="default"] input[type="checkbox"]:focus:before,
-html[theme="default"] .checkbox input[type="checkbox"]:focus:before,
-html[theme="default"] .checkbox-inline input[type="checkbox"]:focus:before {
-  border-color: #267f82;
+html[theme="default"] input[type="checkbox"]:focus:after,
+html[theme="default"] .checkbox input[type="checkbox"]:focus:after,
+html[theme="default"] .checkbox-inline input[type="checkbox"]:focus:after {
+  border-color: #71bdc1 !important;
 }
 html[theme="default"] input[type="checkbox"]:before,
 html[theme="default"] .checkbox input[type="checkbox"]:before,
@@ -9201,7 +9206,8 @@ html[theme="default"] .carousel-caption h6 {
     float: none;
   }
 }
-.btn-primary .icon {
+.btn-primary .icon,
+.bg-primary .icon {
   filter: brightness(0) invert(1);
 }
 .panel-heading a {