From ef9f126cb2651c5c37b5f060d7574a22cad42bb5 Mon Sep 17 00:00:00 2001 From: Nikhil Ponnuru Date: Fri, 7 May 2021 10:06:29 +0530 Subject: [PATCH] fix: change get_all to get_list in employee leave balance summary report to hide descendants when hide descendants is selected in user permissions (#25599) Co-authored-by: nikhilponnuru Co-authored-by: Rucha Mahabal --- .../employee_leave_balance_summary.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/erpnext/hr/report/employee_leave_balance_summary/employee_leave_balance_summary.py b/erpnext/hr/report/employee_leave_balance_summary/employee_leave_balance_summary.py index 92715d3445..e86fa2b1c4 100644 --- a/erpnext/hr/report/employee_leave_balance_summary/employee_leave_balance_summary.py +++ b/erpnext/hr/report/employee_leave_balance_summary/employee_leave_balance_summary.py @@ -47,7 +47,7 @@ def get_data(filters, leave_types): user = frappe.session.user conditions = get_conditions(filters) - active_employees = frappe.get_all("Employee", + active_employees = frappe.get_list("Employee", filters=conditions, fields=["name", "employee_name", "department", "user_id", "leave_approver"]) @@ -72,4 +72,4 @@ def get_data(filters, leave_types): data.append(row) - return data \ No newline at end of file + return data