[IMP] fields.property: be more explicit when testing if something is a browse_record.

bzr revid: vmt@openerp.com-20111005154857-ic3yzb3dead33us3

diff --git a/openerp/osv/fields.py b/openerp/osv/fields.py
index dc6a05b..4e226f1 100644 (file)
--- a/openerp/osv/fields.py
+++ b/openerp/osv/fields.py
@@ -41,6 +41,7 @@ import warnings
 import xmlrpclib
 from psycopg2 import Binary
 
+import openerp
 import openerp.netsvc as netsvc
 import openerp.tools as tools
 from openerp.tools.translate import _
@@ -1216,10 +1217,10 @@ class property(function):
         default_val = self._get_default(obj, cr, uid, prop_name, context)
 
         property_create = False
-        if hasattr(default_val, '_id') and default_val:
-            if default_val._id != id_val:
+        if isinstance(default_val, openerp.osv.orm.browse_record):
+            if default_val.id != id_val:
                 property_create = True
-        elif id_val != default_val:
+        elif default_val != id_val:
             property_create = True
 
         if property_create: