From 30f24d1ca7d467dd3194eb75c44ba20a56f36f7f Mon Sep 17 00:00:00 2001 From: Jeremy Kersten Date: Fri, 13 Mar 2020 15:16:15 +0000 Subject: [PATCH] [FIX] website: perf - don't search backend menu if you will not see it Before this commit, in all cases, we search the backend menu to open if you click on edit backend menu. But the backend menu edit is useless for non editor user. This feature has been introduce in v13, since eLearning have his own apps, so when you edit in backend, you want to see the slide form view, into the elearning app and no website App. Related to #47257 task-2211013 X-original-commit: 1da7e53f4cdd82574688eb62f7a1cb8ddca871e6 --- addons/website/models/ir_ui_view.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/addons/website/models/ir_ui_view.py b/addons/website/models/ir_ui_view.py index c2eb62acfb5..8a20ecc7f25 100644 --- a/addons/website/models/ir_ui_view.py +++ b/addons/website/models/ir_ui_view.py @@ -384,8 +384,9 @@ class View(models.Model): elif request.env.user.has_group('website.group_website_publisher'): new_context = dict(self._context, inherit_branding_auto=True) if values and 'main_object' in values: - func = getattr(values['main_object'], 'get_backend_menu_id', False) - values['backend_menu_id'] = func and func() or self.env.ref('website.menu_website_configuration').id + if request.env.user.has_group('website.group_website_publisher'): + func = getattr(values['main_object'], 'get_backend_menu_id', False) + values['backend_menu_id'] = func and func() or self.env.ref('website.menu_website_configuration').id # Fallback incase main_object dont't inherit 'website.seo.metadata' if not hasattr(values['main_object'], 'get_website_meta'):