From e8f50932e3a2c6c8e91bd55ff5bdb1347924e79e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Krier?= Date: Sat, 27 Mar 2010 15:58:31 +0100 Subject: [PATCH] Don't validate Function fields without setter It is a waste of resource to validate such Function fields because the value doesn't depend on data users enter. --- trytond/model/modelstorage.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/trytond/model/modelstorage.py b/trytond/model/modelstorage.py index 8e2dc5a036..47bee2106c 100644 --- a/trytond/model/modelstorage.py +++ b/trytond/model/modelstorage.py @@ -973,6 +973,9 @@ class ModelStorage(Model): context.update(ctx_pref) records = self.browse(cursor, user, ids, context=context) for field_name, field in self._columns.iteritems(): + if isinstance(field, fields.Function) and \ + not field.setter: + continue # validate domain if field._type in ('many2one', 'many2many', 'one2many') \ and field.domain: