[MIG] product_alias: Migration to 18.0

sebalix · sebalix · commit ecc80e546a36 · 2026-05-11T15:23:12.000+02:00
diff --git a/product_alias/README.rst b/product_alias/README.rst
@@ -1,6 +1,6 @@
-=========================
-Shopinvader Product Alias
-=========================
+=============
+Product Alias
+=============
 
 .. 
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
@@ -29,7 +29,7 @@ Shopinvader Product Alias
 |badge1| |badge2| |badge3| |badge4| |badge5|
 
 This module allows to add url aliases from the product template for each
-selectected variants (or attribute values's combinaison)
+selectected variants (or attribute values's combinaison).
 
 **Table of contents**
 
diff --git a/product_alias/__manifest__.py b/product_alias/__manifest__.py
@@ -3,9 +3,9 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
 {
-    "name": "Shopinvader Product Alias",
+    "name": "Product Alias",
     "summary": "Alias for each product attribute values's configuration",
-    "version": "14.0.1.0.0",
+    "version": "18.0.1.0.0",
     "category": "Shopinvader",
     "website": "https://github.com/OCA/e-commerce",
     "author": "Akretion, Odoo Community Association (OCA)",
@@ -14,7 +14,7 @@
     "application": False,
     "installable": True,
     "depends": [
-        "shopinvader",
+        "product",
     ],
     "data": [
         "views/product_template.xml",
diff --git a/product_alias/models/product_alias.py b/product_alias/models/product_alias.py
@@ -7,14 +7,14 @@
 
 class ProductAlias(models.Model):
     _name = "product.alias"
+    _description = "Product Alias"
 
     product_tmpl_id = fields.Many2one(
         "product.template",
         required=True,
     )
 
     name = fields.Char(
-        string="Name",
         required=True,
     )
 
diff --git a/product_alias/models/product_product.py b/product_alias/models/product_product.py
@@ -18,7 +18,9 @@ def _get_matching_alias(self):
         values = self.product_template_attribute_value_ids.product_attribute_value_id
         for alias in self.product_alias_ids:
             attrs = alias.attribute_value_ids.attribute_id
-            variant_values = values.filtered(lambda s: s.attribute_id in attrs)
+            variant_values = values.filtered(
+                lambda s, attrs=attrs: s.attribute_id in attrs
+            )
             if variant_values and not (variant_values - alias.attribute_value_ids):
                 match |= alias
         if len(match) > 1:
diff --git a/product_alias/readme/DESCRIPTION.md b/product_alias/readme/DESCRIPTION.md
@@ -1,2 +1,2 @@
 This module allows to add url aliases from the product template for each
-selectected variants (or attribute values's combinaison)
+selectected variants (or attribute values's combinaison).
diff --git a/product_alias/static/description/index.html b/product_alias/static/description/index.html
@@ -3,7 +3,7 @@
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
 <meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
-<title>Shopinvader Product Alias</title>
+<title>Product Alias</title>
 <style type="text/css">
 
 /*
@@ -360,8 +360,8 @@
 </style>
 </head>
 <body>
-<div class="document" id="shopinvader-product-alias">
-<h1 class="title">Shopinvader Product Alias</h1>
+<div class="document" id="product-alias">
+<h1 class="title">Product Alias</h1>
 
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
@@ -371,7 +371,7 @@ <h1 class="title">Shopinvader Product Alias</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/e-commerce/tree/18.0/product_alias"><img alt="OCA/e-commerce" src="https://img.shields.io/badge/github-OCA%2Fe--commerce-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/e-commerce-18-0/e-commerce-18-0-product_alias"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/e-commerce&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module allows to add url aliases from the product template for each
-selectected variants (or attribute values’s combinaison)</p>
+selectected variants (or attribute values’s combinaison).</p>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
diff --git a/product_alias/tests/test_product.py b/product_alias/tests/test_product.py
@@ -2,15 +2,15 @@
 # @author Sébastien BEAU <sebastien.beau@akretion.com>
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-
 from odoo.exceptions import UserError
-from odoo.tests import SavepointCase
+from odoo.tests import TransactionCase
 
 
-class TestProduct(SavepointCase):
+class TestProduct(TransactionCase):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
         cls.tmpl = cls.env.ref("product.product_product_4_product_template")
         cls.product_sw = cls.env.ref("product.product_product_4")  # steel, white
         cls.product_sb = cls.env.ref("product.product_product_4b")  # steel, black
@@ -70,4 +70,4 @@ def test_product_alias_raise(self):
                     "attribute_value_ids": [(6, 0, [self.attr_a.id])],
                 }
             )
-            self.env["product.alias"].flush()
+            self.env["product.alias"].flush_model()
diff --git a/product_alias/views/product_template.xml b/product_alias/views/product_template.xml
@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="UTF-8" ?>
+<?xml version="1.0" encoding="utf-8" ?>
 <!-- Copyright (C) 2023 Akretion (<http://www.akretion.com>).
      @author Kévin Roche <kevin.roche@akretion.com>
      License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). -->
@@ -11,17 +11,16 @@
                 <page
                     name="Alias"
                     string="Alias"
-                    attrs="{'invisible': [('product_variant_count', '&lt;=', 1)]}"
+                    invisible="product_variant_count &lt;= 1"
                 >
                     <field
                         name="product_alias_ids"
                         context="{'default_product_tmpl_id': id}"
                     >
-                        <tree editable="bottom">
+                        <list editable="bottom">
                             <field name="name" />
                             <field name="attribute_value_ids" widget="many2many_tags" />
-                            <field name="available_attribute_value_ids" invisible="1" />
-                        </tree>
+                        </list>
                     </field>
                 </page>
             </xpath>

Original file line number	Diff line number	Diff line change
`@@ -7,14 +7,14 @@`
`7`	`7`
`8`	`8`	`class ProductAlias(models.Model):`
`9`	`9`	`_name = "product.alias"`
	`10`	`+ _description = "Product Alias"`
`10`	`11`
`11`	`12`	`product_tmpl_id = fields.Many2one(`
`12`	`13`	`"product.template",`
`13`	`14`	`required=True,`
`14`	`15`	`)`
`15`	`16`
`16`	`17`	`name = fields.Char(`
`17`		`- string="Name",`
`18`	`18`	`required=True,`
`19`	`19`	`)`
`20`	`20`
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,2 @@`
`1`	`1`	`This module allows to add url aliases from the product template for each`
`2`		`-selectected variants (or attribute values's combinaison)`
	`2`	`+selectected variants (or attribute values's combinaison).`