diff options
| author | Alexandre Fayolle <afayolle.ml@free.fr> | 2014-06-25 10:57:10 +0200 |
|---|---|---|
| committer | Martin Trigaux <mat@openerp.com> | 2014-06-25 10:57:10 +0200 |
| commit | 4bfcbb2a480db6b72b6db84420a39dcf641bbbb1 (patch) | |
| tree | 3821dc40be48f7335f281789b8997c98fdbb011d | |
| parent | 50871bb4fe48a69bdc0256978d112d24882c3ff8 (diff) | |
[FIX] fields property: do not create empty properties
fixes #595
In the case where a property for the company exists but has no related record (e.g. in case of type m2o with no defined value), not setting a value to this field for a new record would create a new property (as browse_null is not an instance of browse_record)
| -rw-r--r-- | openerp/osv/fields.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/openerp/osv/fields.py b/openerp/osv/fields.py index 49421365852..040548ec7cc 100644 --- a/openerp/osv/fields.py +++ b/openerp/osv/fields.py @@ -1428,7 +1428,8 @@ class property(function): default_val = self._get_default(obj, cr, uid, prop_name, context) property_create = False - if isinstance(default_val, openerp.osv.orm.browse_record): + if isinstance(default_val, (openerp.osv.orm.browse_record, + openerp.osv.orm.browse_null)): if default_val.id != id_val: property_create = True elif default_val != id_val: |