vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #7042 from rohitwaghchaure/after_bench_migrate_system_uncheck_disable_prepared_report

Browse source 
fix: after bench migrate system uncheck Disable Prepared Report in the report
This commit is contained in:
rohitwaghchaure 2019-03-07 18:20:38 +05:30 • committed by GitHub
commit 952dcedcc9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.py
View file

@ -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: