tryton-sao 6.8.3 → 6.8.5

package/CHANGELOG

@@ -1,4 +1,14 @@
 
+Version 6.8.5 - 2023-12-01
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
+Version 6.8.4 - 2023-11-03
+--------------------------
+* Bug fixes (see mercurial logs for details)
+
+
 Version 6.8.3 - 2023-10-04
 --------------------------
 * Bug fixes (see mercurial logs for details)

package/dist/tryton-sao.css

@@ -9598,16 +9598,15 @@ img.icon {
 .treeview > table.tree col.optional {
   width: 30px;
 }
-.treeview > table.tree td.draggable-handle,
-.treeview > table.tree td.optional {
-  cursor: grab;
-}
 .treeview > table.tree td.draggable-handle > img,
 .treeview > table.tree td.optional > img {
   display: block;
   margin-right: auto;
   margin-left: auto;
 }
+.treeview > table.tree td.draggable-handle {
+  cursor: grab;
+}
 .treeview > table.tree th.selection-state > input,
 .treeview > table.tree td.selection-state > input {
   display: block;
@@ -9788,12 +9787,13 @@ img.icon {
 }
 .treeview .expander > img {
   float: left;
-  margin-right: 5px;
-  width: 1em;
+  margin-right: 1em;
+  width: 1.42857143em;
+  height: 1.42857143em;
 }
 [dir="rtl"] .treeview .expander > img {
   float: right;
-  margin-left: 5px;
+  margin-left: 1em;
 }
 .treeview label {
   font-weight: normal;

package/dist/tryton-sao.js

@@ -1,7 +1,7 @@
 /* This file is part of Tryton.  The COPYRIGHT file at the top level of
    this repository contains the full copyright notices and license terms. */
 var Sao = {
-    __version__: '6.8.3',
+    __version__: '6.8.5',
 };
 
 (function() {
@@ -778,11 +778,14 @@ var Sao = {
 
     Sao.main_menu_row_activate = function() {
         var screen = Sao.main_menu_screen;
-        // ids is not defined to prevent to add suffix
-        return Sao.Action.exec_keyword('tree_open', {
-            'model': screen.model_name,
-            'id': screen.get_id(),
-        }, null, false);
+        const id = screen.get_id();
+        if (id) {
+            // ids is not defined to prevent to add suffix
+            Sao.Action.exec_keyword('tree_open', {
+                'model': screen.model_name,
+                'id': screen.get_id(),
+            }, null, false);
+        }
     };
 
     Sao.menu = function(preferences) {
@@ -824,6 +827,7 @@ var Sao = {
             view.table.removeClass('table table-bordered');
             view.table.addClass('no-responsive');
             view.table.find('thead').hide();
+            view.table.find('colgroup > col.tree-menu').css('width', 0);
             var gs = new Sao.GlobalSearch();
             jQuery('#global-search').empty();
             jQuery('#global-search').append(gs.el);
@@ -2731,7 +2735,7 @@ var Sao = {
                     'sao_device_cookies', JSON.stringify(device_cookies));
             });
             renew_prm.fail(() => {
-                Sao.error("Cannot renew device cookie");
+                Sao.Logger.error("Cannot renew device cookie");
             });
         }
     });
@@ -6290,7 +6294,7 @@ var Sao = {
                             return img_url;
                         })
                         .fail(() => {
-                            Sao.error("Unknown icon %s", icon_name);
+                            Sao.Logger.error("Unknown icon %s", icon_name);
                         });
                 }
             });
@@ -7576,13 +7580,13 @@ var Sao = {
             if (this.indexOf(record) < 0) {
                 this.splice(position, 0, record);
             }
-            for (var record_rm in this.record_removed) {
+            for (var record_rm of this.record_removed) {
                 if (record_rm.id == record.id) {
                     this.record_removed.splice(
                             this.record_removed.indexOf(record_rm), 1);
                 }
             }
-            for (var record_del in this.record_deleted) {
+            for (var record_del of this.record_deleted) {
                 if (record_del.id == record.id) {
                     this.record_deleted.splice(
                             this.record_deleted.indexOf(record_del), 1);
@@ -8276,10 +8280,13 @@ var Sao = {
                         (field instanceof Sao.field.Reference)) {
                     related = name + '.';
                     this._values[related] = values[related] || {};
-                } else if ((field instanceof Sao.field.Selection) &&
-                        (name + ':string' in values)){
+                } else if (field instanceof Sao.field.Selection) {
                     related = name + ':string';
-                    this._values[related] = values[related];
+                    if (name + ':string' in values) {
+                        this._values[related] = values[related];
+                    } else {
+                        delete this._values[related];
+                    }
                 }
                 this.model.fields[name].set(this, value);
                 this._loaded[name] = true;
@@ -8445,7 +8452,7 @@ var Sao = {
         get_eval: function() {
             var value = {};
             for (var key in this.model.fields) {
-                if (!this.model.fields.hasOwnProperty(key) && this.id >= 0)
+                if (!this._loaded.hasOwnProperty(key) && this.id >= 0)
                     continue;
                 value[key] = this.model.fields[key].get_eval(this);
             }
@@ -9200,10 +9207,16 @@ var Sao = {
     });
 
     Sao.field.Selection = Sao.class_(Sao.field.Field, {
-        _default: null
+        _default: null,
+        set_client: function(record, value, force_change) {
+            // delete before trigger the display
+            delete record._values[this.name + ':string'];
+            Sao.field.Selection._super.set_client.call(
+                this, record, value, force_change);
+        }
     });
 
-    Sao.field.MultiSelection = Sao.class_(Sao.field.Field, {
+    Sao.field.MultiSelection = Sao.class_(Sao.field.Selection, {
         _default: null,
         get: function(record) {
             var value = Sao.field.MultiSelection._super.get.call(this, record);
@@ -13458,11 +13471,14 @@ var Sao = {
         default_row_activate: function() {
             if ((this.current_view.view_type == 'tree') &&
                     (this.current_view.attributes.keyword_open == 1)) {
-                Sao.Action.exec_keyword('tree_open', {
-                    'model': this.model_name,
-                    'id': this.get_id(),
-                    'ids': [this.get_id()]
-                }, this.local_context, false);
+                const id = this.get_id();
+                if (id) {
+                    Sao.Action.exec_keyword('tree_open', {
+                        'model': this.model_name,
+                        'id': this.get_id(),
+                        'ids': [this.get_id()]
+                    }, this.local_context, false);
+                }
             } else {
                 if (!this.modified()) {
                     this.switch_view('form');
@@ -15288,7 +15304,7 @@ function eval_pyson(value){
         hide: function() {
             Sao.View.Form.Page._super.hide.call(this);
             if (this.el.hasClass('active')) {
-                this.el.next(':visible').find('a').tab('show');
+                this.el.siblings(':visible').first().find('a').tab('show');
             }
         }
     });
@@ -21340,6 +21356,11 @@ function eval_pyson(value){
                 this.expander.children().html('&nbsp;');
                 this.expander.on('click keypress',
                         Sao.common.click_press(this.toggle_row.bind(this)));
+                this.expander.dblclick((evt) => {
+                    // prevent calling switch_row on td
+                    evt.preventDefault();
+                    evt.stopImmediatePropagation();
+                });
             }
 
             for (var i = 0; i < this.tree.columns.length; i++) {

package/dist/tryton-sao.min.css

@@ -9598,16 +9598,15 @@ img.icon {
 .treeview > table.tree col.optional {
   width: 30px;
 }
-.treeview > table.tree td.draggable-handle,
-.treeview > table.tree td.optional {
-  cursor: grab;
-}
 .treeview > table.tree td.draggable-handle > img,
 .treeview > table.tree td.optional > img {
   display: block;
   margin-right: auto;
   margin-left: auto;
 }
+.treeview > table.tree td.draggable-handle {
+  cursor: grab;
+}
 .treeview > table.tree th.selection-state > input,
 .treeview > table.tree td.selection-state > input {
   display: block;
@@ -9788,12 +9787,13 @@ img.icon {
 }
 .treeview .expander > img {
   float: left;
-  margin-right: 5px;
-  width: 1em;
+  margin-right: 1em;
+  width: 1.42857143em;
+  height: 1.42857143em;
 }
 [dir="rtl"] .treeview .expander > img {
   float: right;
-  margin-left: 5px;
+  margin-left: 1em;
 }
 .treeview label {
   font-weight: normal;