From c9e4fbeda00428822a6042d51813e7c5b01131a2 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Mon, 20 Jul 2015 12:57:48 +0530 Subject: [PATCH] [fix][report] Show only active employee in Employee leave balance report --- .../employee_leave_balance/employee_leave_balance.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/erpnext/hr/report/employee_leave_balance/employee_leave_balance.py b/erpnext/hr/report/employee_leave_balance/employee_leave_balance.py index 1a70f45223..d6f865ba9f 100644 --- a/erpnext/hr/report/employee_leave_balance/employee_leave_balance.py +++ b/erpnext/hr/report/employee_leave_balance/employee_leave_balance.py @@ -9,8 +9,13 @@ from frappe.desk.reportview import execute as runreport def execute(filters=None): if not filters: filters = {} - employee_filters = filters.get("company") and \ - [["Employee", "company", "=", filters.get("company")]] or None + employee_filters = { + "status": "Active" + } + + if filters.get("company"): + filters["company"] = filters.company + employees = runreport(doctype="Employee", fields=["name", "employee_name", "department"], filters=employee_filters, limit_page_length=None)