[FIX] models: bad setup of inherited custom fields

Assume a custom field F is defined on model 'res.partner'.  The setup of F may
silently fail because of missing stuff.  In that situation, setting up the
field inherited from F on model 'res.users' should also silently fail.

To reproduce the bug, install Invoicing, create a related custom field F on
'res.partner' with 'property_account_position_id.active', and install another
module.  Setting up F after loading module 'base' will fail because the field
'property_account_position_id' does not exist yet.  The error is not caught by
the inheritance of F on model 'res.users', and the installation crashes.

OPW 1835872
This commit is contained in:
Raphael Collet
2018-04-18 17:26:05 +02:00
parent 87b42ad9a8
commit 23431389c3
2 changed files with 4 additions and 2 deletions
+2 -1
View File
@@ -320,6 +320,7 @@ class Field(object):
'automatic': False, # whether the field is automatically created ("magic" field)
'inherited': False, # whether the field is inherited (_inherits)
'inherited_field': None, # the corresponding inherited field
'name': None, # name of the field
'model_name': None, # name of the model of this field
@@ -647,7 +648,7 @@ class Field(object):
@property
def base_field(self):
""" Return the base field of an inherited field, or ``self``. """
return self.related_field.base_field if self.inherited else self
return self.inherited_field.base_field if self.inherited_field else self
#
# Company-dependent fields
+2 -1
View File
@@ -2734,6 +2734,7 @@ class BaseModel(object):
# - copy inherited fields iff their original field is copied
fields[name] = field.new(
inherited=True,
inherited_field=field,
related=(parent_field, name),
related_sudo=False,
copy=field.copy,
@@ -2850,7 +2851,7 @@ class BaseModel(object):
try:
field.setup_full(self)
except Exception:
if partial and field.manual:
if partial and field.base_field.manual:
# Something goes wrong when setup a manual field.
# This can happen with related fields using another manual many2one field
# that hasn't been loaded because the comodel does not exist yet.