Merge lp:~openerp-groupes/openobject-server/trunk-context-constraints into lp:openobject-server

[Olivier Ligot]

This patch passes the context as argument when calling objects constraints.
This allow to use the context in constraints.

Note that the proposal is for trunk and not 6.0 as this is an API change (every constraint must now have context as their last argument in their signature).

[Vo Minh Thu (thu)]

Have you a specific use case for this?

[Olivier Ligot (ligot)]

In our modules, we use the context as a placeholder for extra informations needed by a function when the signature can not be changed (ex: constraints).
For example, we use it to store a (time consuming) computed values that we can pass around thanks to the context (see commits http://bazaar.launchpad.net/~openerp-groupes/hrself-addons/6.0/revision/517 and http://bazaar.launchpad.net/~openerp-groupes/hrself-addons/6.0/revision/518)

[Vo Minh Thu (thu)]

This comment to let you know we will merge this patch, but we have first to check if all addons handle such an additional argument.

Otherwise, I'm not sure the context is the best place to do caching. It should probably be implemented in the function computing the value.

[Antony Lesuisse (OpenERP) (al-openerp)]

This does not make sense. Constraints should only depends on the actual data. If you call the constraints multiple times on the same data it must always give the same result and therefore cannot be influenced by context.

review: Disapprove

Unmerged revisions

3428. By Olivier Ligot on 2011-05-25

[IMP] Pass context as argument when calling constraints functions

=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2011-05-24 10:18:10 +0000
+++ openerp/osv/orm.py	2011-05-25 12:16:16 +0000
@@ -926,7 +926,7 @@
         error_msgs = []
         for constraint in self._constraints:
             fun, msg, fields = constraint
-            if not fun(self, cr, uid, ids):
+            if not fun(self, cr, uid, ids, context=context):
                 # Check presence of __call__ directly instead of using
                 # callable() because it will be deprecated as of Python 3.0
                 if hasattr(msg, '__call__'):