From 9a23312ad7ca3de2d4db78c5bccd67cd1a7bef1c Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Tue, 12 Nov 2019 13:37:51 +0530 Subject: [PATCH] fix(Report): validate 'Script Manager' role only for report type 'Script Report' --- frappe/core/doctype/report/report.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/core/doctype/report/report.py b/frappe/core/doctype/report/report.py index f71179d388..e88a2d3335 100644 --- a/frappe/core/doctype/report/report.py +++ b/frappe/core/doctype/report/report.py @@ -30,7 +30,7 @@ class Report(Document): if self.is_standard == "No": # allow only script manager to edit scripts - if frappe.session.user!="Administrator": + if self.report_type == 'Script Report' and frappe.conf.server_script_enabled and frappe.session.user != "Administrator": frappe.only_for('Script Manager', True) if frappe.db.get_value("Report", self.name, "is_standard") == "Yes":