[MIG] product_set: Migration to 17.0

Author: NICO-SOLUTIONS

product_set/README.rst

@@ -83,6 +83,8 @@ Contributors
 
    -  Pilar Vargas
 
+-  Nils Coenen <nils.coenen@nico-solutions.de>
+
 Other credits
 -------------
 

product_set/__manifest__.py

@@ -6,7 +6,7 @@
     "category": "Sale",
     "license": "AGPL-3",
     "author": "Anybox, Odoo Community Association (OCA)",
-    "version": "16.0.3.0.0",
+    "version": "17.0.1.0.0",
     "website": "https://github.com/OCA/product-attribute",
     "depends": ["product"],
     "data": [

product_set/models/product_set.py

@@ -1,7 +1,7 @@
 # Copyright 2015 Anybox S.A.S
 # Copyright 2016-2018 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class ProductSet(models.Model):
@@ -32,14 +32,17 @@ class ProductSet(models.Model):
         "it's going to be available for all of them.",
     )
 
-    def name_get(self):
-        return [(rec.id, rec._name_get()) for rec in self]
+    display_name = fields.Char(
+        compute="_compute_display_name", string="Display Name", store=True
+    )
 
-    def _name_get(self):
-        parts = []
-        if self.ref:
-            parts.append("[%s]" % self.ref)
-        parts.append(self.name)
-        if self.partner_id:
-            parts.append("@ %s" % self.partner_id.name)
-        return " ".join(parts)
+    @api.depends("name", "ref", "partner_id.name")
+    def _compute_display_name(self):
+        for rec in self:
+            parts = []
+            if rec.ref:
+                parts.append("[%s]" % rec.ref)
+            parts.append(rec.name or "")
+            if rec.partner_id and rec.partner_id.name:
+                parts.append("@ %s" % rec.partner_id.name)
+            rec.display_name = " ".join(map(str, parts))

product_set/readme/CONTRIBUTORS.md

@@ -8,3 +8,4 @@
 - Manuel Regidor \<<manuel.regidor@sygel.es>\>
 - [Tecnativa](https://www.tecnativa.com):
   - Pilar Vargas
+- Nils Coenen \<<nils.coenen@nico-solutions.de>\>

product_set/readme/CONTRIBUTORS.rst

@@ -427,6 +427,7 @@ <h2><a class="toc-backref" href="#toc-entry-5">Contributors</a></h2>
 <li>Pilar Vargas</li>
 </ul>
 </li>
+<li>Nils Coenen &lt;<a class="reference external" href="mailto:nils.coenen&#64;nico-solutions.de">nils.coenen&#64;nico-solutions.de</a>&gt;</li>
 </ul>
 </div>
 <div class="section" id="other-credits">

product_set/readme/CREDITS.rst

@@ -18,13 +18,20 @@ def test_name(self):
         # no ref
         product_set.name = "Foo"
         product_set.ref = ""
-        self.assertEqual(product_set.name_get(), [(product_set.id, "Foo")])
+        self.assertEqual(
+            product_set.read(["display_name"]),
+            [{"id": product_set.id, "display_name": "Foo"}],
+        )
         # with ref
         product_set.ref = "123"
-        self.assertEqual(product_set.name_get(), [(product_set.id, "[123] Foo")])
+        self.assertEqual(
+            product_set.read(["display_name"]),
+            [{"id": product_set.id, "display_name": "[123] Foo"}],
+        )
         # with partner
         partner = self.env.ref("base.res_partner_1")
         product_set.partner_id = partner
         self.assertEqual(
-            product_set.name_get(), [(product_set.id, "[123] Foo @ %s" % partner.name)]
+            product_set.read(["display_name"]),
+            [{"id": product_set.id, "display_name": "[123] Foo @ %s" % partner.name}],
         )