Skip to content

Commit

Permalink
Merge pull request #50 from eporsche/fix_transfer_vacation_entitlement
Browse files Browse the repository at this point in the history
fix transfer of vacation entitlements
  • Loading branch information
eporsche committed Feb 9, 2023
2 parents 63e141a + 55a2e96 commit 622d67f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/Actions/TransferVacationEntitlement.php
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ public function transfer(User $employee, $vacationEntitlementId)
$vacationEntitlementToBeTransferred = VacationEntitlement::findOrFail($vacationEntitlementId);

//fetch a possible vacation entitlement candiate
$useExistingVacationEntitlement = VacationEntitlement::where('ends_at', $vacationEntitlementToBeTransferred->end_of_transfer_period)->first();
$useExistingVacationEntitlement = $employee->vacationEntitlements()->where('ends_at', $vacationEntitlementToBeTransferred->end_of_transfer_period)->first();
if (!empty($useExistingVacationEntitlement)) {
$vacationEntitlementToBeTransferred->transferVacationDays($useExistingVacationEntitlement, [
'days' => $useExistingVacationEntitlement->days->plus($vacationEntitlementToBeTransferred->daysTransferrable())
Expand Down

0 comments on commit 622d67f

Please sign in to comment.