#37 - cancel vacation request (#53)

* #37 - cancel vacation request

* #37 - fix

* #37 - fix

app/Architecture/Providers/EventServiceProvider.php

@@ -8,12 +8,14 @@ use Illuminate\Foundation\Support\Providers\EventServiceProvider as ServiceProvi
 use Toby\Domain\Events\VacationRequestAcceptedByAdministrative;
 use Toby\Domain\Events\VacationRequestAcceptedByTechnical;
 use Toby\Domain\Events\VacationRequestApproved;
+use Toby\Domain\Events\VacationRequestCancelled;
 use Toby\Domain\Events\VacationRequestCreated;
 use Toby\Domain\Events\VacationRequestStateChanged;
 use Toby\Domain\Listeners\CreateVacationRequestActivity;
 use Toby\Domain\Listeners\HandleAcceptedByAdministrativeVacationRequest;
 use Toby\Domain\Listeners\HandleAcceptedByTechnicalVacationRequest;
 use Toby\Domain\Listeners\HandleApprovedVacationRequest;
+use Toby\Domain\Listeners\HandleCancelledVacationRequest;
 use Toby\Domain\Listeners\HandleCreatedVacationRequest;
 
 class EventServiceProvider extends ServiceProvider
@@ -24,5 +26,6 @@ class EventServiceProvider extends ServiceProvider
         VacationRequestAcceptedByTechnical::class => [HandleAcceptedByTechnicalVacationRequest::class],
         VacationRequestAcceptedByAdministrative::class => [HandleAcceptedByAdministrativeVacationRequest::class],
         VacationRequestApproved::class => [HandleApprovedVacationRequest::class],
+        VacationRequestCancelled::class => [HandleCancelledVacationRequest::class],
     ];
 }

app/Domain/Enums/VacationRequestState.php

@@ -7,7 +7,7 @@ namespace Toby\Domain\Enums;
 enum VacationRequestState: string
 {
     case Created = "created";
-    case Canceled = "canceled";
+    case Cancelled = "cancelled";
     case Rejected = "rejected";
     case Approved = "approved";
     case WaitingForTechnical = "waiting_for_technical";
@@ -40,7 +40,7 @@ enum VacationRequestState: string
     {
         return [
             self::Rejected,
-            self::Canceled,
+            self::Cancelled,
         ];
     }
 

app/Domain/Events/VacationRequestCancelled.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Toby\Domain\Events;
+
+use Illuminate\Foundation\Events\Dispatchable;
+use Illuminate\Queue\SerializesModels;
+use Toby\Eloquent\Models\VacationRequest;
+
+class VacationRequestCancelled
+{
+    use Dispatchable;
+    use SerializesModels;
+
+    public function __construct(
+        public VacationRequest $vacationRequest,
+    ) {
+    }
+}

app/Domain/Listeners/HandleApprovedVacationRequest.php

@@ -5,16 +5,10 @@ declare(strict_types=1);
 namespace Toby\Domain\Listeners;
 
 use Toby\Domain\Events\VacationRequestApproved;
-use Toby\Domain\VacationTypeConfigRetriever;
 use Toby\Infrastructure\Jobs\SendVacationRequestDaysToGoogleCalendar;
 
 class HandleApprovedVacationRequest
 {
-    public function __construct(
-        protected VacationTypeConfigRetriever $configRetriever,
-    ) {
-    }
-
     public function handle(VacationRequestApproved $event): void
     {
         SendVacationRequestDaysToGoogleCalendar::dispatch($event->vacationRequest);

app/Domain/Listeners/HandleCancelledVacationRequest.php

@@ -0,0 +1,16 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Toby\Domain\Listeners;
+
+use Toby\Domain\Events\VacationRequestCancelled;
+use Toby\Infrastructure\Jobs\ClearVacationRequestDaysInGoogleCalendar;
+
+class HandleCancelledVacationRequest
+{
+    public function handle(VacationRequestCancelled $event): void
+    {
+        ClearVacationRequestDaysInGoogleCalendar::dispatch($event->vacationRequest);
+    }
+}

app/Domain/VacationRequestStateManager.php

@@ -10,6 +10,7 @@ use Toby\Domain\Enums\VacationRequestState;
 use Toby\Domain\Events\VacationRequestAcceptedByAdministrative;
 use Toby\Domain\Events\VacationRequestAcceptedByTechnical;
 use Toby\Domain\Events\VacationRequestApproved;
+use Toby\Domain\Events\VacationRequestCancelled;
 use Toby\Domain\Events\VacationRequestCreated;
 use Toby\Eloquent\Models\VacationRequest;
 
@@ -42,7 +43,9 @@ class VacationRequestStateManager
 
     public function cancel(VacationRequest $vacationRequest): void
     {
-        $this->changeState($vacationRequest, VacationRequestState::Canceled);
+        $this->changeState($vacationRequest, VacationRequestState::Cancelled);
+
+        $this->dispatcher->dispatch(new VacationRequestCancelled($vacationRequest));
     }
 
     public function acceptAsTechnical(VacationRequest $vacationRequest): void

app/Infrastructure/Http/Controllers/VacationRequestController.php

@@ -116,7 +116,7 @@ class VacationRequestController extends Controller
         $stateManager->cancel($vacationRequest);
 
         return redirect()->back()
-            ->with("success", __("Vacation request has been canceled."));
+            ->with("success", __("Vacation request has been cancelled."));
     }
 
     public function acceptAsTechnical(

app/Infrastructure/Jobs/ClearVacationRequestDaysInGoogleCalendar.php

@@ -0,0 +1,39 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Toby\Infrastructure\Jobs;
+
+use Illuminate\Bus\Queueable;
+use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Spatie\GoogleCalendar\Event;
+use Toby\Eloquent\Models\Vacation;
+use Toby\Eloquent\Models\VacationRequest;
+
+class ClearVacationRequestDaysInGoogleCalendar implements ShouldQueue
+{
+    use Dispatchable;
+    use Queueable;
+
+    public function __construct(
+        protected VacationRequest $vacationRequest,
+    ) {
+    }
+
+    public function handle(): void
+    {
+        $vacations = $this->vacationRequest->vacations()
+            ->whereNotNull("event_id")
+            ->get();
+
+        /** @var Vacation $vacation */
+        foreach ($vacations as $vacation) {
+            Event::find($vacation->event_id)->delete();
+
+            $vacation->update([
+                "event_id" => null,
+            ]);
+        }
+    }
+}