From f7c930d50d7baf5b6281c34811de158efa93d5be Mon Sep 17 00:00:00 2001 From: prssanna Date: Wed, 17 Feb 2021 13:01:13 +0530 Subject: [PATCH] fix: don't show report builder link on report builder route --- frappe/public/js/frappe/list/list_view_select.js | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/frappe/public/js/frappe/list/list_view_select.js b/frappe/public/js/frappe/list/list_view_select.js index 0741108176..e1063c5052 100644 --- a/frappe/public/js/frappe/list/list_view_select.js +++ b/frappe/public/js/frappe/list/list_view_select.js @@ -51,10 +51,15 @@ frappe.views.ListViewSelect = class ListViewSelect { action: () => this.set_route("report"), current_view_handler: () => { const reports = this.get_reports(); - this.setup_dropdown_in_sidebar("Report", reports, { - label: __("Report Builder"), - action: () => this.set_route("report") - }); + let default_action = {}; + // Only add action if current route is not report builder + if (frappe.get_route().length > 3) { + default_action = { + label: __("Report Builder"), + action: () => this.set_route("report") + }; + } + this.setup_dropdown_in_sidebar("Report", reports, default_action); } }, Dashboard: {