Update code style (apply risky rules)

spiral · Feb 7, 2025 · 124ccdf · 124ccdf
1 parent 2439048
commit 124ccdf
Show file tree

Hide file tree

Showing 20 changed files with 46 additions and 46 deletions.
diff --git a/.php-cs-fixer.dist.php b/.php-cs-fixer.dist.php
@@ -9,5 +9,5 @@
     ->include(__DIR__ . '/tests')
     ->include(__FILE__)
     ->cache('./runtime/php-cs-fixer.cache')
-    ->allowRisky(false)
+    ->allowRisky(true)
     ->build();
diff --git a/src/Auth/Token.php b/src/Auth/Token.php
@@ -41,12 +41,12 @@ public function __construct(
         $this->id = $id;
 
         $this->secretValue = $secretValue;
-        $this->hashedValue = hash('sha512', $secretValue);
+        $this->hashedValue = \hash('sha512', $secretValue);
 
         $this->createdAt = $createdAt;
         $this->expiresAt = $expiresAt;
 
-        $this->payload = json_encode($payload);
+        $this->payload = \json_encode($payload);
     }
 
     public function setSecretValue(string $value): void
@@ -56,7 +56,7 @@ public function setSecretValue(string $value): void
 
     public function getID(): string
     {
-        return sprintf('%s:%s', $this->id, $this->secretValue);
+        return \sprintf('%s:%s', $this->id, $this->secretValue);
     }
 
     public function getHashedValue(): string
@@ -76,15 +76,15 @@ public function getExpiresAt(): ?\DateTimeInterface
 
     public function getPayload(): array
     {
-        if (is_array($this->normalizedPayload)) {
+        if (\is_array($this->normalizedPayload)) {
             return $this->normalizedPayload;
         }
 
-        if (is_resource($this->payload)) {
+        if (\is_resource($this->payload)) {
             // postgres
-            $this->normalizedPayload = json_decode(stream_get_contents($this->payload), true);
-        } elseif (is_string($this->payload)) {
-            $this->normalizedPayload = json_decode($this->payload, true);
+            $this->normalizedPayload = \json_decode(\stream_get_contents($this->payload), true);
+        } elseif (\is_string($this->payload)) {
+            $this->normalizedPayload = \json_decode($this->payload, true);
         }
 
         if ($this->normalizedPayload === null) {

diff --git a/src/Auth/TokenStorage.php b/src/Auth/TokenStorage.php
@@ -101,6 +101,6 @@ private function issueID(): string
 
     private function randomHash(int $length): string
     {
-        return \substr(\bin2hex(random_bytes($length)), 0, $length);
+        return \substr(\bin2hex(\random_bytes($length)), 0, $length);
     }
 }
diff --git a/src/Bootloader/DisconnectsBootloader.php b/src/Bootloader/DisconnectsBootloader.php
@@ -17,7 +17,7 @@ final class DisconnectsBootloader extends Bootloader
     public function init(FinalizerInterface $finalizer, ContainerInterface $container): void
     {
         $finalizer->addFinalizer(
-            function (bool $terminate) use ($container): void {
+            static function (bool $terminate) use ($container): void {
                 if ($terminate) {
                     return;
                 }

diff --git a/src/Bootloader/ValidationBootloader.php b/src/Bootloader/ValidationBootloader.php
@@ -10,7 +10,7 @@
 
 final class ValidationBootloader extends Bootloader
 {
-    public function boot(ValidatorBootloader $validation)
+    public function boot(ValidatorBootloader $validation): void
     {
         $validation->addChecker('entity', EntityChecker::class);
     }

diff --git a/src/Console/Command/CycleOrm/RenderCommand.php b/src/Console/Command/CycleOrm/RenderCommand.php
@@ -28,7 +28,7 @@ public function perform(
             'color' => new OutputSchemaRenderer(OutputSchemaRenderer::FORMAT_CONSOLE_COLOR),
             'plain' => new OutputSchemaRenderer(OutputSchemaRenderer::FORMAT_PLAIN_TEXT),
             default => throw new \InvalidArgumentException(
-                sprintf("Format `%s` isn't supported.", $this->argument('format')),
+                \sprintf("Format `%s` isn't supported.", $this->argument('format')),
             ),
         };
 

diff --git a/src/Console/Command/Database/ListCommand.php b/src/Console/Command/Database/ListCommand.php
@@ -65,7 +65,7 @@ public function perform(DatabaseConfig $config, DatabaseProviderInterface $dbal)
             } catch (\Exception $exception) {
                 $this->renderException($grid, $header, $exception);
 
-                if ($database->getName() != end($databases)) {
+                if ($database->getName() != \end($databases)) {
                     $grid->addRow(new TableSeparator());
                 }
 
@@ -74,7 +74,7 @@ public function perform(DatabaseConfig $config, DatabaseProviderInterface $dbal)
 
             $header[] = '<info>connected</info>';
             $this->renderTables($grid, $header, $database);
-            if ($database->getName() != end($databases)) {
+            if ($database->getName() != \end($databases)) {
                 $grid->addRow(new TableSeparator());
             }
         }
@@ -102,14 +102,14 @@ private function renderTables(Table $grid, array $header, Database $database): v
     {
         foreach ($database->getTables() as $table) {
             $grid->addRow(
-                array_merge(
+                \array_merge(
                     $header,
-                    [$table->getName(), number_format($table->count())],
+                    [$table->getName(), \number_format($table->count())],
                 ),
             );
             $header = ['', '', '', '', ''];
         }
 
-        $header[1] && $grid->addRow(array_merge($header, ['no tables', 'no records']));
+        $header[1] && $grid->addRow(\array_merge($header, ['no tables', 'no records']));
     }
 }
diff --git a/src/Console/Command/Migrate/AbstractCommand.php b/src/Console/Command/Migrate/AbstractCommand.php
@@ -56,7 +56,7 @@ protected function verifyEnvironment(string $message = 'Confirmation is required
 
     protected function defineOptions(): array
     {
-        return array_merge(
+        return \array_merge(
             static::OPTIONS,
             [
                 ['force', 's', InputOption::VALUE_NONE, 'Skip safe environment check'],

diff --git a/src/DataGrid/Writer/PostgresQueryWriter.php b/src/DataGrid/Writer/PostgresQueryWriter.php
@@ -28,7 +28,7 @@ public function write(mixed $source, SpecificationInterface $specification, Comp
             return $source->where(
                 $specification->getExpression(),
                 'ILIKE',
-                sprintf($specification->getPattern(), $this->fetchValue($specification->getValue())),
+                \sprintf($specification->getPattern(), $this->fetchValue($specification->getValue())),
             );
         }
 
@@ -43,7 +43,7 @@ public function write(mixed $source, SpecificationInterface $specification, Comp
     protected function targetAcceptable($target): bool
     {
         if (
-            class_exists(SelectQuery::class)
+            \class_exists(SelectQuery::class)
             && $target instanceof SelectQuery
             && $target->getDriver() instanceof PostgresDriver
         ) {

diff --git a/src/Injector/DatabaseInjector.php b/src/Injector/DatabaseInjector.php
@@ -24,7 +24,7 @@ public function createInjection(\ReflectionClass $class, ?string $context = null
         try {
             return $this->dm->database($context);
         } catch (DBALException $e) {
-            if ($context === null || !str_contains($e->getMessage(), ' no presets for ')) {
+            if ($context === null || !\str_contains($e->getMessage(), ' no presets for ')) {
                 throw $e;
             }
             // get default database

diff --git a/tests/src/Auth/TokenStorageTest.php b/tests/src/Auth/TokenStorageTest.php
@@ -13,7 +13,7 @@ final class TokenStorageTest extends BaseTest
 {
     private TokenStorage $storage;
 
-    public function testTokenShouldBeCreatedWithoutExpiration()
+    public function testTokenShouldBeCreatedWithoutExpiration(): void
     {
         $token = $this->storage->create(['foo' => 'bar']);
 
@@ -23,7 +23,7 @@ public function testTokenShouldBeCreatedWithoutExpiration()
         $this->assertSame(['foo' => 'bar'], $token->getPayload());
     }
 
-    public function testTokenShouldBeCreatedWithExpiration()
+    public function testTokenShouldBeCreatedWithExpiration(): void
     {
         $token = $this->storage->create(['foo' => 'bar'], $date = new \DateTimeImmutable('2010-05-05 12:34:56'));
 
@@ -33,7 +33,7 @@ public function testTokenShouldBeCreatedWithExpiration()
         $this->assertSame(['foo' => 'bar'], $token->getPayload());
     }
 
-    public function testTokenShouldBeLoadedById()
+    public function testTokenShouldBeLoadedById(): void
     {
         $token = $this->storage->create(['foo' => 'bar']);
 
@@ -47,7 +47,7 @@ public function testTokenShouldBeLoadedById()
         $this->assertSame($loadedToken->getExpiresAt(), $token->getExpiresAt());
     }
 
-    public function testTokenShouldBeDeleted()
+    public function testTokenShouldBeDeleted(): void
     {
         $token = $this->storage->create(['foo' => 'bar']);
         $this->storage->delete($token);

diff --git a/tests/src/Auth/TokenTest.php b/tests/src/Auth/TokenTest.php
@@ -42,7 +42,7 @@ public function testGetPayloadWithBadPayload(): void
      */
     public function testGetPayloadWithBadResourcePayload(): void
     {
-        $resourcePayload = fopen('php://memory', 'r');
+        $resourcePayload = \fopen('php://memory', 'r');
 
         $token = $this->setProtectedProperty($this->buildToken(), 'payload', $resourcePayload);
 
@@ -60,7 +60,7 @@ public function testGetPayloadWithBadResourcePayload(): void
      */
     protected function setProtectedProperty(object $object, string $property, $value): object
     {
-        $refProjectClass = new \ReflectionClass(get_class($object));
+        $refProjectClass = new \ReflectionClass(\get_class($object));
         $classProperty = $refProjectClass->getProperty($property);
         $classProperty->setAccessible(true);
         $classProperty->setValue($object, $value);

diff --git a/tests/src/BaseTest.php b/tests/src/BaseTest.php
@@ -45,7 +45,7 @@ public function getTestAttributes(string $attribute, ?string $method = null): ar
 
     public function rootDirectory(): string
     {
-        return dirname(__DIR__ . '/../app');
+        return \dirname(__DIR__ . '/../app');
     }
 
     public function defineBootloaders(): array
@@ -111,8 +111,8 @@ public function getRepository(string $role): RepositoryInterface
 
     public function updateConfig(string $key, mixed $data): void
     {
-        [$config, $key] = explode('.', $key, 2);
-        $this->beforeBooting(static function (ConfigsInterface $configs) use ($config, $key, $data) {
+        [$config, $key] = \explode('.', $key, 2);
+        $this->beforeBooting(static function (ConfigsInterface $configs) use ($config, $key, $data): void {
             $configs->modify(
                 $config,
                 new Set($key, $data),

diff --git a/tests/src/Bootloader/CycleOrmWarmedUpBootloaderTest.php b/tests/src/Bootloader/CycleOrmWarmedUpBootloaderTest.php
@@ -43,7 +43,7 @@ protected function setUp(): void
                 return $heap;
             });
 
-        $this->beforeBooting(static function (Container $container) use ($orm) {
+        $this->beforeBooting(static function (Container $container) use ($orm): void {
             $container->bindSingleton(ORMInterface::class, $orm);
             $container->bindSingleton(ORM::class, $orm);
         });

diff --git a/tests/src/Console/Command/CycleOrm/MigrateCommandTest.php b/tests/src/Console/Command/CycleOrm/MigrateCommandTest.php
@@ -67,7 +67,7 @@ public function testMigrationShouldBeCreatedWhenNewEntityAppeared(): void
         $fs = new Files();
 
         $entityPatch = __DIR__ . '/../../../../app/Entities/Tag.php';
-        file_put_contents(
+        \file_put_contents(
             $entityPatch,
             <<<'PHP'
                 <?php

diff --git a/tests/src/Console/Command/CycleOrm/RenderCommandTest.php b/tests/src/Console/Command/CycleOrm/RenderCommandTest.php
@@ -41,7 +41,7 @@ public function testRenderInPHPFormat(): void
             'custom_typecast_handler',
         ],
     ])]
-    public function testRenderInColorFormat()
+    public function testRenderInColorFormat(): void
     {
         $this->assertConsoleCommandOutputContainsStrings('cycle:render', ['format' => 'color'], [
             '[35m[user][39m :: [32mdefault[39m.[32musers[39m',
@@ -67,7 +67,7 @@ public function testRenderInInvalidFormat(): void
             'custom_typecast_handler',
         ],
     ])]
-    public function testRedefineSchemaDefaults()
+    public function testRedefineSchemaDefaults(): void
     {
         $this->assertConsoleCommandOutputContainsStrings('cycle:render', ['format' => 'plain'], [
             'Mapper: custom_mapper',

diff --git a/tests/src/DataGrid/GeneratorTest.php b/tests/src/DataGrid/GeneratorTest.php
@@ -38,7 +38,7 @@ public function testBasePaginate(): void
                 'status' => 'active',
                 'name' => 'Antony',
             ],
-        ], iterator_to_array($view));
+        ], \iterator_to_array($view));
 
         $this->assertNull($view->getOption(Grid::COUNT));
 
@@ -72,7 +72,7 @@ public function testPaginateWithCount(): void
                 'status' => 'active',
                 'name' => 'John',
             ],
-        ], iterator_to_array($view));
+        ], \iterator_to_array($view));
 
         $this->assertSame(3, $view->getOption(Grid::COUNT));
 
@@ -107,7 +107,7 @@ public function testDefaultWithMapping(): void
 
         $this->assertEquals([
             'John',
-        ], iterator_to_array($view));
+        ], \iterator_to_array($view));
 
         $this->assertSame(3, $view->getOption(Grid::COUNT));
 
@@ -139,7 +139,7 @@ public function testSort(): void
             'Bob',
             'John',
             'Antony',
-        ], iterator_to_array($view));
+        ], \iterator_to_array($view));
 
         $this->assertSame([
             'id' => 'desc',
@@ -168,7 +168,7 @@ public function testSortAsc(): void
             'Antony',
             'John',
             'Bob',
-        ], iterator_to_array($view));
+        ], \iterator_to_array($view));
 
         $this->assertSame([
             'id' => 'asc',

diff --git a/tests/src/DataGrid/QueryWriter/WriteSorterTest.php b/tests/src/DataGrid/QueryWriter/WriteSorterTest.php
@@ -120,7 +120,7 @@ public function testBinary($direction, $resultDirection = null): void
             );
 
             $this->assertEqualSQL(
-                sprintf(
+                \sprintf(
                     'SELECT * FROM "users" ORDER BY "balance" %s, "credits" %s',
                     $resultDirection,
                     $resultDirection,
@@ -180,7 +180,7 @@ public function testSortBinary($direction, $resultDirection = null): void
             );
 
             $this->assertEqualSQL(
-                sprintf(
+                \sprintf(
                     'SELECT * FROM "users" ORDER BY "balance" %s, "credits" %s',
                     $resultDirection,
                     $resultDirection,

diff --git a/tests/src/Validation/EntityCheckerTest.php b/tests/src/Validation/EntityCheckerTest.php
@@ -165,14 +165,14 @@ public function badCasesProvider(): iterable
                 'rules' => [
                     'email' => [['entity::exists', User::class, 'email', 'ignoreCase' => true, 'multiple' => true]],
                 ],
-                'entityData' => [strtoupper(self::ENTITY_PK) => [1, 2], 'email' => 'TEST@mail.com'],
+                'entityData' => [\strtoupper(self::ENTITY_PK) => [1, 2], 'email' => 'TEST@mail.com'],
                 'exceptionText' => 'The `exists` rule doesn\'t work in multiple case insensitive mode.',
             ],
             'pk ignore case true multiple - not found' => [
                 'rules' => [
                     'email' => [['entity::exists', User::class, 'email', 'ignoreCase' => true, 'multiple' => true]],
                 ],
-                'entityData' => [strtoupper(self::ENTITY_PK) => [2, 96], 'email' => 'TEST@mail.com'],
+                'entityData' => [\strtoupper(self::ENTITY_PK) => [2, 96], 'email' => 'TEST@mail.com'],
                 'exceptionText' => 'The `exists` rule doesn\'t work in multiple case insensitive mode.',
             ],
         ];

diff --git a/tests/src/Validation/EntityCheckerTrait.php b/tests/src/Validation/EntityCheckerTrait.php
@@ -95,9 +95,9 @@ public function __construct(
 
                     public function where(string $field, string $operator, Parameter $parameter): self
                     {
-                        $this->items = array_filter(
+                        $this->items = \array_filter(
                             $this->items,
-                            fn(mixed $value) => \in_array($value[$field], (array) $parameter->getValue(), true),
+                            static fn(mixed $value) => \in_array($value[$field], (array) $parameter->getValue(), true),
                         );
 
                         return $this;