From 3286ef93b8c087ab558db893635a51f4d33d640e Mon Sep 17 00:00:00 2001 From: Vo Minh Thu Date: Tue, 24 May 2011 12:18:10 +0200 Subject: [PATCH] [IMP] orm: added an assert on the return value of function fields. bzr revid: vmt@openerp.com-20110524101810-fdrj5hqxrc9c6856 --- openerp/osv/orm.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/openerp/osv/orm.py b/openerp/osv/orm.py index 94b7668b913..495b6242b3b 100644 --- a/openerp/osv/orm.py +++ b/openerp/osv/orm.py @@ -3053,6 +3053,9 @@ class orm(orm_template): for key, val in todo.items(): if key: res2 = self._columns[val[0]].get(cr, self, ids, val, user, context=context, values=res) + assert res2 is not None, \ + 'The function field "%s" on the "%s" model returned None\n' \ + '(a dictionary was expected).' % (val[0], self._name) for pos in val: for record in res: if isinstance(res2[record['id']], str): res2[record['id']] = eval(res2[record['id']]) #TOCHECK : why got string instend of dict in python2.6