Field display_name becomes non-stored

[steync027]

Impacted versions:

Tested on 12.0

Steps to reproduce:

Create a new module "module_abc"

class ResPartner(models.Model):
    _inherit = 'res.partner'

    @api.depends('field_1', 'field_b')
    def _compute_display_name(self):
        for record in self:
            record.display_name = 'This logic is irrelevant'

    display_name = fields.Char(compute=_compute_display_name, store=True)

Create a second module "module_def"

class ResPartner(models.Model):
    _inherit = 'res.partner'

    some_other_field1 = fields.Char()
    some_other_field2 = fields.Char()

Current behavior:

• Once you've installed module_abc the display_name is stored

• Once you install module_def the display_name is no longer stored

• Even though module_def does not change anything on display_name, the act of inheriting from res.partner will reverse what you have done in module_abc

• This happens on any model, not only res.partner

Expected behavior:

• The display_name should be still be stored

Extra references:

• https://github.com/OCA/multi-company/pull/165/files

[steync027]

@pedrobaeza I am tagging you because I have a problem exactly the same as you had in the OCA module. Your solution in that module was to re-declare the display_name and re-add the store=True, however I am wondering if this cannot be fixed in Odoo core. If you have any insights I would value it highly.

[pedrobaeza]

I didn't investigate such problem, but it seems to fix it. It may be an ORM limitation, as a such, difficult to be fixed, and not for an stable version. Anyway, I let it tagged for reference.

[steync027]

Thanks @pedrobaeza I will investigate this further and possibly provide a PR for the issue.

[steync027]

@pedrobaeza I have proposed a PR

[pedrobaeza]

Great, let's see what Odoo thinks about it