[MIG] : Migration to 17.0 - (name_get)

Author: marcelsavegnago

l10n_br_fiscal/models/comment.py

@@ -86,13 +86,15 @@ def _name_search(
             name_get_uid=name_get_uid,
         )
 
-    def name_get(self):
+    @api.depends("comment")
+    def _compute_display_name(self):
         def truncate_name(name):
             if len(name) > 60:
                 name = f"{name[:60]}..."
             return name
 
-        return [(r.id, f"{truncate_name(r.name)}") for r in self]
+        for record in self:
+            record.display_name = truncate_name(record.comment)
 
     # format_amount function for fiscal observation
     # This way we can format numbers in currency template on fiscal observation

l10n_br_fiscal/models/data_abstract.py

@@ -91,13 +91,15 @@ def _name_search(
             name, args=args, operator=operator, limit=limit, name_get_uid=name_get_uid
         )
 
-    def name_get(self):
+    @api.depends("name", "code")
+    def _compute_display_name(self):
         def truncate_name(name):
             if len(name) > 60:
                 name = f"{name[:60]}..."
             return name
 
-        if self._context.get("show_code_only"):
-            return [(r.id, f"{r.code}") for r in self]
-
-        return [(r.id, f"{r.code} - {truncate_name(r.name)}") for r in self]
+        for record in self:
+            if self._context.get("show_code_only"):
+                record.display_name = record.code
+            else:
+                record.display_name = f"{record.code} - {truncate_name(record.name)}"

l10n_br_fiscal/models/document.py

@@ -534,11 +534,10 @@ def _compute_document_name(self):
             )
         return name
 
-    def name_get(self):
-        res = []
+    @api.depends("_compute_document_name")
+    def _compute_display_name(self):
         for record in self:
-            res.append((record.id, record._compute_document_name()))
-        return res
+            record.display_name = record._compute_document_name()
 
     @api.depends(
         "issuer",

l10n_br_fiscal/models/document_serie.py

@@ -78,8 +78,10 @@ def create(self, vals_list):
                 vals.update({"internal_sequence_id": self._create_sequence(vals)})
         return super().create(vals_list)
 
-    def name_get(self):
-        return [(r.id, f"{r.name}") for r in self]
+    @api.depends("name")
+    def _compute_display_name(self):
+        for record in self:
+            record.display_name = record.name
 
     def _is_invalid_number(self, document_number):
         self.ensure_one()

l10n_br_fiscal/models/tax_definition.py

@@ -28,16 +28,6 @@ def _compute_display_name(self):
         for record in self:
             record.display_name = record._get_complete_name()
 
-    @api.depends("tax_group_id", "tax_id", "cst_code")
-    def name_get(self):
-        result = []
-        for record in self:
-            name = record._get_complete_name()
-            result.append((record.id, name))
-        return result
-
-    display_name = fields.Char(compute="_compute_display_name", store=True)
-
     code = fields.Char(
         size=8,
     )