[FIX] product: update variants when modifying exclusions

Before this commit, when importing product template attribute exclusions,
variants were not updated accordingly.

Now, variants will be created, archived, or deleted when creating,
modifying, or removing exclusions.

opw-3693065

closes #155103

Signed-off-by: Victor Feyens (vfe) <vfe@odoo.com>

addons/product/models/product_attribute.py

@@ -604,6 +604,27 @@ class ProductTemplateAttributeExclusion(models.Model):
         'product.template.attribute.value', relation="product_attr_exclusion_value_ids_rel",
         string='Attribute Values', domain="[('product_tmpl_id', '=', product_tmpl_id), ('ptav_active', '=', True)]")
 
+    @api.model_create_multi
+    def create(self, vals_list):
+        exclusions = super().create(vals_list)
+        exclusions.product_tmpl_id._create_variant_ids()
+        return exclusions
+
+    def unlink(self):
+        # Keep a reference to the related templates before the deletion.
+        templates = self.product_tmpl_id
+        res = super().unlink()
+        templates._create_variant_ids()
+        return res
+
+    def write(self, values):
+        templates = self.env['product.template']
+        if 'product_tmpl_id' in values:
+            templates = self.product_tmpl_id
+        res = super().write(values)
+        (templates | self.product_tmpl_id)._create_variant_ids()
+        return res
+
 
 class ProductAttributeCustomValue(models.Model):
     _name = "product.attribute.custom.value"

addons/product/tests/test_variants.py

@@ -1401,6 +1401,7 @@ def get_ptav(template, ptav):
             )
 
         cls.smartphone_s = get_ptav(cls.smartphone, cls.size_attribute_s)
+        cls.smartphone_l = get_ptav(cls.smartphone, cls.size_attribute_l)
         cls.smartphone_256 = get_ptav(cls.smartphone, cls.storage_attr_value_256)
         cls.smartphone_128 = get_ptav(cls.smartphone, cls.storage_attr_value_128)
 
@@ -1471,3 +1472,32 @@ def test_variants_2_exclusions_different_lines(self):
             'exclude_for': [(2, self.smartphone_s.exclude_for.ids[0], 0)]
         })
         self.assertEqual(len(self.smartphone.product_variant_ids), 3, 'With one exclusion, the smartphone should have 3 active different variants')
+
+    @mute_logger('odoo.models.unlink')
+    def test_exclusions_crud(self):
+        """ Make sure that exclusions creation, update & delete are correctly handled.
+
+        Exclusions updates are not necessarily done from a specific template.
+        """
+        PTAE = self.env['product.template.attribute.exclusion']
+
+        exclude = PTAE.create({
+            'product_tmpl_id': self.smartphone.id,
+            'product_template_attribute_value_id': self.smartphone_s.id,
+            'value_ids': [Command.set(self.smartphone_256.ids)]
+        })
+        self.assertEqual(len(self.smartphone.product_variant_ids), 3)
+        self.assertNotIn(
+            self.smartphone_s + self.smartphone_256,
+            [product.product_template_attribute_value_ids for product in self.smartphone.product_variant_ids],
+        )
+
+        exclude.value_ids = [Command.set(self.smartphone_128.ids)]
+        self.assertEqual(len(self.smartphone.product_variant_ids), 3)
+        self.assertNotIn(
+            self.smartphone_s + self.smartphone_128,
+            [product.product_template_attribute_value_ids for product in self.smartphone.product_variant_ids],
+        )
+
+        exclude.unlink()
+        self.assertEqual(len(self.smartphone.product_variant_ids), 4)