From c2dac4fb9c2477b2c9828b4d4fe9bdfddea80b8f Mon Sep 17 00:00:00 2001 From: StyleCI Bot Date: Wed, 21 Jan 2026 20:00:06 +0000 Subject: [PATCH] Apply fixes from StyleCI --- rector.php | 6 ++--- src/Reader/Cache/CachedCount.php | 7 +++++- src/Reader/EntityReader.php | 16 +++++++++----- src/Reader/FilterHandler/AllHandler.php | 2 +- src/Reader/FilterHandler/AndXHandler.php | 2 +- src/Reader/FilterHandler/BetweenHandler.php | 4 ++-- src/Reader/FilterHandler/EqualsHandler.php | 2 +- .../FilterHandler/EqualsNullHandler.php | 2 +- .../FilterHandler/GreaterThanHandler.php | 2 +- .../GreaterThanOrEqualHandler.php | 2 +- src/Reader/FilterHandler/InHandler.php | 2 +- src/Reader/FilterHandler/LessThanHandler.php | 2 +- .../FilterHandler/LessThanOrEqualHandler.php | 2 +- .../LikeHandler/BaseLikeHandler.php | 13 ++++++----- .../LikeHandler/LikeHandlerFactory.php | 8 +++---- .../LikeHandler/MysqlLikeHandler.php | 3 +-- .../LikeHandler/PostgresLikeHandler.php | 3 +-- .../LikeHandler/SqlServerLikeHandler.php | 3 +-- .../LikeHandler/SqliteLikeHandler.php | 3 +-- src/Reader/FilterHandler/NoneHandler.php | 2 +- src/Reader/FilterHandler/NotHandler.php | 17 +++++++------- src/Reader/FilterHandler/OrXHandler.php | 2 +- src/Writer/EntityWriter.php | 6 +++-- .../Base/Reader/BaseEntityReaderTestCase.php | 4 ++-- tests/Feature/DataTrait.php | 22 +++++++++---------- .../Feature/Mssql/Reader/EntityReaderTest.php | 2 +- .../Feature/Mysql/Reader/EntityReaderTest.php | 2 +- tests/Support/NotSupportedFilter.php | 4 +++- tests/Support/StubFilter.php | 4 +++- .../Reader/Cache/CachedCollectionTest.php | 4 ++-- tests/Unit/Reader/Cache/CachedCountTest.php | 2 +- tests/Unit/Reader/EntityReaderTest.php | 2 +- 32 files changed, 85 insertions(+), 72 deletions(-) diff --git a/rector.php b/rector.php index 54537c2..c49ca30 100644 --- a/rector.php +++ b/rector.php @@ -5,13 +5,13 @@ use Rector\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector; use Rector\Config\RectorConfig; use Rector\Php74\Rector\Closure\ClosureToArrowFunctionRector; -use Rector\Php81\Rector\Property\ReadOnlyPropertyRector; use Rector\Php81\Rector\FuncCall\NullToStrictStringFuncCallArgRector; +use Rector\Php81\Rector\Property\ReadOnlyPropertyRector; return RectorConfig::configure() ->withPaths([ - __DIR__ . '/src', - __DIR__ . '/tests', + __DIR__.'/src', + __DIR__.'/tests', ]) ->withPhpSets(php81: true) ->withRules([ diff --git a/src/Reader/Cache/CachedCount.php b/src/Reader/Cache/CachedCount.php index 6aec74b..429561e 100644 --- a/src/Reader/Cache/CachedCount.php +++ b/src/Reader/Cache/CachedCount.php @@ -13,10 +13,13 @@ final class CachedCount */ private ?int $count = null; - public function __construct(private ?Countable $collection) {} + public function __construct(private ?Countable $collection) + { + } /** * @psalm-internal Yiisoft\Data\Cycle\Reader + * * @psalm-return non-negative-int */ public function getCount(): int @@ -31,10 +34,12 @@ private function cacheCount(): int { /** * @psalm-suppress PossiblyNullReference + * * @psalm-var non-negative-int */ $this->count = $this->collection->count(); $this->collection = null; + return $this->count; } } diff --git a/src/Reader/EntityReader.php b/src/Reader/EntityReader.php index 8d3d34e..8271a03 100644 --- a/src/Reader/EntityReader.php +++ b/src/Reader/EntityReader.php @@ -10,15 +10,15 @@ use Cycle\ORM\Select\QueryBuilder; use Generator; use InvalidArgumentException; +use Override; use Yiisoft\Data\Cycle\Exception\NotSupportedFilterException; +use Yiisoft\Data\Cycle\Reader\Cache\CachedCollection; +use Yiisoft\Data\Cycle\Reader\Cache\CachedCount; use Yiisoft\Data\Cycle\Reader\FilterHandler\LikeHandler\LikeHandlerFactory; use Yiisoft\Data\Reader\DataReaderInterface; use Yiisoft\Data\Reader\Filter\All; use Yiisoft\Data\Reader\FilterInterface; use Yiisoft\Data\Reader\Sort; -use Yiisoft\Data\Cycle\Reader\Cache\CachedCollection; -use Yiisoft\Data\Cycle\Reader\Cache\CachedCount; -use Override; use function array_key_exists; use function is_int; @@ -105,6 +105,7 @@ public function withLimit(?int $limit): static $new->limit = $limit; $new->itemsCache = new CachedCollection(); } + return $new; } @@ -119,6 +120,7 @@ public function withOffset(int $offset): static $new->offset = $offset; $new->itemsCache = new CachedCollection(); } + return $new; } @@ -134,6 +136,7 @@ public function withSort(?Sort $sort): static $new->itemsCache = new CachedCollection(); $new->oneItemCache = new CachedCollection(); } + return $new; } @@ -151,6 +154,7 @@ public function withFilter(FilterInterface $filter): static /** @psalm-suppress ImpureMethodCall */ $new->resetCountCache(); } + return $new; } @@ -167,6 +171,7 @@ public function read(): iterable $query = $this->buildSelectQuery(); $this->itemsCache->setCollection($query->fetchAll()); } + return $this->itemsCache->getCollection(); } @@ -186,7 +191,7 @@ public function readOne(): array|object|null } /** - * Get Iterator without caching + * Get Iterator without caching. */ #[Override] public function getIterator(): Generator @@ -197,6 +202,7 @@ public function getIterator(): Generator public function getSql(): string { $query = $this->buildSelectQuery(); + return (string) ($query instanceof Select ? $query->buildQuery() : $query); } @@ -270,7 +276,7 @@ private function normalizeSortingCriteria(array $criteria): array if (is_int($direction)) { $direction = match ($direction) { SORT_DESC => 'DESC', - default => 'ASC', + default => 'ASC', }; } $criteria[$field] = $direction; diff --git a/src/Reader/FilterHandler/AllHandler.php b/src/Reader/FilterHandler/AllHandler.php index e7b24c4..e7e51ed 100644 --- a/src/Reader/FilterHandler/AllHandler.php +++ b/src/Reader/FilterHandler/AllHandler.php @@ -7,10 +7,10 @@ use Cycle\Database\Injection\Expression; use Cycle\Database\Query\SelectQuery; use Cycle\ORM\Select\QueryBuilder; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\All; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class AllHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/AndXHandler.php b/src/Reader/FilterHandler/AndXHandler.php index 6dfcbfe..a4337b7 100644 --- a/src/Reader/FilterHandler/AndXHandler.php +++ b/src/Reader/FilterHandler/AndXHandler.php @@ -6,11 +6,11 @@ use Cycle\Database\Query\SelectQuery; use Cycle\ORM\Select\QueryBuilder; +use Override; use Yiisoft\Data\Cycle\Exception\NotSupportedFilterException; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\AndX; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class AndXHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/BetweenHandler.php b/src/Reader/FilterHandler/BetweenHandler.php index a2361e5..a02305d 100644 --- a/src/Reader/FilterHandler/BetweenHandler.php +++ b/src/Reader/FilterHandler/BetweenHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\Between; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class BetweenHandler implements QueryBuilderFilterHandler { @@ -20,7 +20,7 @@ public function getFilterClass(): string #[Override] public function getAsWhereArguments(FilterInterface $filter, array $handlers): array { - /** @var Between $filter */ + /** @var Between $filter */ return [$filter->field, 'between', $filter->minValue, $filter->maxValue]; } diff --git a/src/Reader/FilterHandler/EqualsHandler.php b/src/Reader/FilterHandler/EqualsHandler.php index 1cc1229..65e66c9 100644 --- a/src/Reader/FilterHandler/EqualsHandler.php +++ b/src/Reader/FilterHandler/EqualsHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\Equals; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class EqualsHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/EqualsNullHandler.php b/src/Reader/FilterHandler/EqualsNullHandler.php index fd9059d..215d7ba 100644 --- a/src/Reader/FilterHandler/EqualsNullHandler.php +++ b/src/Reader/FilterHandler/EqualsNullHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\EqualsNull; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class EqualsNullHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/GreaterThanHandler.php b/src/Reader/FilterHandler/GreaterThanHandler.php index 0fc85e4..cefdf13 100644 --- a/src/Reader/FilterHandler/GreaterThanHandler.php +++ b/src/Reader/FilterHandler/GreaterThanHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\GreaterThan; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class GreaterThanHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/GreaterThanOrEqualHandler.php b/src/Reader/FilterHandler/GreaterThanOrEqualHandler.php index 22abdb6..b64855e 100644 --- a/src/Reader/FilterHandler/GreaterThanOrEqualHandler.php +++ b/src/Reader/FilterHandler/GreaterThanOrEqualHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\GreaterThanOrEqual; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class GreaterThanOrEqualHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/InHandler.php b/src/Reader/FilterHandler/InHandler.php index fd521d8..18b7b70 100644 --- a/src/Reader/FilterHandler/InHandler.php +++ b/src/Reader/FilterHandler/InHandler.php @@ -5,10 +5,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; use Cycle\Database\Injection\Parameter; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\In; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class InHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/LessThanHandler.php b/src/Reader/FilterHandler/LessThanHandler.php index 43d8d3c..851c8ff 100644 --- a/src/Reader/FilterHandler/LessThanHandler.php +++ b/src/Reader/FilterHandler/LessThanHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\LessThan; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class LessThanHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/LessThanOrEqualHandler.php b/src/Reader/FilterHandler/LessThanOrEqualHandler.php index 83ddc85..702e07c 100644 --- a/src/Reader/FilterHandler/LessThanOrEqualHandler.php +++ b/src/Reader/FilterHandler/LessThanOrEqualHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\LessThanOrEqual; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class LessThanOrEqualHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/LikeHandler/BaseLikeHandler.php b/src/Reader/FilterHandler/LikeHandler/BaseLikeHandler.php index 027d416..8109ca8 100644 --- a/src/Reader/FilterHandler/LikeHandler/BaseLikeHandler.php +++ b/src/Reader/FilterHandler/LikeHandler/BaseLikeHandler.php @@ -4,17 +4,17 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler\LikeHandler; +use Override; use Stringable; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\Like; use Yiisoft\Data\Reader\Filter\LikeMode; -use Override; abstract class BaseLikeHandler implements QueryBuilderFilterHandler { protected array $escapingReplacements = [ - '%' => '\%', - '_' => '\_', + '%' => '\%', + '_' => '\_', '\\' => '\\\\', ]; @@ -27,10 +27,11 @@ public function getFilterClass(): string protected function prepareValue(string|Stringable $value, LikeMode $mode): string { $value = strtr((string) $value, $this->escapingReplacements); + return match ($mode) { - LikeMode::Contains => '%' . $value . '%', - LikeMode::StartsWith => $value . '%', - LikeMode::EndsWith => '%' . $value, + LikeMode::Contains => '%'.$value.'%', + LikeMode::StartsWith => $value.'%', + LikeMode::EndsWith => '%'.$value, }; } } diff --git a/src/Reader/FilterHandler/LikeHandler/LikeHandlerFactory.php b/src/Reader/FilterHandler/LikeHandler/LikeHandlerFactory.php index c6bcf85..8c9b3c8 100644 --- a/src/Reader/FilterHandler/LikeHandler/LikeHandlerFactory.php +++ b/src/Reader/FilterHandler/LikeHandler/LikeHandlerFactory.php @@ -17,11 +17,11 @@ public static function getLikeHandler(string $driverType): QueryBuilderFilterHan // default - ignored due to the complexity of testing and preventing splitting of databaseDriver argument. // @codeCoverageIgnoreStart return match ($driverType) { - 'SQLite' => new SqliteLikeHandler(), - 'MySQL' => new MysqlLikeHandler(), - 'Postgres' => new PostgresLikeHandler(), + 'SQLite' => new SqliteLikeHandler(), + 'MySQL' => new MysqlLikeHandler(), + 'Postgres' => new PostgresLikeHandler(), 'SQLServer' => new SqlServerLikeHandler(), - default => throw new RuntimeException("$driverType database driver is not supported."), + default => throw new RuntimeException("$driverType database driver is not supported."), }; // @codeCoverageIgnoreEnd } diff --git a/src/Reader/FilterHandler/LikeHandler/MysqlLikeHandler.php b/src/Reader/FilterHandler/LikeHandler/MysqlLikeHandler.php index 5e5fa2a..78f66df 100644 --- a/src/Reader/FilterHandler/LikeHandler/MysqlLikeHandler.php +++ b/src/Reader/FilterHandler/LikeHandler/MysqlLikeHandler.php @@ -4,9 +4,9 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler\LikeHandler; +use Override; use Yiisoft\Data\Reader\Filter\Like; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class MysqlLikeHandler extends BaseLikeHandler { @@ -14,7 +14,6 @@ final class MysqlLikeHandler extends BaseLikeHandler public function getAsWhereArguments(FilterInterface $filter, array $handlers): array { /** @var Like $filter */ - if ($filter->caseSensitive !== true) { return [$filter->field, 'like', $this->prepareValue($filter->value, $filter->mode)]; } diff --git a/src/Reader/FilterHandler/LikeHandler/PostgresLikeHandler.php b/src/Reader/FilterHandler/LikeHandler/PostgresLikeHandler.php index a5a06e6..d885e3e 100644 --- a/src/Reader/FilterHandler/LikeHandler/PostgresLikeHandler.php +++ b/src/Reader/FilterHandler/LikeHandler/PostgresLikeHandler.php @@ -4,9 +4,9 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler\LikeHandler; +use Override; use Yiisoft\Data\Reader\Filter\Like; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class PostgresLikeHandler extends BaseLikeHandler { @@ -14,7 +14,6 @@ final class PostgresLikeHandler extends BaseLikeHandler public function getAsWhereArguments(FilterInterface $filter, array $handlers): array { /** @var Like $filter */ - if ($filter->caseSensitive !== true) { return [$filter->field, 'ilike', $this->prepareValue($filter->value, $filter->mode)]; } diff --git a/src/Reader/FilterHandler/LikeHandler/SqlServerLikeHandler.php b/src/Reader/FilterHandler/LikeHandler/SqlServerLikeHandler.php index ff875c7..49aacbf 100644 --- a/src/Reader/FilterHandler/LikeHandler/SqlServerLikeHandler.php +++ b/src/Reader/FilterHandler/LikeHandler/SqlServerLikeHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler\LikeHandler; +use Override; use Yiisoft\Data\Cycle\Exception\NotSupportedFilterOptionException; use Yiisoft\Data\Reader\Filter\Like; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class SqlServerLikeHandler extends BaseLikeHandler { @@ -20,7 +20,6 @@ public function __construct() public function getAsWhereArguments(FilterInterface $filter, array $handlers): array { /** @var Like $filter */ - if ($filter->caseSensitive === true) { throw new NotSupportedFilterOptionException(optionName: 'caseSensitive', driverType: 'SQLServer'); } diff --git a/src/Reader/FilterHandler/LikeHandler/SqliteLikeHandler.php b/src/Reader/FilterHandler/LikeHandler/SqliteLikeHandler.php index 3786d96..2a851a5 100644 --- a/src/Reader/FilterHandler/LikeHandler/SqliteLikeHandler.php +++ b/src/Reader/FilterHandler/LikeHandler/SqliteLikeHandler.php @@ -4,10 +4,10 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler\LikeHandler; +use Override; use Yiisoft\Data\Cycle\Exception\NotSupportedFilterOptionException; use Yiisoft\Data\Reader\Filter\Like; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class SqliteLikeHandler extends BaseLikeHandler { @@ -20,7 +20,6 @@ public function __construct() public function getAsWhereArguments(FilterInterface $filter, array $handlers): array { /** @var Like $filter */ - if ($filter->caseSensitive === true) { throw new NotSupportedFilterOptionException(optionName: 'caseSensitive', driverType: 'SQLite'); } diff --git a/src/Reader/FilterHandler/NoneHandler.php b/src/Reader/FilterHandler/NoneHandler.php index d6f0bfe..deaf342 100644 --- a/src/Reader/FilterHandler/NoneHandler.php +++ b/src/Reader/FilterHandler/NoneHandler.php @@ -7,10 +7,10 @@ use Cycle\Database\Injection\Expression; use Cycle\Database\Query\SelectQuery; use Cycle\ORM\Select\QueryBuilder; +use Override; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\None; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class NoneHandler implements QueryBuilderFilterHandler { diff --git a/src/Reader/FilterHandler/NotHandler.php b/src/Reader/FilterHandler/NotHandler.php index 37ce2ee..457b198 100644 --- a/src/Reader/FilterHandler/NotHandler.php +++ b/src/Reader/FilterHandler/NotHandler.php @@ -4,6 +4,7 @@ namespace Yiisoft\Data\Cycle\Reader\FilterHandler; +use Override; use Yiisoft\Data\Cycle\Exception\NotSupportedFilterException; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\AndX; @@ -19,7 +20,6 @@ use Yiisoft\Data\Reader\Filter\Not; use Yiisoft\Data\Reader\Filter\OrX; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class NotHandler implements QueryBuilderFilterHandler { @@ -33,7 +33,6 @@ public function getFilterClass(): string public function getAsWhereArguments(FilterInterface $filter, array $handlers): array { /** @var Not $filter */ - $convertedFilter = $this->convertFilter($filter->filter); $handledFilter = $convertedFilter instanceof Not ? $convertedFilter->filter : $convertedFilter; $handler = $handlers[$handledFilter::class] ?? null; @@ -49,7 +48,7 @@ public function getAsWhereArguments(FilterInterface $filter, array $handlers): a $operator = $where[1]; $where[1] = match ($operator) { 'between', 'in', 'like' => "not $operator", - '=' => '!=', + '=' => '!=', default => $operator, }; @@ -63,23 +62,23 @@ private function convertFilter(FilterInterface $filter, int $notCount = 1): Filt return match ($filter::class) { AndX::class => new OrX( ...array_map( - static fn(FilterInterface $subFilter): FilterInterface => $handler->convertFilter($subFilter), + static fn (FilterInterface $subFilter): FilterInterface => $handler->convertFilter($subFilter), $filter->filters, ), ), OrX::class => new AndX( ...array_map( - static fn(FilterInterface $subFilter): FilterInterface => $handler->convertFilter($subFilter), + static fn (FilterInterface $subFilter): FilterInterface => $handler->convertFilter($subFilter), $filter->filters, ), ), - GreaterThan::class => new LessThanOrEqual($filter->field, $filter->value), + GreaterThan::class => new LessThanOrEqual($filter->field, $filter->value), GreaterThanOrEqual::class => new LessThan($filter->field, $filter->value), - LessThan::class => new GreaterThanOrEqual($filter->field, $filter->value), - LessThanOrEqual::class => new GreaterThan($filter->field, $filter->value), + LessThan::class => new GreaterThanOrEqual($filter->field, $filter->value), + LessThanOrEqual::class => new GreaterThan($filter->field, $filter->value), Between::class, Equals::class, EqualsNull::class, In::class, Like::class => new Not($filter), Not::class => $this->convertNot($filter, $notCount), - default => $filter, + default => $filter, }; } diff --git a/src/Reader/FilterHandler/OrXHandler.php b/src/Reader/FilterHandler/OrXHandler.php index e79da00..44d5792 100644 --- a/src/Reader/FilterHandler/OrXHandler.php +++ b/src/Reader/FilterHandler/OrXHandler.php @@ -6,11 +6,11 @@ use Cycle\Database\Query\SelectQuery; use Cycle\ORM\Select\QueryBuilder; +use Override; use Yiisoft\Data\Cycle\Exception\NotSupportedFilterException; use Yiisoft\Data\Cycle\Reader\QueryBuilderFilterHandler; use Yiisoft\Data\Reader\Filter\OrX; use Yiisoft\Data\Reader\FilterInterface; -use Override; final class OrXHandler implements QueryBuilderFilterHandler { diff --git a/src/Writer/EntityWriter.php b/src/Writer/EntityWriter.php index d6bc72e..deb562c 100644 --- a/src/Writer/EntityWriter.php +++ b/src/Writer/EntityWriter.php @@ -5,13 +5,15 @@ namespace Yiisoft\Data\Cycle\Writer; use Cycle\ORM\EntityManagerInterface; +use Override; use Throwable; use Yiisoft\Data\Writer\DataWriterInterface; -use Override; final class EntityWriter implements DataWriterInterface { - public function __construct(private EntityManagerInterface $entityManager) {} + public function __construct(private EntityManagerInterface $entityManager) + { + } /** * @throws Throwable diff --git a/tests/Feature/Base/Reader/BaseEntityReaderTestCase.php b/tests/Feature/Base/Reader/BaseEntityReaderTestCase.php index a03c679..4b42b1f 100644 --- a/tests/Feature/Base/Reader/BaseEntityReaderTestCase.php +++ b/tests/Feature/Base/Reader/BaseEntityReaderTestCase.php @@ -20,9 +20,9 @@ use Yiisoft\Data\Reader\Sort; use Yiisoft\Data\Tests\Common\FixtureTrait; +use function count; use function iterator_to_array; use function preg_replace; -use function count; use function sprintf; abstract class BaseEntityReaderTestCase extends TestCase @@ -189,7 +189,7 @@ public static function dataGetSql(): array { return [ 'base' => [ - << ['connection' => static::$DRIVER ?? 'sqlite'], - 'sqlite' => ['connection' => 'sqlite'], + 'sqlite' => ['connection' => 'sqlite'], ]; $connections = [ 'sqlite' => new SQLiteDriverConfig( @@ -220,27 +220,27 @@ private function createOrm(): ORMInterface } /** - * Cycle ORM Schema + * Cycle ORM Schema. */ private function createSchema(): SchemaInterface { return new Schema([ 'user' => [ - SchemaInterface::MAPPER => StdMapper::class, - SchemaInterface::DATABASE => 'default', - SchemaInterface::TABLE => 'user', + SchemaInterface::MAPPER => StdMapper::class, + SchemaInterface::DATABASE => 'default', + SchemaInterface::TABLE => 'user', SchemaInterface::PRIMARY_KEY => 'id', - SchemaInterface::COLUMNS => [ + SchemaInterface::COLUMNS => [ // property => column - 'id' => 'id', - 'number' => 'number', - 'email' => 'email', + 'id' => 'id', + 'number' => 'number', + 'email' => 'email', 'balance' => 'balance', 'born_at' => 'born_at', ], SchemaInterface::TYPECAST => [ - 'id' => 'int', - 'number' => 'int', + 'id' => 'int', + 'number' => 'int', 'balance' => 'float', 'born_at' => 'datetime', ], diff --git a/tests/Feature/Mssql/Reader/EntityReaderTest.php b/tests/Feature/Mssql/Reader/EntityReaderTest.php index b525cee..ac047f6 100644 --- a/tests/Feature/Mssql/Reader/EntityReaderTest.php +++ b/tests/Feature/Mssql/Reader/EntityReaderTest.php @@ -14,7 +14,7 @@ public static function dataGetSql(): array { return [ 'base' => [ - << [ - <<