spiral / framework

rector.php

Spacing +40 added lines, -40 removed lines

@@ -27,62 +27,62 @@ 
 
 return RectorConfig::configure()
     ->withPaths([
-        __DIR__ . '/src/*/src',
-        __DIR__ . '/src/*/*/src',
-        __DIR__ . '/src/*/*/tests',
-        __DIR__ . '/src/*/tests',
-        __DIR__ . '/tests',
+        __DIR__.'/src/*/src',
+        __DIR__.'/src/*/*/src',
+        __DIR__.'/src/*/*/tests',
+        __DIR__.'/src/*/tests',
+        __DIR__.'/tests',
     ])
     ->withParallel()
     ->withSkip([
         IfIssetToCoalescingRector::class,
         RemoveUnusedPrivatePropertyRector::class => [
-            __DIR__ . '/src/Scaffolder/src/Command/BootloaderCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/CommandCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/ConfigCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/ControllerCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/FilterCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/JobHandlerCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/MiddlewareCommand.php',
-            __DIR__ . '/src/Console/tests/PromptArgumentsTest.php',
+            __DIR__.'/src/Scaffolder/src/Command/BootloaderCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/CommandCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/ConfigCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/ControllerCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/FilterCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/JobHandlerCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/MiddlewareCommand.php',
+            __DIR__.'/src/Console/tests/PromptArgumentsTest.php',
         ],
         RemoveUnusedPrivateMethodRector::class => [
-            __DIR__ . '/src/Boot/src/Bootloader/ConfigurationBootloader.php',
-            __DIR__ . '/src/Broadcasting/src/Bootloader/BroadcastingBootloader.php',
-            __DIR__ . '/src/Cache/src/Bootloader/CacheBootloader.php',
-            __DIR__ . '/src/Serializer/src/Bootloader/SerializerBootloader.php',
-            __DIR__ . '/src/Validation/src/Bootloader/ValidationBootloader.php',
-            __DIR__ . '/src/Translator/tests/IndexerTest.php',
-            __DIR__ . '/src/Tokenizer/tests/ReflectionFileTest.php',
-            __DIR__ . '/src/Core/tests/SingletonsTest.php',
+            __DIR__.'/src/Boot/src/Bootloader/ConfigurationBootloader.php',
+            __DIR__.'/src/Broadcasting/src/Bootloader/BroadcastingBootloader.php',
+            __DIR__.'/src/Cache/src/Bootloader/CacheBootloader.php',
+            __DIR__.'/src/Serializer/src/Bootloader/SerializerBootloader.php',
+            __DIR__.'/src/Validation/src/Bootloader/ValidationBootloader.php',
+            __DIR__.'/src/Translator/tests/IndexerTest.php',
+            __DIR__.'/src/Tokenizer/tests/ReflectionFileTest.php',
+            __DIR__.'/src/Core/tests/SingletonsTest.php',
         ],
         RemoveUselessVarTagRector::class => [
-            __DIR__ . '/src/Console/src/Traits/HelpersTrait.php',
+            __DIR__.'/src/Console/src/Traits/HelpersTrait.php',
         ],
         RemoveAlwaysTrueIfConditionRector::class => [
-            __DIR__ . '/src/Boot/src/BootloadManager/Initializer.php',
-            __DIR__ . '/src/Stempler/src/Traverser.php',
-            __DIR__ . '/src/Prototype/src/NodeVisitors/LocateProperties.php',
-            __DIR__ . '/src/Prototype/src/NodeVisitors/RemoveTrait.php',
-            __DIR__ . '/src/Logger/src/ListenerRegistry.php',
-            __DIR__ . '/src/Stempler/src/Transform/Merge/ExtendsParent.php',
-            __DIR__ . '/src/Bridge/Stempler/src/StemplerEngine.php',
+            __DIR__.'/src/Boot/src/BootloadManager/Initializer.php',
+            __DIR__.'/src/Stempler/src/Traverser.php',
+            __DIR__.'/src/Prototype/src/NodeVisitors/LocateProperties.php',
+            __DIR__.'/src/Prototype/src/NodeVisitors/RemoveTrait.php',
+            __DIR__.'/src/Logger/src/ListenerRegistry.php',
+            __DIR__.'/src/Stempler/src/Transform/Merge/ExtendsParent.php',
+            __DIR__.'/src/Bridge/Stempler/src/StemplerEngine.php',
         ],
         RemoveExtraParametersRector::class => [
-            __DIR__ . '/src/Boot/src/BootloadManager/AbstractBootloadManager.php',
+            __DIR__.'/src/Boot/src/BootloadManager/AbstractBootloadManager.php',
         ],
         RemoveUnusedPrivateMethodParameterRector::class => [
-            __DIR__ . '/src/Core/src/Internal/Factory.php',
-            __DIR__ . '/src/Core/tests/InjectableTest.php',
+            __DIR__.'/src/Core/src/Internal/Factory.php',
+            __DIR__.'/src/Core/tests/InjectableTest.php',
         ],
         RemoveDoubleAssignRector::class => [
-            __DIR__ . '/src/Core/tests/Scope/FinalizeAttributeTest.php',
+            __DIR__.'/src/Core/tests/Scope/FinalizeAttributeTest.php',
         ],
         RemoveUnusedVariableAssignRector::class => [
-            __DIR__ . '/src/Core/tests/ExceptionsTest.php',
+            __DIR__.'/src/Core/tests/ExceptionsTest.php',
         ],
         RemoveDeadStmtRector::class => [
-            __DIR__ . '/src/Core/tests/ExceptionsTest.php',
+            __DIR__.'/src/Core/tests/ExceptionsTest.php',
         ],
 
         // to be enabled later for bc break 4.x
@@ -92,8 +92,8 @@ 
         NewInInitializerRector::class,
 
         // start with short open tag
-        __DIR__ . '/src/Views/tests/fixtures/other/var.php',
-        __DIR__ . '/tests/app/views/native.php',
+        __DIR__.'/src/Views/tests/fixtures/other/var.php',
+        __DIR__.'/tests/app/views/native.php',
 
         // example code for test
         '*/Fixture/*',
@@ -103,18 +103,18 @@ 
         '*/Stubs/*',
         '*/tests/Classes/*',
         '*/tests/Internal/*',
-        __DIR__ . '/src/Console/tests/Configurator',
+        __DIR__.'/src/Console/tests/Configurator',
 
         // cache
         '*/runtime/cache/*',
 
         ReadOnlyPropertyRector::class => [
             // used by Configurator
-            __DIR__ . '/src/Scaffolder/src/Command',
+            __DIR__.'/src/Scaffolder/src/Command',
         ],
 
         FirstClassCallableRector::class => [
-            __DIR__ . '/src/Core/tests/Scope/UseCaseTest.php',
+            __DIR__.'/src/Core/tests/Scope/UseCaseTest.php',
         ],
 
         PreferPHPUnitThisCallRector::class,

src/Bridge/Monolog/tests/FactoryTest.php

Spacing +4 added lines, -4 removed lines

@@ -55,7 +55,7 @@ 
         $logger = $factory->getLogger();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;
@@ -84,7 +84,7 @@ 
         $factory = new LogFactory(new MonologConfig([]), new ListenerRegistry(), new Container());
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;
@@ -111,7 +111,7 @@ 
     public function testFinalizerShouldResetDefaultLogger(): void
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;
@@ -151,7 +151,7 @@ 
     public function testFinalizerShouldNotResetLoggerWhenApplicationTerminating(): void
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;

Braces +8 added lines, -4 removed lines

@@ -55,7 +55,8 @@ 
         $logger = $factory->getLogger();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;
@@ -84,7 +85,8 @@ 
         $factory = new LogFactory(new MonologConfig([]), new ListenerRegistry(), new Container());
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;
@@ -111,7 +113,8 @@ 
     public function testFinalizerShouldResetDefaultLogger(): void
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;
@@ -151,7 +154,8 @@ 
     public function testFinalizerShouldNotResetLoggerWhenApplicationTerminating(): void
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;

src/Bridge/Monolog/tests/TraitTest.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
         $this->container->bind(
             ConfiguratorInterface::class,
             new ConfigManager(
-                new class() implements LoaderInterface {
+                new class() implements LoaderInterface{
                     public function has(string $section): bool
                     {
                         return false;

Braces +2 added lines, -1 removed lines

@@ -73,7 +73,8 @@
         $this->container->bind(
             ConfiguratorInterface::class,
             new ConfigManager(
-                new class() implements LoaderInterface {
+                new class() implements LoaderInterface
+                {
                     public function has(string $section): bool
                     {
                         return false;

src/Bridge/Monolog/tests/LoggerTest.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     public function testLoggerShouldBeReset(): void
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;

Braces +2 added lines, -1 removed lines

@@ -26,7 +26,8 @@
     public function testLoggerShouldBeReset(): void
     {
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface
+            {
                 public function has(string $section): bool
                 {
                     return false;

src/Bridge/Monolog/tests/ProcessorsTest.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
         $finalizer->shouldReceive('addFinalizer')->once();
 
         $this->container->bind(ConfiguratorInterface::class, new ConfigManager(
-            new class() implements LoaderInterface {
+            new class() implements LoaderInterface{
                 public function has(string $section): bool
                 {
                     return false;
@@ -78,7 +78,7 @@ 
             ]
         ]));
 
-        $logger = $this->container->get(LogsInterface::class)->getLogger('test');;
+        $logger = $this->container->get(LogsInterface::class)->getLogger('test'); ;
 
         self::assertCount(1, $logger->getProcessors());
         self::assertInstanceOf(PsrLogMessageProcessor::class, $logger->getProcessors()[0]);

src/Bridge/Dotenv/src/Bootloader/DotenvBootloader.php

Spacing +6 added lines, -6 removed lines

@@ -22,22 +22,22 @@
 
     public function loadEnvVariables(DirectoriesInterface $dirs, EnvironmentInterface $env): void
     {
-        if ($this->init) {
+        if ($this->init){
             return;
         }
 
         $this->init = true;
 
-        $dotenvPath = $env->get('DOTENV_PATH', $dirs->get('root') . '.env');
+        $dotenvPath = $env->get('DOTENV_PATH', $dirs->get('root').'.env');
 
-        if (!\file_exists($dotenvPath)) {
+        if (!\file_exists($dotenvPath)){
             return;
         }
 
-        $path = \dirname((string) $dotenvPath);
-        $file = \basename((string) $dotenvPath);
+        $path = \dirname((string)$dotenvPath);
+        $file = \basename((string)$dotenvPath);
 
-        foreach (Dotenv::createImmutable($path, $file)->load() as $key => $value) {
+        foreach (Dotenv::createImmutable($path, $file)->load() as $key => $value){
             $env->set($key, $value);
         }
     }

Braces +6 added lines, -3 removed lines

@@ -22,7 +22,8 @@ 
 
     public function loadEnvVariables(DirectoriesInterface $dirs, EnvironmentInterface $env): void
     {
-        if ($this->init) {
+        if ($this->init)
+        {
             return;
         }
 
@@ -30,14 +31,16 @@ 
 
         $dotenvPath = $env->get('DOTENV_PATH', $dirs->get('root') . '.env');
 
-        if (!\file_exists($dotenvPath)) {
+        if (!\file_exists($dotenvPath))
+        {
             return;
         }
 
         $path = \dirname((string) $dotenvPath);
         $file = \basename((string) $dotenvPath);
 
-        foreach (Dotenv::createImmutable($path, $file)->load() as $key => $value) {
+        foreach (Dotenv::createImmutable($path, $file)->load() as $key => $value)
+        {
             $env->set($key, $value);
         }
     }

src/Bridge/Stempler/tests/EngineTest.php

Spacing +6 added lines, -6 removed lines

@@ -43,10 +43,10 @@ 
     {
         $s = $this->getStempler();
 
-        try {
+        try{
             $s->get('echo', new ViewContext())->render();
             $this->fail('Exception expected');
-        } catch (RenderException $e) {
+        }catch (RenderException $e){
             $t = $e->getUserTrace()[0];
 
             self::assertSame(2, $t['line']);
@@ -58,10 +58,10 @@ 
     {
         $s = $this->getStempler();
 
-        try {
+        try{
             $s->get('other:echo-in', new ViewContext())->render();
             $this->fail('Exception expected');
-        } catch (RenderException $e) {
+        }catch (RenderException $e){
             $t = $e->getUserTrace();
             self::assertCount(2, $t);
 
@@ -77,9 +77,9 @@ 
     {
         $twig = $this->getStempler();
 
-        try {
+        try{
             $twig->get('other:bad', new ViewContext());
-        } catch (CompileException $e) {
+        }catch (CompileException $e){
             self::assertStringContainsString('bad.dark.php', $e->getFile());
         }
     }

Braces +15 added lines, -6 removed lines

@@ -43,10 +43,13 @@ 
     {
         $s = $this->getStempler();
 
-        try {
+        try
+        {
             $s->get('echo', new ViewContext())->render();
             $this->fail('Exception expected');
-        } catch (RenderException $e) {
+        }
+        catch (RenderException $e)
+        {
             $t = $e->getUserTrace()[0];
 
             self::assertSame(2, $t['line']);
@@ -58,10 +61,13 @@ 
     {
         $s = $this->getStempler();
 
-        try {
+        try
+        {
             $s->get('other:echo-in', new ViewContext())->render();
             $this->fail('Exception expected');
-        } catch (RenderException $e) {
+        }
+        catch (RenderException $e)
+        {
             $t = $e->getUserTrace();
             self::assertCount(2, $t);
 
@@ -77,9 +83,12 @@ 
     {
         $twig = $this->getStempler();
 
-        try {
+        try
+        {
             $twig->get('other:bad', new ViewContext());
-        } catch (CompileException $e) {
+        }
+        catch (CompileException $e)
+        {
             self::assertStringContainsString('bad.dark.php', $e->getFile());
         }
     }

src/Bridge/Stempler/tests/CacheTest.php

Spacing +3 added lines, -3 removed lines

@@ -30,13 +30,13 @@
     #[TestScope("http")]
     public function testCache(): void
     {
-        self::assertCount(0, $this->files->getFiles(__DIR__ . '/cache/', '*.php'));
+        self::assertCount(0, $this->files->getFiles(__DIR__.'/cache/', '*.php'));
 
         $s = $this->getStempler();
         self::assertSame('test', $s->get('test', new ViewContext())->render([]));
-        self::assertCount(2, $this->files->getFiles(__DIR__ . '/cache/', '*.php'));
+        self::assertCount(2, $this->files->getFiles(__DIR__.'/cache/', '*.php'));
 
         $s->reset('test', new ViewContext());
-        self::assertCount(0, $this->files->getFiles(__DIR__ . '/../cache/', '*.php'));
+        self::assertCount(0, $this->files->getFiles(__DIR__.'/../cache/', '*.php'));
     }
 }

src/Bridge/Stempler/src/Directive/ContainerDirective.php

Spacing +3 added lines, -3 removed lines

@@ -17,14 +17,14 @@ 
      */
     public function renderInject(Directive $directive): string
     {
-        if (\count($directive->values) < 2 || (string) $directive->values[0] === '') {
+        if (\count($directive->values) < 2 || (string)$directive->values[0] === ''){
             throw new DirectiveException(
                 'Unable to call @inject directive, 2 values required',
                 $directive->getContext()
             );
         }
 
-        if ($directive->values[0][0] === '$') {
+        if ($directive->values[0][0] === '$'){
             return \sprintf(
                 '<?php %s = $this->container->get(%s); ?>',
                 $directive->values[0],
@@ -34,7 +34,7 @@ 
 
         return \sprintf(
             '<?php $%s = $this->container->get(%s); ?>',
-            \trim((string) $directive->values[0], '\'"'),
+            \trim((string)$directive->values[0], '\'"'),
             $directive->values[1]
         );
     }