Core: Remove name_get and use display_name instead

addons/account/models/account_account.py

@@ -597,12 +597,10 @@ def _onchange_name(self):
             self.name = name
             self.code = code
 
-    def name_get(self):
-        result = []
+    @api.depends('code')
+    def _compute_display_name(self):
         for account in self:
-            name = account.code + ' ' + account.name
-            result.append((account.id, name))
-        return result
+            account.display_name = f"{account.code} {account.name}"
 
     @api.returns('self', lambda value: value.id)
     def copy(self, default=None):
@@ -701,7 +699,8 @@ def name_create(self, name):
         """
         if 'import_file' in self.env.context:
             code, name = self._split_code_name(name)
-            return self.create({'code': code, 'name': name}).name_get()[0]
+            record = self.create({'code': code, 'name': name})
+            return record.id, record.display_name
         raise ValidationError(_("Please create new accounts from the Chart of Accounts menu."))
 
     @api.model_create_multi
@@ -828,15 +827,15 @@ def _compute_code_prefix_start(self):
             if not group.code_prefix_start or group.code_prefix_start > group.code_prefix_end:
                 group.code_prefix_start = group.code_prefix_end
 
-    def name_get(self):
-        result = []
+    @api.depends('code_prefix_start', 'code_prefix_end')
+    def _compute_display_name(self):
         for group in self:
             prefix = group.code_prefix_start and str(group.code_prefix_start)
             if prefix and group.code_prefix_end != group.code_prefix_start:
                 prefix += '-' + str(group.code_prefix_end)
             name = (prefix and (prefix + ' ') or '') + group.name
-            result.append((group.id, name))
-        return result
+            group.display_name = name
+
 
     @api.model
     def _name_search(self, name, domain=None, operator='ilike', limit=None, order=None):

addons/account/models/account_account_tag.py

@@ -14,18 +14,17 @@ class AccountAccountTag(models.Model):
     tax_negate = fields.Boolean(string="Negate Tax Balance", help="Check this box to negate the absolute value of the balance of the lines associated with this tag in tax report computation.")
     country_id = fields.Many2one(string="Country", comodel_name='res.country', help="Country for which this tag is available, when applied on taxes.")
 
-    def name_get(self):
+    @api.depends('applicability', 'country_id')
+    @api.depends_context('company')
+    def _compute_display_name(self):
         if not self.env.company.multi_vat_foreign_country_ids:
-            return super().name_get()
+            return super()._compute_display_name()
 
-        res = []
         for tag in self:
             name = tag.name
             if tag.applicability == "taxes" and tag.country_id and tag.country_id != self.env.company.account_fiscal_country_id:
                 name = _("%s (%s)", tag.name, tag.country_id.code)
-            res.append((tag.id, name,))
-
-        return res
+            tag.display_name = name
 
     @api.model
     def _get_tax_tags(self, tag_name, country_id):

addons/account/models/account_incoterms.py

@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
-from odoo import fields, models
+from odoo import api, fields, models
 
 
 class AccountIncoterms(models.Model):
@@ -18,6 +18,7 @@ class AccountIncoterms(models.Model):
         'Active', default=True,
         help="By unchecking the active field, you may hide an INCOTERM you will not use.")
 
-    def name_get(self):
-        return [(incoterm.id, '%s%s' % (incoterm.code and '[%s] ' % incoterm.code or '', incoterm.name))
-                for incoterm in self]
+    @api.depends('code')
+    def _compute_display_name(self):
+        for incoterm in self:
+            incoterm.display_name = '%s%s' % (incoterm.code and '[%s] ' % incoterm.code or '', incoterm.name)

addons/account/models/account_journal.py

@@ -696,14 +696,13 @@ def set_bank_account(self, acc_number, bank_id=None):
                 'journal_id': self,
             }).id
 
-    def name_get(self):
-        res = []
+    @api.depends('currency_id')
+    def _compute_display_name(self):
         for journal in self:
             name = journal.name
             if journal.currency_id and journal.currency_id != journal.company_id.currency_id:
-                name = "%s (%s)" % (name, journal.currency_id.name)
-            res += [(journal.id, name)]
-        return res
+                name = f"{name} ({journal.currency_id.name})"
+            journal.display_name = name
 
     def action_configure_bank_journal(self):
         """ This function is called by the "configure" button of bank journals,

addons/account/models/account_move.py

@@ -2387,11 +2387,11 @@ def unlink(self):
         self.line_ids.unlink()
         return super().unlink()
 
-    def name_get(self):
-        result = []
+    @api.depends('partner_id', 'date', 'state', 'move_type')
+    @api.depends_context('input_full_display_name')
+    def _compute_display_name(self):
         for move in self:
-            result.append((move.id, move._get_move_display_name(show_ref=True)))
-        return result
+            move.display_name = move._get_move_display_name(show_ref=True)
 
     def onchange(self, values, field_name, field_onchange):
         if field_name in ('line_ids', 'invoice_line_ids'):

addons/account/models/account_move_line.py

@@ -1525,14 +1525,16 @@ def unlink(self):
 
         return res
 
-    def name_get(self):
-        return [(line.id, " ".join(
-            element for element in (
-                line.move_id.name,
-                line.ref and f"({line.ref})",
-                line.name or line.product_id.display_name,
-            ) if element
-        )) for line in self]
+    @api.depends('move_id', 'ref', 'product_id')
+    def _compute_display_name(self):
+        for line in self:
+            line.display_name = " ".join(
+                element for element in (
+                    line.move_id.name,
+                    line.ref and f"({line.ref})",
+                    line.name or line.product_id.display_name,
+                ) if element
+            )
 
     def copy_data(self, default=None):
         data_list = super().copy_data(default=default)

addons/account/models/account_payment.py

@@ -740,8 +740,9 @@ def unlink(self):
         return res
 
     @api.depends('move_id.name')
-    def name_get(self):
-        return [(payment.id, payment.move_id.name != '/' and payment.move_id.name or _('Draft Payment')) for payment in self]
+    def _compute_display_name(self):
+        for payment in self:
+            payment.display_name = payment.move_id.name if payment.move_id.name != '/' else _('Draft Payment')
 
     # -------------------------------------------------------------------------
     # SYNCHRONIZATION account.payment <-> account.move

addons/account/models/account_payment_method.py

@@ -119,10 +119,13 @@ class AccountPaymentMethodLine(models.Model):
     company_id = fields.Many2one(related='journal_id.company_id')
     available_payment_method_ids = fields.Many2many(related='journal_id.available_payment_method_ids')
 
-    def name_get(self):
-        if self.env.context.get('show_payment_journal_id'):
-            return [(method.id, "%s (%s)" % (method.name, method.journal_id.name)) for method in self]
-        return super().name_get()
+    @api.depends('journal_id')
+    @api.depends_context('show_payment_journal_id')
+    def _compute_display_name(self):
+        if not self.env.context.get('show_payment_journal_id'):
+            return super()._compute_display_name()
+        for method in self:
+            method.display_name = f"{method.name} ({method.journal_id.name})"
 
     @api.depends('payment_method_id.name')
     def _compute_name(self):

addons/account/models/account_report.py

@@ -203,11 +203,9 @@ def _get_copied_name(self):
         return name
 
     @api.depends('name', 'country_id')
-    def name_get(self):
-        result = []
+    def _compute_display_name(self):
         for report in self:
-            result.append((report.id, report.name + (f' ({report.country_id.code})' if report.country_id else '')))
-        return result
+            report.display_name = report.name + (f' ({report.country_id.code})' if report.country_id else '')
 
 
 class AccountReportLine(models.Model):
@@ -561,8 +559,11 @@ def _unlink_archive_used_tags(self):
             tags_to_archive.active = False
             tags_to_unlink.unlink()
 
-    def name_get(self):
-        return [(expr.id, f'{expr.report_line_name} [{expr.label}]') for expr in self]
+    @api.depends('report_line_name', 'label')
+    def _compute_display_name(self):
+        for expr in self:
+            expr.display_name = f'{expr.report_line_name} [{expr.label}]'
+
 
     def _expand_aggregations(self):
         """Return self and its full aggregation expression dependency"""

addons/account/models/account_tax.py

@@ -362,8 +362,9 @@ def copy(self, default=None):
             default['name'] = _("%s (Copy)") % self.name
         return super(AccountTax, self).copy(default=default)
 
-    def name_get(self):
-        name_list = []
+    @api.depends('type_tax_use', 'tax_scope')
+    @api.depends_context('append_type_to_tax_name')
+    def _compute_display_name(self):
         type_tax_use = dict(self._fields['type_tax_use']._description_selection(self.env))
         tax_scope = dict(self._fields['tax_scope']._description_selection(self.env))
         for record in self:
@@ -372,8 +373,7 @@ def name_get(self):
                 name += ' (%s)' % type_tax_use.get(record.type_tax_use)
             if record.tax_scope:
                 name += ' (%s)' % tax_scope.get(record.tax_scope)
-            name_list += [(record.id, name)]
-        return name_list
+            record.display_name = name
 
     @api.onchange('amount')
     def onchange_amount(self):

addons/account/models/res_partner_bank.py

@@ -307,15 +307,15 @@ def unlink(self):
             account.partner_id._message_log(body=msg)
         return super().unlink()
 
-    def name_get(self):
-        res = super().name_get()
+    @api.depends('allow_out_payment', 'acc_number', 'bank_id')
+    @api.depends_context('display_account_trust')
+    def _compute_display_name(self):
+        super()._compute_display_name()
         if self.env.context.get('display_account_trust'):
-            res = []
             for acc in self:
                 trusted_label = _('trusted') if acc.allow_out_payment else _('untrusted')
                 if acc.bank_id:
-                    name = '{} - {} ({})'.format(acc.acc_number, acc.bank_id.name, trusted_label)
+                    name = f'{acc.acc_number} - {acc.bank_id.name} ({trusted_label})'
                 else:
-                    name = '{} ({})'.format(acc.acc_number, trusted_label)
-                res.append((acc.id, name))
-        return res
+                    name = f'{acc.acc_number} ({trusted_label})'
+                acc.display_name = name

addons/analytic/models/analytic_account.py

@@ -115,16 +115,15 @@ def _check_company_consistency(self):
         if self._cr.fetchone():
             raise UserError(_("You can't set a different company on your analytic account since there are some analytic items linked to it."))
 
-    def name_get(self):
-        res = []
+    @api.depends('code', 'partner_id')
+    def _compute_display_name(self):
         for analytic in self:
             name = analytic.name
             if analytic.code:
                 name = f'[{analytic.code}] {name}'
             if analytic.partner_id.commercial_partner_id.name:
                 name = f'{name} - {analytic.partner_id.commercial_partner_id.name}'
-            res.append((analytic.id, name))
-        return res
+            analytic.display_name = name
 
     def copy_data(self, default=None):
         default = dict(default or {})

addons/base_address_extended/models/res_city.py

@@ -15,9 +15,8 @@ class City(models.Model):
     country_id = fields.Many2one(comodel_name='res.country', string='Country', required=True)
     state_id = fields.Many2one(comodel_name='res.country.state', string='State', domain="[('country_id', '=', country_id)]")
 
-    def name_get(self):
-        res = []
+    @api.depends('zipcode')
+    def _compute_display_name(self):
         for city in self:
-            name = city.name if not city.zipcode else '%s (%s)' % (city.name, city.zipcode)
-            res.append((city.id, name))
-        return res
+            name = city.name if not city.zipcode else f'{city.name} ({city.zipcode})'
+            city.display_name = name

addons/base_sparse_field/models/fields.py

@@ -59,7 +59,7 @@ def _compute_sparse(self, records):
 def _inverse_sparse(self, records):
     for record in records:
         values = record[self.sparse]
-        value = self.convert_to_read(record[self.name], record, use_name_get=False)
+        value = self.convert_to_read(record[self.name], record, use_display_name=False)
         if value:
             if values.get(self.name) != value:
                 values[self.name] = value

addons/calendar/models/calendar_event.py

@@ -645,7 +645,8 @@ def write(self, values):
 
         return True
 
-    def name_get(self):
+    @api.depends('privacy', 'user_id')
+    def _compute_display_name(self):
         """ Hide private events' name for events which don't belong to the current user
         """
         hidden = self.filtered(
@@ -654,11 +655,8 @@ def name_get(self):
                 evt.user_id.id != self.env.uid and
                 self.env.user.partner_id not in evt.partner_ids
         )
-
-        shown = self - hidden
-        shown_names = super(Meeting, shown).name_get()
-        obfuscated_names = [(eid, _('Busy')) for eid in hidden.ids]
-        return shown_names + obfuscated_names
+        hidden.display_name = _('Busy')
+        super(Meeting, self - hidden)._compute_display_name()
 
     @api.model
     def read_group(self, domain, fields, groupby, offset=0, limit=None, orderby=False, lazy=True):

addons/crm_iap_mine/models/crm_iap_lead_role.py

@@ -17,9 +17,9 @@ class PeopleRole(models.Model):
         ('name_uniq', 'unique (name)', 'Role name already exists!'),
     ]
 
-    @api.depends('name')
-    def name_get(self):
-        return [(role.id, role.name.replace('_', ' ').title()) for role in self]
+    def _compute_display_name(self):
+        for role in self:
+            role.display_name = role.name.replace('_', ' ').title()
 
 
 class PeopleSeniority(models.Model):
@@ -34,6 +34,6 @@ class PeopleSeniority(models.Model):
         ('name_uniq', 'unique (name)', 'Name already exists!'),
     ]
 
-    @api.depends('name')
-    def name_get(self):
-        return [(seniority.id, seniority.name.replace('_', ' ').title()) for seniority in self]
+    def _compute_display_name(self):
+        for seniority in self:
+            seniority.display_name = seniority.name.replace('_', ' ').title()

addons/crm_iap_mine/models/crm_iap_lead_seniority.py

@@ -17,5 +17,6 @@ class PeopleSeniority(models.Model):
     ]
 
     @api.depends('name')
-    def name_get(self):
-        return [(seniority.id, seniority.name.replace('_', ' ').title()) for seniority in self]
+    def _compute_display_name(self):
+        for seniority in self:
+            seniority.display_name = seniority.name.replace('_', ' ').title()

addons/event/models/event_event.py

@@ -589,11 +589,12 @@ def write(self, vals):
             self.message_subscribe([vals['organizer_id']])
         return res
 
-    def name_get(self):
+    @api.depends('event_registrations_sold_out', 'seats_limited', 'seats_max', 'seats_available')
+    @api.depends_context('name_with_seats_availability')
+    def _compute_display_name(self):
         """Adds ticket seats availability if requested by context."""
         if not self.env.context.get('name_with_seats_availability'):
-            return super().name_get()
-        res = []
+            return super()._compute_display_name()
         for event in self:
             # event or its tickets are sold out
             if event.event_registrations_sold_out:
@@ -606,8 +607,7 @@ def name_get(self):
                 )
             else:
                 name = event.name
-            res.append((event.id, name))
-        return res
+            event.display_name = name
 
     @api.returns('self', lambda value: value.id)
     def copy(self, default=None):

addons/event/models/event_registration.py

@@ -162,10 +162,11 @@ def write(self, vals):
 
         return ret
 
-    def name_get(self):
-        """ Custom name_get in case a registration is nott linked to an attendee
+    def _compute_display_name(self):
+        """ Custom display_name in case a registration is nott linked to an attendee
         """
-        return [(registration.id, registration.name or f"#{registration.id}") for registration in self]
+        for registration in self:
+            registration.display_name = registration.name or f"#{registration.id}"
 
     def toggle_active(self):
         pre_inactive = self - self.filtered(self._active_name)