✨ hooks ready

Author: GautierDele

src/Concerns/PerformsRestOperations.php

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

src/Concerns/Resource/HasResourceHooks.php

@@ -4,48 +4,106 @@
 
 use Illuminate\Database\Eloquent\Model;
 use Lomkit\Rest\Http\Requests\DestroyRequest;
+use Lomkit\Rest\Http\Requests\ForceDestroyRequest;
 use Lomkit\Rest\Http\Requests\MutateRequest;
+use Lomkit\Rest\Http\Requests\RestoreRequest;
 
 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
+    /**
+     * Executed when a model has been "mutating".
+     *
+     * @param MutateRequest $request
+     * @param array $requestBody
+     * @param Model $model
+     * @return void
+     */
+    public function mutating(MutateRequest $request, array $requestBody, Model $model): void
     {
         //
     }
 
-    public function afterMutating(MutateRequest $request, array $requestBody, Model $model): void
+    /**
+     * Executed when a model has been "mutated".
+     *
+     * @param MutateRequest $request
+     * @param array $requestBody
+     * @param Model $model
+     * @return void
+     */
+    public function mutated(MutateRequest $request, array $requestBody, Model $model): void
     {
         //
     }
 
-    public function beforeDestroying(DestroyRequest $request, Model $model): void
+    /**
+     * Executed when a model has been "destroying".
+     *
+     * @param DestroyRequest $request
+     * @param Model $model
+     * @return void
+     */
+    public function destroying(DestroyRequest $request, Model $model): void
     {
         //
     }
 
-    public function afterDestroying(DestroyRequest $request, Model $model): void
+    /**
+     * Executed when a model has been "destroyed".
+     *
+     * @param DestroyRequest $request
+     * @param Model $model
+     * @return void
+     */
+    public function destroyed(DestroyRequest $request, Model $model): void
     {
         //
     }
 
-    public function beforeRestoring(MutateRequest $request, Model $model): void
+    /**
+     * Executed when a model has been "restoring".
+     *
+     * @param RestoreRequest $request
+     * @param Model $model
+     * @return void
+     */
+    public function restoring(RestoreRequest $request, Model $model): void
     {
         //
     }
 
-    public function afterRestoring(MutateRequest $request, Model $model): void
+    /**
+     * Executed when a model has been "restored".
+     *
+     * @param RestoreRequest $request
+     * @param Model $model
+     * @return void
+     */
+    public function restored(RestoreRequest $request, Model $model): void
     {
         //
     }
 
-    public function beforeForceDestroying(MutateRequest $request, Model $model): void
+    /**
+     * Executed when a model has been "forceDestroying".
+     *
+     * @param ForceDestroyRequest $request
+     * @param Model $model
+     * @return void
+     */
+    public function forceDestroying(ForceDestroyRequest $request, Model $model): void
     {
         //
     }
 
-    public function afterForceDestroying(MutateRequest $request, Model $model): void
+    /**
+     * Executed when a model has been "forceDestroyed".
+     *
+     * @param ForceDestroyRequest $request
+     * @param Model $model
+     * @return void
+     */
+    public function forceDestroyed(ForceDestroyRequest $request, Model $model): void
     {
         //
     }

src/Http/Controllers/Traits/HasControllerHooks.php

@@ -2,6 +2,7 @@
 
 namespace Lomkit\Rest\Http\Controllers\Traits;
 
+use Illuminate\Database\Eloquent\Model;
 use Lomkit\Rest\Http\Requests\DestroyRequest;
 use Lomkit\Rest\Http\Requests\DetailsRequest;
 use Lomkit\Rest\Http\Requests\ForceDestroyRequest;
@@ -12,66 +13,144 @@
 
 trait HasControllerHooks
 {
+    /**
+     * Executed before details.
+     *
+     * @param DetailsRequest $request
+     * @return void
+     */
     protected function beforeDetails(DetailsRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed before search.
+     *
+     * @param SearchRequest $request
+     * @return void
+     */
     protected function beforeSearch(SearchRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed after search.
+     *
+     * @param SearchRequest $request
+     * @return void
+     */
     protected function afterSearch(SearchRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed before mutate.
+     *
+     * @param MutateRequest $request
+     * @return void
+     */
     protected function beforeMutate(MutateRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed after mutate.
+     *
+     * @param MutateRequest $request
+     * @return void
+     */
     protected function afterMutate(MutateRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed before operate.
+     *
+     * @param OperateRequest $request
+     * @return void
+     */
     protected function beforeOperate(OperateRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed after operate.
+     *
+     * @param OperateRequest $request
+     * @return void
+     */
     protected function afterOperate(OperateRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed before destroy.
+     *
+     * @param DestroyRequest $request
+     * @return void
+     */
     protected function beforeDestroy(DestroyRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed after destroy.
+     *
+     * @param DestroyRequest $request
+     * @return void
+     */
     protected function afterDestroy(DestroyRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed before restore.
+     *
+     * @param RestoreRequest $request
+     * @return void
+     */
     protected function beforeRestore(RestoreRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed after restore.
+     *
+     * @param RestoreRequest $request
+     * @return void
+     */
     protected function afterRestore(RestoreRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed before force destroy.
+     *
+     * @param ForceDestroyRequest $request
+     * @return void
+     */
     protected function beforeForceDestroy(ForceDestroyRequest $request): void
     {
         //
     }
 
+    /**
+     * Executed after force destroy.
+     *
+     * @param ForceDestroyRequest $request
+     * @return void
+     */
     protected function afterForceDestroy(ForceDestroyRequest $request): void
     {
         //

src/Query/Traits/PerformMutation.php

@@ -103,7 +103,7 @@ function ($relation) use ($relations) {
 
         $this
             ->resource
-            ->beforeMutating(app(MutateRequest::class), $mutation, $model);
+            ->mutating(app(MutateRequest::class), $mutation, $model);
 
         $model
             ->forceFill($attributes)
@@ -115,7 +115,7 @@ function ($relation) use ($relations) {
 
         $this
             ->resource
-            ->afterMutating(app(MutateRequest::class), $mutation, $model);
+            ->mutated(app(MutateRequest::class), $mutation, $model);
 
         return $model;
     }