🚧 model events

Author: Gautier Deleglise

src/Concerns/PerformsRestOperations.php

@@ -138,7 +138,11 @@ public function destroy(DestroyRequest $request)
         foreach ($models as $model) {
             self::newResource()->authorizeTo('delete', $model);
 
+            $resource->beforeDestroying($request, $model);
+
             $resource->performDelete($request, $model);
+
+            $resource->afterDestroying($request, $model);
         }
 
         $this->afterDestroy($request);
@@ -171,7 +175,11 @@ public function restore(RestoreRequest $request)
         foreach ($models as $model) {
             self::newResource()->authorizeTo('restore', $model);
 
+            $resource->beforeRestoring($request, $model);
+
             $resource->performRestore($request, $model);
+
+            $resource->afterRestoring($request, $model);
         }
 
         $this->afterRestore($request);
@@ -205,7 +213,11 @@ public function forceDelete(ForceDestroyRequest $request)
         foreach ($models as $model) {
             self::newResource()->authorizeTo('forceDelete', $model);
 
+            $resource->beforeForceDestroying($request, $model);
+
             $resource->performForceDelete($request, $model);
+
+            $resource->afterForceDestroying($request, $model);
         }
 
         $this->afterForceDestroy($request);

src/Concerns/Resource/HasResourceHooks.php

@@ -3,10 +3,13 @@
 namespace Lomkit\Rest\Concerns\Resource;
 
 use Illuminate\Database\Eloquent\Model;
+use Lomkit\Rest\Http\Requests\DestroyRequest;
 use Lomkit\Rest\Http\Requests\MutateRequest;
 
 trait HasResourceHooks
 {
+    // @TODO: PHP DOC
+    // @TODO: separate functioning by types ? beforeCreating, beforeUpdating, etc ? seems more logical
     public function beforeMutating(MutateRequest $request, array $requestBody, Model $model): void
     {
         //
@@ -16,4 +19,34 @@ public function afterMutating(MutateRequest $request, array $requestBody, Model
     {
         //
     }
+
+    public function beforeDestroying(DestroyRequest $request, Model $model): void
+    {
+        //
+    }
+
+    public function afterDestroying(DestroyRequest $request, Model $model): void
+    {
+        //
+    }
+
+    public function beforeRestoring(MutateRequest $request, Model $model): void
+    {
+        //
+    }
+
+    public function afterRestoring(MutateRequest $request, Model $model): void
+    {
+        //
+    }
+
+    public function beforeForceDestroying(MutateRequest $request, Model $model): void
+    {
+        //
+    }
+
+    public function afterForceDestroying(MutateRequest $request, Model $model): void
+    {
+        //
+    }
 }

src/Query/Traits/PerformMutation.php

@@ -5,6 +5,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Lomkit\Rest\Http\Requests\MutateRequest;
 use Lomkit\Rest\Http\Requests\RestRequest;
+use Lomkit\Rest\Tests\Support\Rest\Resources\BelongsToManyWithHooksResource;
 
 trait PerformMutation
 {
@@ -55,39 +56,23 @@ public function applyMutation(array $mutation = [], $attributes = [])
 
         if ($mutation['operation'] === 'create') {
             $model = $this->resource::newModel();
-
-            $this->resource->authorizeTo('create', $model);
-
-            return $this->mutateModel(
-                $model,
-                $allAttributes,
-                $mutation
-            );
-        }
-
-        if (in_array($mutation['operation'], ['update', 'touch', 'sync'])) {
+        } else {
             $model = $this->resource::newModel()::findOrFail($mutation['key']);
+        }
 
-            $this
-                ->resource
-                ->beforeMutating(app(MutateRequest::class), $mutation, $model);
-
+        if ($mutation['operation'] === 'create') {
+            $this->resource->authorizeTo('create', $model);
+        } elseif ($mutation['operation'] === 'update') {
             $this->resource->authorizeTo('update', $model);
-
-            return $this->mutateModel(
-                $model,
-                $allAttributes,
-                $mutation
-            );
+        } else {
+            $this->resource->authorizeTo('view', $model);
         }
 
-        $newModel = $this->resource::newModel()::findOrFail($mutation['key']);
-
-        $newModel
-            ->forceFill($allAttributes)
-            ->save();
-
-        return $newModel;
+        return $this->mutateModel(
+            $model,
+            $allAttributes,
+            $mutation
+        );
     }
 
     /**