[IMP]product_packaging_level: recover name_get policy flow

product_packaging_level/models/product_packaging.py

@@ -10,6 +10,7 @@ class ProductPackaging(models.Model):
     _inherit = "product.packaging"
     _order = "product_id, level_sequence"
 
+    name = fields.Char(compute="_compute_name", store=True, readonly=False)
     packaging_level_id = fields.Many2one(
         "product.packaging.level",
         required=True,
@@ -126,11 +127,27 @@ def _onchange_package_type(self):
         if package_type_level_id:
             self.packaging_level_id = package_type_level_id
 
-    @api.onchange("packaging_level_id", "package_type_id", "name", "name_policy")
-    def _onchange_name(self):
-        new_name = self.name
-        if self.name_policy == "by_package_level":
-            new_name = self.packaging_level_id.display_name
-        elif self.name_policy == "by_package_type":
-            new_name = self.package_type_id.name
-        self.name = new_name
+    @api.depends(
+        "packaging_level_id",
+        "package_type_id",
+        "name",
+        "name_policy",
+    )
+    def _compute_name(self):
+        for rec in self:
+            new_name = rec.name
+            if rec.name_policy == "by_package_level":
+                new_name = rec.packaging_level_id.display_name
+            elif rec.name_policy == "by_package_type":
+                new_name = rec.package_type_id.name
+            rec.name = new_name
+
+    # Keep this method to respect translations on level name
+    def name_get(self):
+        result = []
+        for record in self:
+            if record.product_id and record.packaging_level_id:
+                result.append((record.id, record.packaging_level_id.display_name))
+            else:
+                result.append((record.id, record.name))
+        return result

product_packaging_level/tests/test_product_packaging_level.py

@@ -39,7 +39,7 @@ def test_display_name(self):
 
     def test_name_by_level(self):
         self.packaging.name_policy = "by_package_level"
-        self.assertEqual(self.packaging.name, "Packaging Test")
+        self.assertEqual(self.packaging.name, "Packaging Level Test (TEST2)")
 
     def test_name_by_package_type(self):
         # Required for `package_type_id` to be visible in the view
@@ -48,7 +48,6 @@ def test_name_by_package_type(self):
         )
         self.packaging.name_policy = "by_package_type"
         self.packaging.package_type_id = self.package_type_box
-        self.packaging._onchange_name()
         self.assertEqual(self.packaging.name, "Box")
 
     def test_name_by_user_defined(self):
@@ -61,12 +60,10 @@ def test_name_by_user_defined(self):
         self.packaging.name = packaging_name
         # try to change package_type_id
         self.packaging.package_type_id = self.package_type_box
-        self.packaging._onchange_name()
         self.assertEqual(self.packaging.name, packaging_name)
 
         # try to change packaging level
         self.packaging.packaging_level_id = self.level
-        self.packaging._onchange_name()
         self.assertEqual(self.packaging.name, packaging_name)
 
     def test_name_by_package_type_without_group_tracking_lot(self):