nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-06-05 13:24:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: fetch employee department (#19433)

Browse Source
This commit is contained in:
Mangesh-Khairnar
2019-10-31 16:00:52 +05:30
committed by Nabin Hait
parent 519ca54f53
commit 45c18b3184
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/hr/report/employee_leave_balance_summary/employee_leave_balance_summary.py
View File

@@ -75,7 +75,7 @@ def get_data(filters):
leave_approvers = department_approver_map.get(employee.department_name, []).append(employee.leave_approver) leave_approvers = department_approver_map.get(employee.department_name, []).append(employee.leave_approver)
if (len(leave_approvers) and user in leave_approvers) or (user in ["Administrator", employee.user_id]) \ if (leave_approvers and len(leave_approvers) and user in leave_approvers) or (user in ["Administrator", employee.user_id]) \
or ("HR Manager" in frappe.get_roles(user)): or ("HR Manager" in frappe.get_roles(user)):
row = frappe._dict({ row = frappe._dict({
'employee': employee.name, 'employee': employee.name,
@@ -111,10 +111,10 @@ def get_conditions(filters):
def get_department_leave_approver_map(department=None): def get_department_leave_approver_map(department=None):
conditions='' conditions=''
if department: if department:
conditions='and department_name = %(department)s or parent_department = %(department)s'%{'department': department} conditions="and (department_name = '%(department)s' or parent_department = '%(department)s')"%{'department': department}
# get current department and all its child # get current department and all its child
department_list = frappe.db.sql_list(''' SELECT name FROM `tabDepartment` WHERE disabled=0 {0}'''.format(conditions)) #nosec department_list = frappe.db.sql_list(""" SELECT name FROM `tabDepartment` WHERE disabled=0 {0}""".format(conditions)) #nosec
# retrieve approvers list from current department and from its subsequent child departments # retrieve approvers list from current department and from its subsequent child departments
approver_list = frappe.get_all('Department Approver', filters={ approver_list = frappe.get_all('Department Approver', filters={