From 7fe33574ffc615f31a4065e8bd56a5897dcd99d4 Mon Sep 17 00:00:00 2001 From: deepeshgarg007 Date: Mon, 15 Apr 2019 16:32:23 +0530 Subject: [PATCH] fix: Show button for custom report --- frappe/core/doctype/report/report.js | 33 +++------------------------- 1 file changed, 3 insertions(+), 30 deletions(-) diff --git a/frappe/core/doctype/report/report.js b/frappe/core/doctype/report/report.js index c464e772a1..9afc247c06 100644 --- a/frappe/core/doctype/report/report.js +++ b/frappe/core/doctype/report/report.js @@ -30,6 +30,9 @@ cur_frm.cscript.refresh = function(doc) { 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"); @@ -58,36 +61,6 @@ frappe.ui.form.on('Report', { // make the document read-only frm.set_read_only(); } - - 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.is_standard === "Yes") { - frm.add_custom_button(doc.disabled ? __("Enable Report") : __("Disable Report"), function() { - frm.call('toggle_disable', { - disable: doc.disabled ? 0 : 1 - }).then(() => { - frm.reload_doc(); - }); - }, doc.disabled ? "fa fa-check" : "fa fa-off"); - } - - frm.events.report_type(frm); }, ref_doctype: function(frm) {