tryton-sao 7.4.21 → 7.4.23

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 7.4.23 - 2025-12-20
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 7.4.22 - 2025-12-17
+---------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 7.4.21 - 2025-11-21
 ---------------------------
 * 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.21',
+    __version__: '7.4.23',
 };
 /* eslint-enable no-redeclare */
 
@@ -4606,11 +4606,12 @@ var Sao = {
                 var name = clause[0];
                 var operator = clause[1];
                 var value = clause[2];
-                if (name.endsWith('.rec_name')) {
+                if (name.endsWith('.rec_name')
+                    && (value || (clause.length > 3))) {
                     name = name.slice(0, -9);
                 }
                 if (!(name in this.fields)) {
-                    if (this.is_full_text(value)) {
+                    if ((value !== null) && this.is_full_text(value)) {
                         value = value.slice(1, -1);
                     }
                     return this.quote(value);
@@ -5532,7 +5533,7 @@ var Sao = {
                         break;
                     }
                 }
-                return target + ',' + value;
+                return target + ',' + (value || '');
             };
 
             var converts = {
@@ -6955,7 +6956,10 @@ var Sao = {
             }).appendTo(dialog.body);
             alert_.append(jQuery('<h4/>')
                 .text(title)
-                .css('white-space', 'pre-wrap'));
+                .css({
+                    'white-space': 'pre-wrap',
+                    'word-break': 'break-all',
+                }));
             alert_.append(jQuery('<p/>').append(jQuery('<a/>', {
                 'class': 'btn btn-default',
                 role: 'button',
@@ -8775,7 +8779,7 @@ var Sao = {
                 this.on_change(fieldnames);
                 this.on_change_with(fieldnames);
                 if (validate) {
-                    return this.validate(null, true);
+                    this.validate(null, true);
                 }
                 if (modified) {
                     this.set_modified();
@@ -11157,7 +11161,7 @@ var Sao = {
             if (other.compare(attributes)) {
                 Sao.common.scrollIntoViewIfNeeded(
                     tablist.find('a[href="#' + other.id + '"]').tab('show'));
-                return;
+                return jQuery.when();
             }
         }
         var tab;
@@ -12258,6 +12262,9 @@ var Sao = {
             const view_type = this.screen.current_view.view_type;
             var next_view_type = this.screen.next_view_type;
             const has_views = this.screen.number_of_views > 1;
+            let sensitive = (
+                record_id >= 0 ? record_id !== null && record_id !== undefined :
+                false);
             var buttons = ['print', 'relate', 'email', 'attach'];
             for (const button_id of buttons) {
                 const button = this.buttons[button_id];
@@ -12274,7 +12281,7 @@ var Sao = {
                             return keyword == button_id;
                         });
                 }
-                set_sensitive(button_id, position && can_be_sensitive);
+                set_sensitive(button_id, sensitive && can_be_sensitive);
             }
             set_sensitive(
                 'switch_',
@@ -17132,7 +17139,7 @@ function eval_pyson(value){
         },
         get width() {
             var digits = this.digits;
-            if (digits) {
+            if (digits && digits.every(d => d !== null)) {
                 return digits.reduce(function(acc, cur) {
                     return acc + cur;
                 });
@@ -17764,15 +17771,21 @@ function eval_pyson(value){
             } else {
                 this._popup = true;
             }
+            let view_ids = (this.attributes.view_ids || '').split(',');
             if (this.has_target(value)) {
                 var m2o_id =
                     this.id_from_value(record.field_get(this.field_name));
                 if (evt && (evt.ctrlKey || evt.metaKey)) {
+                    if (!jQuery.isEmptyObject(view_ids)) {
+                        // Remove the first tree view as mode is form only
+                        view_ids.shift();
+                    }
                     var params = {};
                     params.model = this.get_model();
                     params.res_id = m2o_id;
                     params.mode = ['form'];
                     params.name = this.attributes.string;
+                    params.view_ids = view_ids;
                     params.context = this.field.get_context(this.record);
                     Sao.Tab.create(params);
                     this._popup = false;
@@ -17821,8 +17834,7 @@ function eval_pyson(value){
                             context: context,
                             domain: domain,
                             order: order,
-                            view_ids: (this.attributes.view_ids ||
-                                '').split(','),
+                            view_ids: view_ids,
                             views_preload: (this.attributes.views || {}),
                             new_: this.create_access,
                             search_filter: parser.quote(text),
@@ -22818,9 +22830,11 @@ function eval_pyson(value){
                 return;
             }
 
-            body = listener = jQuery(document.body);
+            body = jQuery(document.body);
             if (body.hasClass('modal-open')) {
                 listener = this.tree.el.parents('.modal').last();
+            } else {
+                listener = this.tree.el.parents('.tab-pane').last();
             }
             const handler = event_ => {
                 if ((event_.currentTarget == body[0]) &&
@@ -22833,7 +22847,7 @@ function eval_pyson(value){
                     event_.stopPropagation();
                     return;
                 }
-                body.off('click.sao.editabletree');
+                listener.off('click.sao.editabletree');
                 this.tree.edit_row(null);
                 return true;
             };
@@ -22962,7 +22976,10 @@ function eval_pyson(value){
                             }
                         }
                         this._get_column_td(next_column)
-                            .find(':input,[tabindex=0]').focus();
+                            .find(':input,[tabindex=0]')
+                            .filter(':visible')
+                            .first()
+                            .focus();
                     } else {
                         var prm = jQuery.when();
                         if (!this.tree.screen.group.parent) {
@@ -23011,7 +23028,10 @@ function eval_pyson(value){
                                 this._get_column_td(
                                     next_column, next_row)
                                     .trigger('click')
+                                    .trigger('click')
                                     .find(':input,[tabindex=0]')
+                                    .filter(':visible')
+                                    .first()
                                     .focus();
                             });
                         }

package/package.json

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

package/src/common.js

@@ -1257,11 +1257,12 @@
                 var name = clause[0];
                 var operator = clause[1];
                 var value = clause[2];
-                if (name.endsWith('.rec_name')) {
+                if (name.endsWith('.rec_name')
+                    && (value || (clause.length > 3))) {
                     name = name.slice(0, -9);
                 }
                 if (!(name in this.fields)) {
-                    if (this.is_full_text(value)) {
+                    if ((value !== null) && this.is_full_text(value)) {
                         value = value.slice(1, -1);
                     }
                     return this.quote(value);
@@ -2183,7 +2184,7 @@
                         break;
                     }
                 }
-                return target + ',' + value;
+                return target + ',' + (value || '');
             };
 
             var converts = {
@@ -3606,7 +3607,10 @@
             }).appendTo(dialog.body);
             alert_.append(jQuery('<h4/>')
                 .text(title)
-                .css('white-space', 'pre-wrap'));
+                .css({
+                    'white-space': 'pre-wrap',
+                    'word-break': 'break-all',
+                }));
             alert_.append(jQuery('<p/>').append(jQuery('<a/>', {
                 'class': 'btn btn-default',
                 role: 'button',

package/src/model.js

@@ -1021,7 +1021,7 @@
                 this.on_change(fieldnames);
                 this.on_change_with(fieldnames);
                 if (validate) {
-                    return this.validate(null, true);
+                    this.validate(null, true);
                 }
                 if (modified) {
                     this.set_modified();

package/src/sao.js

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

package/src/tab.js

@@ -414,7 +414,7 @@
             if (other.compare(attributes)) {
                 Sao.common.scrollIntoViewIfNeeded(
                     tablist.find('a[href="#' + other.id + '"]').tab('show'));
-                return;
+                return jQuery.when();
             }
         }
         var tab;
@@ -1515,6 +1515,9 @@
             const view_type = this.screen.current_view.view_type;
             var next_view_type = this.screen.next_view_type;
             const has_views = this.screen.number_of_views > 1;
+            let sensitive = (
+                record_id >= 0 ? record_id !== null && record_id !== undefined :
+                false);
             var buttons = ['print', 'relate', 'email', 'attach'];
             for (const button_id of buttons) {
                 const button = this.buttons[button_id];
@@ -1531,7 +1534,7 @@
                             return keyword == button_id;
                         });
                 }
-                set_sensitive(button_id, position && can_be_sensitive);
+                set_sensitive(button_id, sensitive && can_be_sensitive);
             }
             set_sensitive(
                 'switch_',

package/src/view/form.js

@@ -2170,7 +2170,7 @@ function eval_pyson(value){
         },
         get width() {
             var digits = this.digits;
-            if (digits) {
+            if (digits && digits.every(d => d !== null)) {
                 return digits.reduce(function(acc, cur) {
                     return acc + cur;
                 });
@@ -2802,15 +2802,21 @@ function eval_pyson(value){
             } else {
                 this._popup = true;
             }
+            let view_ids = (this.attributes.view_ids || '').split(',');
             if (this.has_target(value)) {
                 var m2o_id =
                     this.id_from_value(record.field_get(this.field_name));
                 if (evt && (evt.ctrlKey || evt.metaKey)) {
+                    if (!jQuery.isEmptyObject(view_ids)) {
+                        // Remove the first tree view as mode is form only
+                        view_ids.shift();
+                    }
                     var params = {};
                     params.model = this.get_model();
                     params.res_id = m2o_id;
                     params.mode = ['form'];
                     params.name = this.attributes.string;
+                    params.view_ids = view_ids;
                     params.context = this.field.get_context(this.record);
                     Sao.Tab.create(params);
                     this._popup = false;
@@ -2859,8 +2865,7 @@ function eval_pyson(value){
                             context: context,
                             domain: domain,
                             order: order,
-                            view_ids: (this.attributes.view_ids ||
-                                '').split(','),
+                            view_ids: view_ids,
                             views_preload: (this.attributes.views || {}),
                             new_: this.create_access,
                             search_filter: parser.quote(text),

package/src/view/tree.js

@@ -2026,9 +2026,11 @@
                 return;
             }
 
-            body = listener = jQuery(document.body);
+            body = jQuery(document.body);
             if (body.hasClass('modal-open')) {
                 listener = this.tree.el.parents('.modal').last();
+            } else {
+                listener = this.tree.el.parents('.tab-pane').last();
             }
             const handler = event_ => {
                 if ((event_.currentTarget == body[0]) &&
@@ -2041,7 +2043,7 @@
                     event_.stopPropagation();
                     return;
                 }
-                body.off('click.sao.editabletree');
+                listener.off('click.sao.editabletree');
                 this.tree.edit_row(null);
                 return true;
             };
@@ -2170,7 +2172,10 @@
                             }
                         }
                         this._get_column_td(next_column)
-                            .find(':input,[tabindex=0]').focus();
+                            .find(':input,[tabindex=0]')
+                            .filter(':visible')
+                            .first()
+                            .focus();
                     } else {
                         var prm = jQuery.when();
                         if (!this.tree.screen.group.parent) {
@@ -2219,7 +2224,10 @@
                                 this._get_column_td(
                                     next_column, next_row)
                                     .trigger('click')
+                                    .trigger('click')
                                     .find(':input,[tabindex=0]')
+                                    .filter(':visible')
+                                    .first()
                                     .focus();
                             });
                         }

package/tests/sao.js

@@ -2048,6 +2048,9 @@
         [[c(['Reference', null, 'Spam,bar'])], [
             c(['reference.rec_name', 'ilike', '%bar%', 'spam'])
             ]],
+        [[c(['Reference', null, 'Spam,'])], [
+            c(['reference.rec_name', 'ilike', '%', 'spam'])
+            ]],
         [[c(['Reference', null, ['foo', 'bar']])], [
             c(['reference', 'in', ['foo', 'bar']])
             ]],
@@ -2376,6 +2379,7 @@
         [[['multiselection', 'not in', ['foo', 'bar']]], "MultiSelection: !Foo;Bar"],
         [[['reference', 'ilike', '%foo%']], 'Reference: foo'],
         [[['reference', 'ilike', '%bar%', 'spam']], 'Reference: Spam,bar'],
+        [[['reference.rec_name', '=', null, 'spam']], 'Reference: =Spam,'],
         [[['reference', 'in', ['foo', 'bar']]], 'Reference: foo;bar'],
         [[['many2one', 'ilike', '%John%']], 'Many2One: John'],
         [[['many2one.rec_name', 'in', ['John', 'Jane']]], 'Many2One: John;Jane'],