Merge lp:~yann-papouin/ocb-addons/6.1-bug-944197-action-rule-register-hook into lp:ocb-addons/6.1

[Yann Papouin]

Automatically derived from https://code.launchpad.net/~yann-papouin/openobject-addons/6.1-bug-944197-action-rule-register-hook for https://code.launchpad.net/~openerp/openobject-addons/6.1. Below is a copy of the original description.

Changes are mainly to stay compatible with openobject-server

Server part here: https://code.launchpad.net/~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start/+merge/189219

[Stefan Rijnhart (Opener) (stefan-opener)]

LGTM, thanks! Note that this part is always safe to merge or update even if you don't update the server on your OpenERP installation. I think we can get the serverside equally independent from the other part.

review: Approve

[Stefan Rijnhart (Opener) (stefan-opener)]

Thanks!

review: Approve

[Holger Brunn (Therp) (hbrunn)]

review: Approve (code review)

=== modified file 'base_action_rule/base_action_rule.py'
--- base_action_rule/base_action_rule.py	2012-04-26 08:03:00 +0000
+++ base_action_rule/base_action_rule.py	2013-10-08 13:54:53 +0000
@@ -20,6 +20,7 @@
 ##############################################################################
 
 from osv import fields, osv, orm
+from openerp import SUPERUSER_ID
 from tools.translate import _
 from datetime import datetime
 from datetime import timedelta
@@ -202,11 +203,13 @@
             return True
         return wrapper
 
-    def _register_hook(self, cr, uid, ids, context=None):
+    def _register_hook(self, cr, uid=SUPERUSER_ID, ids=None, context=None):
         """
         Wrap every `create` and `write` methods of the models specified by
         the rules (given by `ids`).
         """
+        if ids == [] or ids is None:
+            ids = self.search(cr, uid, [])
         for action_rule in self.browse(cr, uid, ids, context=context):
             model = action_rule.model_id.model
             obj_pool = self.pool.get(model)