From 86ff396e8e4fd48f2628bb2dfdf79e01d707c216 Mon Sep 17 00:00:00 2001
From: Carlos Roca <carlos.roca@tecnativa.com>
Date: Tue, 4 Jul 2023 10:41:09 +0200
Subject: [PATCH] [IMP] web_widget_one2many_product_picker: Allow set first
 active group instead of set All by default

---
 .../js/widgets/field_one2many_product_picker.js    | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/web_widget_one2many_product_picker/static/src/js/widgets/field_one2many_product_picker.js b/web_widget_one2many_product_picker/static/src/js/widgets/field_one2many_product_picker.js
index 87ea2dfbe..612c0aafd 100644
--- a/web_widget_one2many_product_picker/static/src/js/widgets/field_one2many_product_picker.js
+++ b/web_widget_one2many_product_picker/static/src/js/widgets/field_one2many_product_picker.js
@@ -81,8 +81,9 @@ odoo.define("web_widget_one2many_product_picker.FieldOne2ManyProductPicker", fun
 
             // FIXME: Choose a better way to get the active controller or model objects
             this.parent_controller = parent.getParent();
+            this.indexGroup = 0;
             if (this.view) {
-                this._processGroups();
+                this.indexGroup = this._processGroups();
             }
 
             this._currentSearchBatchID = 0;
@@ -127,7 +128,7 @@ odoo.define("web_widget_one2many_product_picker.FieldOne2ManyProductPicker", fun
                         // Show Lines
                         self._updateSearchContext(-1);
                     } else {
-                        self._updateSearchContext(0);
+                        self._updateSearchContext(self.indexGroup);
                     }
                     self._searchContext.text = "";
                 });
@@ -268,13 +269,15 @@ odoo.define("web_widget_one2many_product_picker.FieldOne2ManyProductPicker", fun
          */
         _processGroups: function() {
             this.searchGroups = [];
+            var activeIndex = 0;
             var hasUserActive = false;
             var groups = this.options.groups;
             for (var groupIndex in groups) {
                 var group_def = groups[groupIndex];
-                if (group_def.active) {
-                    group_def.active = !hasUserActive;
+                if (group_def.active && !hasUserActive) {
                     hasUserActive = true;
+                    // Set index + 1 because the first position is for All option
+                    activeIndex = parseInt(groupIndex) + 1;
                 }
                 if (!group_def.records_per_page) {
                     group_def.records_per_page = 16;
@@ -290,7 +293,8 @@ odoo.define("web_widget_one2many_product_picker.FieldOne2ManyProductPicker", fun
                 active: !hasUserActive,
                 records_per_page: 16,
             });
-            this._activeSearchGroup = this.searchGroups[0];
+            this._activeSearchGroup = this.searchGroups[activeIndex];
+            return activeIndex;
         },
 
         /**