From 5722d9c2ba4d88774b160941c5dc075dd6dfdb30 Mon Sep 17 00:00:00 2001 From: 18alantom <2.alan.tom@gmail.com> Date: Wed, 29 Nov 2023 12:04:18 +0530 Subject: [PATCH] fix: set sidebar only if not present The get_context of some templates set their own sidebar if this is done, no need to override --- frappe/website/page_renderers/template_page.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/frappe/website/page_renderers/template_page.py b/frappe/website/page_renderers/template_page.py index 2feda50881..e04649d89c 100644 --- a/frappe/website/page_renderers/template_page.py +++ b/frappe/website/page_renderers/template_page.py @@ -109,7 +109,8 @@ class TemplatePage(BaseTemplatePage): super().post_process_context() def add_sidebar_and_breadcrumbs(self): - self.context.sidebar_items = get_sidebar_items(self.context.website_sidebar, self.basepath) + if not self.context.sidebar_items: + self.context.sidebar_items = get_sidebar_items(self.context.website_sidebar, self.basepath) if self.context.add_breadcrumbs and not self.context.parents: parent_path = os.path.dirname(self.path)