Merge lp:~openerp-dev/openobject-server/trunk-imp-register_hook_improved-cto into lp:openobject-server

[Samus CTO (OpenERP)]

Hello,

Here is a fix for the missing commit after calling _register_hook and also I added the update_module parameter which is very useful to know after having loaded a pool and can't be guessed with the tools.config because this variable is reset before the call of _register_hook.

This branch is related: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-imp-register_hook_improved-cto

Thanks

Unmerged revisions

5081. By Samus CTO (OpenERP) on 2014-02-18

[IMP] after all _register_hook calls, the cursor is committed + added the update_module flag in argument of the call because it can be useful

=== modified file 'openerp/modules/loading.py'
--- openerp/modules/loading.py	2014-02-09 00:40:05 +0000
+++ openerp/modules/loading.py	2014-02-18 12:50:46 +0000
@@ -419,7 +419,9 @@
 
         # STEP 8: call _register_hook on every model
         for model in registry.models.values():
-            model._register_hook(cr)
+            model._register_hook(cr, update_module)
+        # final commit to apply everything did in the _register_hook calls
+        cr.commit()
 
     finally:
         cr.close()

=== modified file 'openerp/osv/orm.py'
--- openerp/osv/orm.py	2014-02-14 14:52:19 +0000
+++ openerp/osv/orm.py	2014-02-18 12:50:46 +0000
@@ -5033,7 +5033,7 @@
         index = dict((r['id'], r) for r in result)
         return [index[x] for x in record_ids if x in index]
 
-    def _register_hook(self, cr):
+    def _register_hook(self, cr, update_module):
         """ stuff to do right after the registry is built """
         pass