conditional action log

src/Actions/Action.php

@@ -4,7 +4,7 @@
 
 use Binaryk\LaravelRestify\Http\Requests\ActionRequest;
 use Binaryk\LaravelRestify\Http\Requests\RestifyRequest;
-use Binaryk\LaravelRestify\Models\ActionLog;
+use Binaryk\LaravelRestify\Models\Concerns\HasActionLogs;
 use Binaryk\LaravelRestify\Restify;
 use Binaryk\LaravelRestify\Traits\AuthorizedToSee;
 use Binaryk\LaravelRestify\Traits\Make;
@@ -82,8 +82,8 @@ public function uriKey()
     /**
      * Determine if the action is executable for the given request.
      *
-     * @param Request $request
-     * @param Model $model
+     * @param  Request  $request
+     * @param  Model  $model
      * @return bool
      */
     public function authorizedToRun(Request $request, $model)
@@ -94,7 +94,7 @@ public function authorizedToRun(Request $request, $model)
     /**
      * Set the callback to be run to authorize running the action.
      *
-     * @param Closure $callback
+     * @param  Closure  $callback
      * @return $this
      */
     public function canRun(Closure $callback)
@@ -107,8 +107,8 @@ public function canRun(Closure $callback)
     /**
      * Get the payload available on the action.
      *
-     * @deprecated Use rules instead
      * @return array
+     * @deprecated Use rules instead
      */
     public function payload(): array
     {
@@ -128,7 +128,7 @@ public function rules(): array
     /**
      * Make current action being standalone. No model query will be performed.
      *
-     * @param bool $standalone
+     * @param  bool  $standalone
      * @return self
      */
     public function standalone(bool $standalone = true): self
@@ -167,7 +167,11 @@ public function handleRequest(ActionRequest $request)
                 Transaction::run(function () use ($models, $request, &$response) {
                     $response = $this->handle($request, $models);
 
-                    $models->each(fn (Model $model) => ActionLog::forRepositoryAction($this, $model, $request->user())->save());
+                    $models->each(function (Model $model) use ($request) {
+                        if (in_array(HasActionLogs::class, class_uses_recursive($model), true)) {
+                            Restify::actionLog()::forRepositoryAction($this, $model, $request->user())->save();
+                        }
+                    });
                 });
             });
         } else {
@@ -179,11 +183,13 @@ public function handleRequest(ActionRequest $request)
                     })->firstOrFail()
                 );
 
-                Restify::actionLog()::forRepositoryAction(
-                    $this,
-                    $model,
-                    $request->user()
-                )->save();
+                if (in_array(HasActionLogs::class, class_uses_recursive($model), true)) {
+                    Restify::actionLog()::forRepositoryAction(
+                        $this,
+                        $model,
+                        $request->user()
+                    )->save();
+                }
             });
         }
 

src/Filters/MatchesCollection.php

@@ -50,8 +50,8 @@ public function inQuery(RestifyRequest $request): self
     {
         return $this->filter(function (MatchFilter $filter) use ($request) {
             $possibleKeys = collect([
-                $filter->getColumn(),
-                "-{$filter->getColumn()}",
+                $filter->column(),
+                "-{$filter->column()}",
             ]);
 
             if ($filters = collect($request->input('filter', []))) {
@@ -60,7 +60,7 @@ public function inQuery(RestifyRequest $request): self
                 }
             }
 
-            return $request->has("-{$filter->getColumn()}") || $request->has($filter->getColumn());
+            return (bool) ($request->query("-{$filter->column()}") || $request->query($filter->column()));
         });
     }
 

src/Http/Requests/ActionRequest.php

@@ -45,7 +45,7 @@ public function builder(Action $action, int $size): Builder
             ->latest($this->model()->getKeyName());
     }
 
-    public function collectRepositories(Action $action, $count, Closure $callback)
+    public function collectRepositories(Action $action, $count, Closure $callback): array
     {
         $output = [];