Fix CS

Author: hlecorche

src/Paginator/AbstractDoctrinePaginator.php

@@ -46,6 +46,6 @@ final protected function defineDoctrineOptions(OptionsResolver $resolver): void
         ]);
         $resolver->setAllowedTypes('by_identifier', ['string', 'null']);
         $resolver->setAllowedTypes('count', ['int', 'array']);
-        $resolver->setAllowedValues('count', fn (int|array $value) => \is_array($value) || $value >= 0);
+        $resolver->setAllowedValues('count', static fn (int|array $value) => \is_array($value) || $value >= 0);
     }
 }

src/Paginator/DoctrineORMPaginator.php

@@ -90,7 +90,7 @@ protected function buildIterator(): \Traversable
         $doctrinePaginator->setUseOutputWalkers(true);
         /** @var array<array-key, array{pk: mixed}> $iterator */
         $iterator = (array) $doctrinePaginator->getIterator();
-        $ids = array_map(fn ($row): mixed => $row['pk'], $iterator);
+        $ids = array_map(static fn ($row): mixed => $row['pk'], $iterator);
 
         $resultsByIdsQueryBuilder = clone $queryBuilder;
         $resultsByIdsQueryBuilder->resetDQLPart('where');
@@ -112,7 +112,7 @@ protected function configureOptions(OptionsResolver $resolver): void
             'fetch_join_collection' => null,
         ]);
         $resolver->setAllowedTypes('query_builder', QueryBuilder::class);
-        $resolver->setNormalizer('count', function (Options $options, int|array $count) {
+        $resolver->setNormalizer('count', static function (Options $options, int|array $count) {
             if (\is_int($count)) {
                 return $count;
             }
@@ -126,7 +126,7 @@ protected function configureOptions(OptionsResolver $resolver): void
             return $count;
         });
         $resolver->setAllowedTypes('simplified_request', ['bool', 'null']);
-        $resolver->setNormalizer('simplified_request', function (Options $options, ?bool $simplifiedRequest): ?bool {
+        $resolver->setNormalizer('simplified_request', static function (Options $options, ?bool $simplifiedRequest): ?bool {
             if (null === $options['by_identifier'] && null === $simplifiedRequest) {
                 return true;
             } elseif (null !== $options['by_identifier'] && null !== $simplifiedRequest) {
@@ -136,7 +136,7 @@ protected function configureOptions(OptionsResolver $resolver): void
             return $simplifiedRequest;
         });
         $resolver->setAllowedTypes('fetch_join_collection', ['bool', 'null']);
-        $resolver->setNormalizer('fetch_join_collection', function (Options $options, ?bool $fetchJoinCollection): ?bool {
+        $resolver->setNormalizer('fetch_join_collection', static function (Options $options, ?bool $fetchJoinCollection): ?bool {
             if (null === $options['by_identifier'] && null === $fetchJoinCollection) {
                 return false;
             } elseif (null !== $options['by_identifier'] && null !== $fetchJoinCollection) {

src/Paginator/DoctrinePaginatorBuilder.php

@@ -65,7 +65,7 @@ final public static function countQueryBuilder(array $options): int
         $resolver = new OptionsResolver();
         $resolver->setRequired('query_builder');
         $resolver->setDefaults([
-            'behavior' => function (Options $options): string {
+            'behavior' => static function (Options $options): string {
                 /** @var QueryBuilderDBAL|QueryBuilderORM $queryBuilder */
                 $queryBuilder = $options['query_builder'];
 
@@ -78,15 +78,15 @@ final public static function countQueryBuilder(array $options): int
         ]);
         $resolver->setAllowedTypes('query_builder', [QueryBuilderDBAL::class, QueryBuilderORM::class]);
         $resolver->setAllowedTypes('behavior', 'string');
-        $resolver->setAllowedValues('behavior', function (string $behavior) use ($options): bool {
+        $resolver->setAllowedValues('behavior', static function (string $behavior) use ($options): bool {
             if ($options['query_builder'] instanceof QueryBuilderDBAL) {
                 return \in_array($behavior, ['count_by_alias', 'count_by_sub_request', 'count_by_select_all']);
             }
 
             return \in_array($behavior, ['count_by_alias', 'count_by_sub_request', 'orm']);
         });
         $resolver->setAllowedTypes('alias', ['string', 'null']);
-        $resolver->setNormalizer('alias', function (Options $options, ?string $alias): ?string {
+        $resolver->setNormalizer('alias', static function (Options $options, ?string $alias): ?string {
             if ('count_by_alias' === $options['behavior'] && null === $alias) {
                 throw new MissingOptionsException('When "behavior" option is set to "count_by_alias", "alias" option is required');
             } elseif ('count_by_alias' !== $options['behavior'] && null !== $alias) {
@@ -96,7 +96,7 @@ final public static function countQueryBuilder(array $options): int
             return $alias;
         });
         $resolver->setAllowedTypes('distinct_alias', ['bool', 'null']);
-        $resolver->setNormalizer('distinct_alias', function (Options $options, ?bool $distinctAlias): ?bool {
+        $resolver->setNormalizer('distinct_alias', static function (Options $options, ?bool $distinctAlias): ?bool {
             if ('count_by_alias' === $options['behavior'] && null === $distinctAlias) {
                 return true;
             } elseif ('count_by_alias' !== $options['behavior'] && null !== $distinctAlias) {
@@ -106,7 +106,7 @@ final public static function countQueryBuilder(array $options): int
             return $distinctAlias;
         });
         $resolver->setAllowedTypes('simplified_request', ['bool', 'null']);
-        $resolver->setNormalizer('simplified_request', function (Options $options, ?bool $simplifiedRequest): ?bool {
+        $resolver->setNormalizer('simplified_request', static function (Options $options, ?bool $simplifiedRequest): ?bool {
             if (null !== $simplifiedRequest && !($options['query_builder'] instanceof QueryBuilderORM)) {
                 throw new InvalidOptionsException('The "simplified_request" option can only be used with ORM QueryBuilder');
             }
@@ -119,7 +119,7 @@ final public static function countQueryBuilder(array $options): int
             return $simplifiedRequest;
         });
         $resolver->setAllowedTypes('connection', [Connection::class, 'null']);
-        $resolver->setNormalizer('connection', function (Options $options, ?Connection $connection): ?Connection {
+        $resolver->setNormalizer('connection', static function (Options $options, ?Connection $connection): ?Connection {
             if ('count_by_sub_request' === $options['behavior'] && $options['query_builder'] instanceof QueryBuilderDBAL && null === $connection) {
                 throw new MissingOptionsException('When "behavior" option is set to "count_by_sub_request" with DBAL QueryBuilder, "connection" option is required');
             } elseif (null !== $connection && !($options['query_builder'] instanceof QueryBuilderDBAL)) {

tests/Paginator/AbstractDoctrinePaginatorTestCase.php

@@ -136,7 +136,7 @@ public function testCount(mixed $page, int $maxPerPage, ?\Closure $queryBuilderU
 
     public static function getTestCountProvider(): array
     {
-        $queryBuilderUpdaterNoData = function (QueryBuilderDBAL|QueryBuilderORM $queryBuilder): void {
+        $queryBuilderUpdaterNoData = static function (QueryBuilderDBAL|QueryBuilderORM $queryBuilder): void {
             $queryBuilder->andWhere('0 = 1');
         };