Shiloh/brotherton-erpnext
1
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed issue where no Holidays would cause the report to break

Browse Source
This commit is contained in:
Valmik Jangla 2016-01-30 10:59:40 +05:30
parent 6e33d91443
commit 0a76b8b29a
1 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
erpnext/hr/report/employee_holiday_attendance/employee_holiday_attendance.py
View File

@ -33,15 +33,17 @@ def get_employees():
for holiday in holidays:
holidays_list.append(holiday.holiday_date)
holiday_names[holiday.holiday_date] = holiday.description
if(holidays_list):
employee_list = frappe.db.sql("""select
employee, employee_name, att_date, status
from tabAttendance
where
att_date in ({0})""".format(', '.join(["%s"]*len(holidays_list))),
holidays_list, as_list=True)
employee_list = frappe.db.sql("""select
employee, employee_name, att_date, status
from tabAttendance
where
att_date in ({0})""".format(', '.join(["%s"]*len(holidays_list))),
holidays_list, as_list=True)
for employee_data in employee_list:
employee_data.append(holiday_names[employee_data[2]])
for employee_data in employee_list:
employee_data.append(holiday_names[employee_data[2]])
return employee_list
return employee_list
else:
return None