From 4bfcbb2a480db6b72b6db84420a39dcf641bbbb1 Mon Sep 17 00:00:00 2001 From: Alexandre Fayolle Date: Wed, 25 Jun 2014 10:57:10 +0200 Subject: [PATCH] [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) --- openerp/osv/fields.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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: