[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:
+2
-1
@@ -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
@@ -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.
|
||||
|
||||
Reference in New Issue
Block a user