From 80bb7cba8f25b0ca6e5679f2ce64ea0b8ba52cca Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Thu, 7 Mar 2019 17:31:09 +0530 Subject: [PATCH] fix: after bench migrate system uncheck Disable Prepared Report in the report --- .../role_permission_for_page_and_report.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.py b/frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.py index d7462d1d95..f5081ef595 100644 --- a/frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.py +++ b/frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.py @@ -66,7 +66,9 @@ class RolePermissionforPageandReport(Document): def update_disable_prepared_report(self): if self.report: - frappe.db.set_value('Report', self.report, 'disable_prepared_report', self.disable_prepared_report) + # intentionally written update query in frappe.db.sql instead of frappe.db.set_value + frappe.db.sql(""" update `tabReport` set disable_prepared_report = %s + where name = %s""", (self.disable_prepared_report, self.report)) def get_args(self, row=None): name = self.page if self.set_role_for == 'Page' else self.report @@ -75,7 +77,7 @@ class RolePermissionforPageandReport(Document): return { check_for_field: name } - + def get_roles(self): roles = [] for data in self.roles: