Skip to content

Commit 108281d

Browse files
committed
refactor: return early if wrong values
1 parent b32edd0 commit 108281d

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

src/Laravel/Eloquent/Filter/BooleanFilter.php

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,21 +21,21 @@ final class BooleanFilter implements FilterInterface
2121
{
2222
use QueryPropertyTrait;
2323

24+
private const BOOLEAN_VALUES = [
25+
'true' => true,
26+
'false' => false,
27+
'1' => true,
28+
'0' => false,
29+
];
30+
2431
/**
2532
* @param Builder<Model> $builder
2633
* @param array<string, mixed> $context
2734
*/
2835
public function apply(Builder $builder, mixed $values, Parameter $parameter, array $context = []): Builder
2936
{
30-
$booleanValues = [
31-
'true' => true,
32-
'false' => false,
33-
'1' => true,
34-
'0' => false,
35-
];
36-
37-
if (array_key_exists($values, $booleanValues)) {
38-
$values = $booleanValues[$values];
37+
if (!is_string($values) || !array_key_exists($values, self::BOOLEAN_VALUES)) {
38+
return $builder;
3939
}
4040

4141
return $builder->{$context['whereClause'] ?? 'where'}($this->getQueryProperty($parameter), $values);

0 commit comments

Comments
 (0)