Fix percentage column in vacation summary

Use the correct value for used hours
Igalia · May 7, 2024 · 333a498 · 333a498
1 parent 415f7fa
commit 333a498
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/model/facade/action/GetHolidaySummaryReportAction.php b/model/facade/action/GetHolidaySummaryReportAction.php
@@ -94,7 +94,7 @@ protected function doExecute()
             'usedHours' => round($usedHours, 2),
             'pendingHours' => round($summary['pendingHours'][$this->user->getLogin()], 2),
             'scheduledHours' => round($summary['scheduledHours'][$this->user->getLogin()] - $usedHours, 2),
-            'percentage' =>  $summary['availableHours'][$this->user->getLogin()] ? round(($summary['usedHours'][$this->user->getLogin()] / $summary['availableHours'][$this->user->getLogin()]) * 100, 2) : 0,
+            'percentage' =>  $summary['availableHours'][$this->user->getLogin()] ? round(($usedHours / $summary['availableHours'][$this->user->getLogin()]) * 100, 2) : 0,
             'hoursDay' => $validJourney,
             'holidays' => $leaves,
         ];