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

Allowing user to add leaves which may results in negative leave balance.

Browse Source
This commit is contained in:
mayur-patel 2013-03-05 16:21:00 +11:00
parent 9bba9ed4d2
commit 639392a0b1
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
hr/doctype/leave_application/leave_application.py
View File

@ -124,10 +124,13 @@ class DocType(DocListController):
if not is_lwp(self.doc.leave_type): if not is_lwp(self.doc.leave_type):
self.doc.leave_balance = get_leave_balance(self.doc.employee, self.doc.leave_balance = get_leave_balance(self.doc.employee,
self.doc.leave_type, self.doc.fiscal_year)["leave_balance"] self.doc.leave_type, self.doc.fiscal_year)["leave_balance"]
# Allowing user to add leavs which will result in negative balance. This is needed for Sick Leave and other common exceptional cases.
#System will check and warn but continue with saving the application.
#This may needs to be changed if other ERPNext customer may not want this behaviour.
if self.doc.leave_balance - self.doc.total_leave_days < 0: if self.doc.leave_balance - self.doc.total_leave_days < 0:
msgprint("There is not enough leave balance for Leave Type: %s" % \ msgprint("There is not enough leave balance for Leave Type: %s" % \
(self.doc.leave_type,), raise_exception=1) (self.doc.leave_type,))
# , raise_exception=1)
def validate_leave_overlap(self): def validate_leave_overlap(self):
if not self.doc.name: if not self.doc.name: