credativ team mailing list archive

["Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>]

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/openupgrade-server/7.0-lp1226086-menu_parent_store into lp:openupgrade-server with lp:~therp-nl/openupgrade-server/7.0-deferred_steps as a prerequisite.

Commit message:
[FIX] lp:1226086, compute menu parent store. Courtesy of Sylvain LE GAL.

Requested reviews:
  OpenUpgrade Committers (openupgrade-committers)
Related bugs:
  Bug #1226086 in OpenUpgrade Server: "[7.0] menu item does'nt appear correctly"
  https://bugs.launchpad.net/openupgrade-server/+bug/1226086

For more details, see:
https://code.launchpad.net/~therp-nl/openupgrade-server/7.0-lp1226086-menu_parent_store/+merge/186944
-- 
https://code.launchpad.net/~therp-nl/openupgrade-server/7.0-lp1226086-menu_parent_store/+merge/186944
Your team OpenUpgrade Committers is requested to review the proposed merge of lp:~therp-nl/openupgrade-server/7.0-lp1226086-menu_parent_store into lp:openupgrade-server.

=== modified file 'openerp/openupgrade/deferred_70.py'
--- openerp/openupgrade/deferred_70.py	2013-09-22 18:52:32 +0000
+++ openerp/openupgrade/deferred_70.py	2013-09-22 18:52:32 +0000
@@ -53,3 +53,6 @@
 
 def migrate_deferred(cr, pool):
     sync_commercial_fields(cr, pool)
+    # Newly introduced _parent_store on menu items leaves gaps
+    # after upgrade of other modules for *some* reason. See lp:1226086
+    pool.get('ir.ui.menu')._parent_store_compute(cr)