[MIG] product_variant_sale_price: Migration to 16.0

product_variant_sale_price/__manifest__.py

@@ -5,7 +5,7 @@
 {
     "name": "Product Variant Sale Price",
     "summary": "Allows to write fixed prices in product variants",
-    "version": "15.0.1.0.0",
+    "version": "16.0.1.0.0",
     "category": "Product Management",
     "website": "https://github.com/OCA/product-variant",
     "author": "Tecnativa, " "Odoo Community Association (OCA)",

product_variant_sale_price/models/product_product.py

@@ -11,10 +11,11 @@ def _update_fix_price(self, vals):
         if "list_price" in vals:
             self.mapped("product_variant_ids").write({"fix_price": vals["list_price"]})
 
-    @api.model
-    def create(self, vals):
-        product_tmpl = super().create(vals)
-        product_tmpl._update_fix_price(vals)
+    @api.model_create_multi
+    def create(self, vals_list):
+        product_tmpl = super().create(vals_list)
+        for prod, vals in zip(product_tmpl, vals_list):
+            prod._update_fix_price(vals)
         return product_tmpl
 
     def write(self, vals):
@@ -34,27 +35,25 @@ def _compute_lst_price(self):
         uom_model = self.env["uom.uom"]
         for product in self:
             price = product.fix_price or product.list_price
-            if "uom" in self.env.context:
-                price = product.uom_id._compute_price(
-                    price, uom_model.browse(self.env.context["uom"])
-                )
+            if self.env.context.get("uom"):
+                context_uom = uom_model.browse(self.env.context["uom"])
+                price = product.uom_id._compute_price(price, context_uom)
             product.lst_price = price
 
     def _compute_list_price(self):
         uom_model = self.env["uom.uom"]
         for product in self:
             price = product.fix_price or product.product_tmpl_id.list_price
-            if "uom" in self.env.context:
-                price = product.uom_id._compute_price(
-                    price, uom_model.browse(self.env.context["uom"])
-                )
+            if self.env.context.get("uom"):
+                context_uom = uom_model.browse(self.env.context["uom"])
+                price = product.uom_id._compute_price(price, context_uom)
             product.list_price = price
 
     def _inverse_product_lst_price(self):
         uom_model = self.env["uom.uom"]
         for product in self:
             vals = {}
-            if "uom" in self.env.context:
+            if self.env.context.get("uom"):
                 vals["fix_price"] = product.uom_id._compute_price(
                     product.lst_price, uom_model.browse(self.env.context["uom"])
                 )

product_variant_sale_price/tests/test_product_product.py

@@ -67,11 +67,11 @@ def test_post_init_hook(self):
         )
 
         # Flush the records to DB before direct SQL
-        self.product_blue.flush()
-        self.product_red.flush()
+        self.product_blue.env.flush_all()
+        self.product_red.env.flush_all()
 
         set_sale_price_on_variant(self.cr, None, self.product_template.id)
-        self.product_template.product_variant_ids.invalidate_cache()
+        self.product_template.product_variant_ids.invalidate_recordset()
         self.assertEqual(
             self.product_template.list_price + 100.00, self.product_blue.lst_price
         )

setup/product_variant_sale_price/odoo/addons/product_variant_sale_price

@@ -0,0 +1 @@
+../../../../product_variant_sale_price

setup/product_variant_sale_price/setup.py

@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)