From 0fb8c1b12b049af4c27926743461883c36027f07 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20D=2E=20D=C3=ADaz?=
 <alexandre.diaz@tecnativa.com>
Date: Wed, 27 Jan 2021 14:14:39 +0100
Subject: [PATCH] [FIX] web_widget_one2many_product_picker: Update widget state

---
 .../views/One2ManyProductPicker/renderer.js   | 20 +++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/web_widget_one2many_product_picker/static/src/js/views/One2ManyProductPicker/renderer.js b/web_widget_one2many_product_picker/static/src/js/views/One2ManyProductPicker/renderer.js
index a78808fe6..92376ca2f 100644
--- a/web_widget_one2many_product_picker/static/src/js/views/One2ManyProductPicker/renderer.js
+++ b/web_widget_one2many_product_picker/static/src/js/views/One2ManyProductPicker/renderer.js
@@ -119,7 +119,7 @@ odoo.define("web_widget_one2many_product_picker.One2ManyProductPickerRenderer",
          * @param {Array[Object]} states
          * @returns {Deferred}
          */
-        _removeRecords: function (states) {
+        _removeRecords: function (states, new_states) {
             var defs = [];
             var to_destroy = [];
             for (var index_state in states) {
@@ -145,6 +145,7 @@ odoo.define("web_widget_one2many_product_picker.One2ManyProductPickerRenderer",
                 var widget_product_id = widget_destroyed.state
                     .data[this.options.field_map.product].data.id;
                 var found = false;
+                // If already exists a widget for the product don't try create a new one
                 for (var eb = this.widgets.length-1; eb>=0; --eb) {
                     var widget = this.widgets[eb];
                     if (
@@ -156,9 +157,21 @@ odoo.define("web_widget_one2many_product_picker.One2ManyProductPickerRenderer",
                         break;
                     }
                 }
+
                 if (!found) {
+                    // Get the new state ID if exists to link it with the new record
+                    var new_state_id = undefined;
+                    for (var eb = new_states.length-1; eb>=0; --eb) {
+                        var state = new_states[eb];
+                        if (
+                            state.data[this.options.field_map.product].data.id === widget_product_id
+                        ) {
+                            new_state_id = state.id;
+                            break;
+                        }
+                    }
                     var search_record = _.find(this.search_data, {id: widget_product_id});
-                    var new_search_record = _.extend({}, search_record, {__id: state.id});
+                    var new_search_record = _.extend({}, search_record, {__id: new_state_id});
                     var search_record_index = widget_destroyed.$el.index();
                     defs.push(
                         this.appendSearchRecords(
@@ -202,8 +215,7 @@ odoo.define("web_widget_one2many_product_picker.One2ManyProductPickerRenderer",
                     states_to_destroy.push(old_state);
                 }
             }
-
-            this._removeRecords(states_to_destroy);
+            this._removeRecords(states_to_destroy, this.state.data);
 
             // Records to Update or Create
             var defs = [];