| commit | ccb15de1c9e4c23487787b3f8be3bf3e8d7dfb86 | [log] [tgz] |
|---|---|---|
| author | Hussain Nagaria <hussainbhaitech@gmail.com> | Mon Apr 05 11:23:06 2021 +0530 |
| committer | Hussain Nagaria <hussainbhaitech@gmail.com> | Mon Apr 05 11:23:06 2021 +0530 |
| tree | 0722d288e0c7dd8d0bf2041306bb7dc372d24239 | |
| parent | 40abd2a2e12b52a814dd414115e7b29e8729a0dd [diff] |
fix: Handle overtime edge case
diff --git a/erpnext/projects/report/employee_hours_utilisation_based_on_timesheet/employee_hours_utilisation_based_on_timesheet.py b/erpnext/projects/report/employee_hours_utilisation_based_on_timesheet/employee_hours_utilisation_based_on_timesheet.py index c27efc9..19cb1c9 100644 --- a/erpnext/projects/report/employee_hours_utilisation_based_on_timesheet/employee_hours_utilisation_based_on_timesheet.py +++ b/erpnext/projects/report/employee_hours_utilisation_based_on_timesheet/employee_hours_utilisation_based_on_timesheet.py
@@ -134,6 +134,11 @@ for emp, data in iteritems(self.stats_by_employee): data['total_hours'] = TOTAL_HOURS data['untracked_hours'] = flt(TOTAL_HOURS - data['billed_hours'] - data['non_billed_hours'], 2) + + # To handle overtime edge-case + if data['untracked_hours'] < 0: + data['untracked_hours'] = 0.0 + data['per_util'] = flt(((data['billed_hours'] + data['non_billed_hours']) / TOTAL_HOURS) * 100, 2) def generate_report_summary(self):