Skip to content

Commit

Permalink
Use matches from the query only. (#440)
Browse files Browse the repository at this point in the history
* fix: Fixing query matches. (#439)

* fix: Use only query params for matching.
  • Loading branch information
binaryk authored Nov 25, 2021
1 parent 58acad2 commit 96021a0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions src/Filters/MatchesCollection.php
Original file line number Diff line number Diff line change
Expand Up @@ -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', []))) {
Expand All @@ -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()));
});
}

Expand Down
2 changes: 1 addition & 1 deletion src/Http/Requests/ActionRequest.php
Original file line number Diff line number Diff line change
Expand Up @@ -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 = [];

Expand Down

0 comments on commit 96021a0

Please sign in to comment.