Merge pull request #27850 from pateljannat/auto-attendance-issue

fix: exclude inactive  employees from auto attendance
This commit is contained in:
Jannat Patel 2021-10-14 12:48:22 +05:30 committed by GitHub
commit 8de5f1f04d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -97,7 +97,7 @@ class ShiftType(Document):
assigned_employees = [x[0] for x in assigned_employees]
if consider_default_shift:
filters = {'default_shift': self.name}
filters = {'default_shift': self.name, 'status': ['!=', 'Inactive']}
default_shift_employees = frappe.get_all('Employee', 'name', filters, as_list=True)
default_shift_employees = [x[0] for x in default_shift_employees]
return list(set(assigned_employees+default_shift_employees))