From af24bbff1bcf3614e310e80cfac52d01b5181044 Mon Sep 17 00:00:00 2001 From: Cleo Menezes Jr <54215258+CleoMenezesJr@users.noreply.github.com> Date: Fri, 30 Sep 2022 15:40:55 -0300 Subject: [PATCH] fix: Show Report button should appear only if the doc is saved (#18268) --- frappe/core/doctype/report/report.js | 43 +++++++++++++++------------- 1 file changed, 23 insertions(+), 20 deletions(-) diff --git a/frappe/core/doctype/report/report.js b/frappe/core/doctype/report/report.js index c912e217a6..0e0bfeea9a 100644 --- a/frappe/core/doctype/report/report.js +++ b/frappe/core/doctype/report/report.js @@ -8,26 +8,29 @@ frappe.ui.form.on("Report", { } let doc = frm.doc; - frm.add_custom_button( - __("Show Report"), - function () { - switch (doc.report_type) { - case "Report Builder": - frappe.set_route("List", doc.ref_doctype, "Report", doc.name); - break; - case "Query Report": - frappe.set_route("query-report", doc.name); - break; - case "Script Report": - frappe.set_route("query-report", doc.name); - break; - case "Custom Report": - frappe.set_route("query-report", doc.name); - break; - } - }, - "fa fa-table" - ); + if (!doc.__islocal) { + frm.add_custom_button( + __("Show Report"), + function () { + switch (doc.report_type) { + case "Report Builder": + frappe.set_route("List", doc.ref_doctype, "Report", doc.name); + break; + case "Query Report": + frappe.set_route("query-report", doc.name); + break; + case "Script Report": + frappe.set_route("query-report", doc.name); + break; + case "Custom Report": + frappe.set_route("query-report", doc.name); + break; + } + }, + "fa fa-table" + ); + } + if (doc.is_standard === "Yes" && frm.perm[0].write) { frm.add_custom_button(