openerp-community-reviewer team mailing list archive

[Yann Papouin <yann.papouin@xxxxxxxxx>]

Yann Papouin has proposed merging lp:~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start into lp:ocb-server/6.1.

Requested reviews:
  OpenERP Community Backports Team (ocb)
Related bugs:
  Bug #944197 in OpenERP Community Backports (Server): "Action Rules don't work right after server is started"
  https://bugs.launchpad.net/ocb-server/+bug/944197

For more details, see:
https://code.launchpad.net/~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start/+merge/189219

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

Backport from trunk
-- 
https://code.launchpad.net/~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start/+merge/189219
Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~yann-papouin/ocb-server/6.1-bug-944197-register-hook-server-start into lp:ocb-server/6.1.

=== modified file 'openerp/modules/loading.py'
--- openerp/modules/loading.py	2012-02-20 10:59:43 +0000
+++ openerp/modules/loading.py	2013-10-04 06:55:56 +0000
@@ -415,6 +415,11 @@
             cr.commit()
 
         _logger.info('Modules loaded.')
+        
+        # STEP 7: call _register_hook on every model
+        for model in pool.models.values():
+            model._register_hook(cr)    
+                
     finally:
         cr.close()
 

=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2013-06-20 14:02:50 +0000
+++ openerp/osv/orm.py	2013-10-04 06:55:56 +0000
@@ -4987,7 +4987,11 @@
             if command in (0, 1): item.update(record)
             record_dicts.append(item)
         return record_dicts
-
+    
+    def _register_hook(self, cr):
+        """ stuff to do right after the registry is built """
+        pass    
+    
 # keep this import here, at top it will cause dependency cycle errors
 import expression