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

feat: set leave allocation on carry forward check

Browse Source
This commit is contained in:
Mangesh-Khairnar 2019-04-05 11:58:35 +05:30
parent 1208ca6a36
commit 7c6b6eae5b
1 changed files with 31 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
erpnext/hr/doctype/leave_allocation/leave_allocation.py
View File

@ -62,11 +62,18 @@ class LeaveAllocation(Document):
if flt(self.new_leaves_allocated) % 0.5: if flt(self.new_leaves_allocated) % 0.5:
frappe.throw(_("Leaves must be allocated in multiples of 0.5"), ValueMultiplierError) frappe.throw(_("Leaves must be allocated in multiples of 0.5"), ValueMultiplierError)
def validate_allocation_overlap(self): def validate_allocation_overlap(self, carry_forward=0):
leave_allocation = frappe.db.sql(""" leave_allocation = frappe.db.sql("""
select name from `tabLeave Allocation` SELECT
where employee=%s and leave_type=%s and docstatus=1 name
and to_date >= %s and from_date <= %s""", FROM `tabLeave Allocation`
WHERE
employee=%s
AND leave_type=%s
AND docstatus=1
AND is_carry_forward={0}
AND to_date >= %s
AND from_date <= %s""".format(carry_forward),
(self.employee, self.leave_type, self.from_date, self.to_date)) (self.employee, self.leave_type, self.from_date, self.to_date))
if leave_allocation: if leave_allocation:
@ -111,6 +118,11 @@ class LeaveAllocation(Document):
else: else:
frappe.throw(_("Total allocated leaves {0} cannot be less than already approved leaves {1} for the period").format(self.total_leaves_allocated, leaves_taken), LessAllocationError) frappe.throw(_("Total allocated leaves {0} cannot be less than already approved leaves {1} for the period").format(self.total_leaves_allocated, leaves_taken), LessAllocationError)
def set_carry_forward_leaves(self):
self.validate_allocation_overlap(carry_forward=1)
self.old_leaves_allocated = get_carry_forwarded_leaves(self.employee, self.leave_type,
self.from_date, self.is_carry_forward)
def get_leave_allocation_for_period(employee, leave_type, from_date, to_date): def get_leave_allocation_for_period(employee, leave_type, from_date, to_date):
leave_allocated = 0 leave_allocated = 0
leave_allocations = frappe.db.sql(""" leave_allocations = frappe.db.sql("""
@ -135,17 +147,25 @@ def get_leave_allocation_for_period(employee, leave_type, from_date, to_date):
return leave_allocated return leave_allocated
@frappe.whitelist() @frappe.whitelist()
def get_carry_forwarded_leaves(employee, leave_type, date, carry_forward=None): def get_carry_forwarded_leaves(employee, leave_type, date, is_carry_forward=None):
carry_forwarded_leaves = 0 carry_forwarded_leaves = 0
if carry_forward: if is_carry_forward:
validate_carry_forward(leave_type) validate_carry_forward(leave_type)
previous_allocation = frappe.db.sql(""" previous_allocation = frappe.db.sql("""
select name, from_date, to_date, total_leaves_allocated SELECT
from `tabLeave Allocation` name,
where employee=%s and leave_type=%s and docstatus=1 and to_date < %s from_date,
order by to_date desc limit 1 to_date,
total_leaves_allocated
FROM `tabLeave Allocation`
WHERE
employee=%s
AND leave_type=%s
AND docstatus=1
AND to_date < %s
ORDER BY to_date desc limit 1
""", (employee, leave_type, date), as_dict=1) """, (employee, leave_type, date), as_dict=1)
if previous_allocation: if previous_allocation:
leaves_taken = get_approved_leaves_for_period(employee, leave_type, leaves_taken = get_approved_leaves_for_period(employee, leave_type,
@ -157,4 +177,4 @@ def get_carry_forwarded_leaves(employee, leave_type, date, carry_forward=None):
def validate_carry_forward(leave_type): def validate_carry_forward(leave_type):
if not frappe.db.get_value("Leave Type", leave_type, "is_carry_forward"): if not frappe.db.get_value("Leave Type", leave_type, "is_carry_forward"):
frappe.throw(_("Leave Type {0} cannot be carry-forwarded").format(leave_type)) frappe.throw(_("Leave Type {0} cannot be carry-forwarded").format(leave_type))