From e493303e1839cbc801c0b2ea939722a06d135e62 Mon Sep 17 00:00:00 2001 From: Olivier Dony Date: Wed, 12 Dec 2012 22:00:27 +0100 Subject: [PATCH] [REVERT] Undo r.4617, introduced a bug in menu hierarchy retrieval Browsing the menu data as super-user is not correct because when we load the children menu the ORM recursively calls menu.search(), this time as admin, hence the groups of the admin are applied or submenus, instead of the groups of the user. The patch was also un-necessary, so there is no reason to keep it. r.4617 = chs@openerp.com-20121129172145-0ionmbffc72hwxoa bzr revid: odo@openerp.com-20121212210027-i5yn1uyzmfho0jh0 --- openerp/addons/base/ir/ir_ui_menu.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/openerp/addons/base/ir/ir_ui_menu.py b/openerp/addons/base/ir/ir_ui_menu.py index 80e084502e5..a1f75800456 100644 --- a/openerp/addons/base/ir/ir_ui_menu.py +++ b/openerp/addons/base/ir/ir_ui_menu.py @@ -66,7 +66,7 @@ class ir_ui_menu(osv.osv): modelaccess = self.pool.get('ir.model.access') user_groups = set(self.pool.get('res.users').read(cr, SUPERUSER_ID, uid, ['groups_id'])['groups_id']) result = [] - for menu in self.browse(cr, SUPERUSER_ID, ids, context=context): + for menu in self.browse(cr, uid, ids, context=context): # this key works because user access rights are all based on user's groups (cfr ir_model_access.check) key = (cr.dbname, menu.id, tuple(user_groups)) if key in self._cache: