diff --git a/src/Exception/FilterParameterValueIsNullException.php b/src/Exception/FilterParameterValueIsNullException.php new file mode 100644 index 0000000000000000000000000000000000000000..237e74aa218aaffb5ac3904e0429426c885db359 --- /dev/null +++ b/src/Exception/FilterParameterValueIsNullException.php @@ -0,0 +1,15 @@ +<?php + +declare(strict_types=1); + +namespace IQDEV\Packages\DoctrineHttpFilter\Exception; + +use Exception; + +class FilterParameterValueIsNullException extends Exception +{ + public function __construct(string $key) + { + parent::__construct('Filter parameter value of key [' . $key . '] is null.'); + } +} diff --git a/src/Filter/Where.php b/src/Filter/Where.php index cbfc9c7f3c80efcbe9e18cdd3a9f9b7efba9903d..5db21aeab74cdd339302e82b6a1c25bd6084f6f4 100644 --- a/src/Filter/Where.php +++ b/src/Filter/Where.php @@ -5,16 +5,22 @@ declare(strict_types=1); namespace IQDEV\Packages\DoctrineHttpFilter\Filter; use Doctrine\ORM\QueryBuilder; +use IQDEV\Packages\DoctrineHttpFilter\Exception\FilterParameterValueIsNullException; use IQDEV\Packages\DoctrineHttpFilter\HttpFilter; final class Where extends HttpFilter { + /** @throws FilterParameterValueIsNullException */ public function addToQuery(QueryBuilder $queryBuilder): QueryBuilder { $queryBuilder->where( $this->getColumn() . ' = :' . $this->getParameterKey(), ); + if ($this->getHttpValue() === null) { + throw new FilterParameterValueIsNullException($this->field); + } + $queryBuilder->setParameter($this->getParameterKey(), $this->getHttpValue()); return $queryBuilder; diff --git a/src/HttpFilter.php b/src/HttpFilter.php index 9f16b43d6841495b17a8593ed8124338cd74d95b..9078b25eb360beef89d3652272d74b4f7b8c2bec 100644 --- a/src/HttpFilter.php +++ b/src/HttpFilter.php @@ -15,7 +15,7 @@ abstract class HttpFilter public function __construct( protected string $tableAlias, - protected string $filed, + protected string $field, ?Request $request = null, ) { $this->request = $request ?? Request::createFromGlobals(); @@ -23,7 +23,7 @@ abstract class HttpFilter protected function getColumn(): string { - return $this->tableAlias . '.' . $this->filed; + return $this->tableAlias . '.' . $this->field; } protected function getHttpValue(): mixed @@ -37,7 +37,7 @@ abstract class HttpFilter return null; } - return $filter[$this->filed] ?? null; + return $filter[$this->field] ?? null; } public function getParameterKey(): string