From d84302682e4bd51d760887c9f3d24f5b97c45eff Mon Sep 17 00:00:00 2001 From: Anand Chitipothu Date: Fri, 3 Dec 2021 16:03:14 +0530 Subject: [PATCH] fix: permission issue on cohort page Using get_all instead of get_list to disable permission check. Issue #271 --- school/lms/doctype/cohort/cohort.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/school/lms/doctype/cohort/cohort.py b/school/lms/doctype/cohort/cohort.py index c2ace214..a2b2df08 100644 --- a/school/lms/doctype/cohort/cohort.py +++ b/school/lms/doctype/cohort/cohort.py @@ -24,7 +24,7 @@ class Cohort(Document): return subgroups def _get_subgroup_counts(self, doctype, **kw): - rows = frappe.get_list(doctype, + rows = frappe.get_all(doctype, filters={"cohort": self.name, **kw}, fields=['subgroup', 'count(*) as count'], group_by='subgroup')