spiral / framework

src/Prototype/tests/BackwardCompatibilityTrait.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
      */
     public static function assertRegExp(string $pattern, string $string, string $message = ''): void
     {
-        if (\method_exists(Assert::class, 'assertMatchesRegularExpression')) {
+        if (\method_exists(Assert::class, 'assertMatchesRegularExpression')){
             Assert::assertMatchesRegularExpression($pattern, $string, $message);
 
             return;
@@ -35,7 +35,7 @@ 
      */
     public static function assertNotRegExp(string $pattern, string $string, string $message = ''): void
     {
-        if (\method_exists(Assert::class, 'assertDoesNotMatchRegularExpression')) {
+        if (\method_exists(Assert::class, 'assertDoesNotMatchRegularExpression')){
             Assert::assertDoesNotMatchRegularExpression($pattern, $string, $message);
 
             return;

Braces +4 added lines, -2 removed lines

@@ -19,7 +19,8 @@ 
      */
     public static function assertRegExp(string $pattern, string $string, string $message = ''): void
     {
-        if (\method_exists(Assert::class, 'assertMatchesRegularExpression')) {
+        if (\method_exists(Assert::class, 'assertMatchesRegularExpression'))
+        {
             Assert::assertMatchesRegularExpression($pattern, $string, $message);
 
             return;
@@ -35,7 +36,8 @@ 
      */
     public static function assertNotRegExp(string $pattern, string $string, string $message = ''): void
     {
-        if (\method_exists(Assert::class, 'assertDoesNotMatchRegularExpression')) {
+        if (\method_exists(Assert::class, 'assertDoesNotMatchRegularExpression'))
+        {
             Assert::assertDoesNotMatchRegularExpression($pattern, $string, $message);
 
             return;

src/Prototype/tests/InjectorTest.php

Braces +12 added lines, -5 removed lines

@@ -27,7 +27,8 @@ 
 
     public function setUp(): void
     {
-        if ((string)ini_get('zend.assertions') === 1) {
+        if ((string)ini_get('zend.assertions') === 1)
+        {
             ini_set('zend.assertions', 0);
         }
     }
@@ -75,15 +76,21 @@ 
             $noPhpDoc
         );
 
-        if ($expectedTypedProperty) {
+        if ($expectedTypedProperty)
+        {
             $this->assertStringContainsString('private TestClass $testClass;', $printed);
-        } else {
+        }
+        else
+        {
             $this->assertStringNotContainsString('private TestClass $testClass;', $printed);
         }
 
-        if ($expectedPhpDoc) {
+        if ($expectedPhpDoc)
+        {
             $this->assertRegExp('/@var TestClass[\s|\r\n]/', $printed);
-        } else {
+        }
+        else
+        {
             $this->assertNotRegExp('/@var TestClass[\s|\r\n]/', $printed);
         }
     }

Spacing +14 added lines, -14 removed lines

@@ -27,7 +27,7 @@ 
 
     public function setUp(): void
     {
-        if ((string)ini_get('zend.assertions') === 1) {
+        if ((string)ini_get('zend.assertions') === 1){
             ini_set('zend.assertions', 0);
         }
     }
@@ -40,7 +40,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $printed = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -66,7 +66,7 @@ 
     ): void {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $printed = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class]),
@@ -75,15 +75,15 @@ 
             $noPhpDoc
         );
 
-        if ($expectedTypedProperty) {
+        if ($expectedTypedProperty){
             $this->assertStringContainsString('private TestClass $testClass;', $printed);
-        } else {
+        }else{
             $this->assertStringNotContainsString('private TestClass $testClass;', $printed);
         }
 
-        if ($expectedPhpDoc) {
+        if ($expectedPhpDoc){
             $this->assertRegExp('/@var TestClass[\s|\r\n]/', $printed);
-        } else {
+        }else{
             $this->assertNotRegExp('/@var TestClass[\s|\r\n]/', $printed);
         }
     }
@@ -106,7 +106,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestEmptyClass.php';
+        $filename = __DIR__.'/Fixtures/TestEmptyClass.php';
         $content = file_get_contents($filename);
         $printed = $i->injectDependencies(
             file_get_contents($filename),
@@ -124,7 +124,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -149,7 +149,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/ChildClass.php';
+        $filename = __DIR__.'/Fixtures/ChildClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -167,7 +167,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/ChildWithConstructorClass.php';
+        $filename = __DIR__.'/Fixtures/ChildWithConstructorClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition($filename, ['testClass' => TestClass::class])
@@ -183,7 +183,7 @@ 
      */
     public function testModifyConstructor(): void
     {
-        $filename = __DIR__ . '/Fixtures/WithConstructor.php';
+        $filename = __DIR__.'/Fixtures/WithConstructor.php';
         $extractor = new Traverse\Extractor();
 
         $parameters = $extractor->extractFromFilename($filename);
@@ -209,7 +209,7 @@ 
      */
     public function testPriorOptionalConstructorParameters(): void
     {
-        $filename = __DIR__ . '/Fixtures/OptionalConstructorArgsClass.php';
+        $filename = __DIR__.'/Fixtures/OptionalConstructorArgsClass.php';
         $extractor = new Traverse\Extractor();
 
         $parameters = $extractor->extractFromFilename($filename);
@@ -240,7 +240,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/ClassNode/ConflictResolver/Fixtures/ChildClass.php';
+        $filename = __DIR__.'/ClassNode/ConflictResolver/Fixtures/ChildClass.php';
         $printed = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(

src/Prototype/tests/ClassNode/ConflictResolver/ConflictResolverTest.php

Spacing +20 added lines, -20 removed lines

@@ -23,7 +23,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClass.php';
+        $filename = __DIR__.'/Fixtures/TestClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -36,16 +36,16 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Test::class.';', $r);
         $this->assertRegExp('/@var Test[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param Test $test', $r);
 
-        $this->assertStringContainsString(Fixtures\SubFolder\Test::class . ' as Test2;', $r);
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\SubFolder\Test::class.' as Test2;', $r);
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class.';', $r);
         $this->assertRegExp('/@var Test2[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param Test2 $test2', $r);
 
-        $this->assertStringContainsString(Fixtures\ATest3::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\ATest3::class.';', $r);
         $this->assertRegExp('/@var ATest3[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param ATest3 $test3', $r);
     }
@@ -57,7 +57,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/TestClassWithImports.php';
+        $filename = __DIR__.'/Fixtures/TestClassWithImports.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -70,18 +70,18 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Test::class . ' as FTest;', $r);
-        $this->assertStringNotContainsString(Fixtures\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Test::class.' as FTest;', $r);
+        $this->assertStringNotContainsString(Fixtures\Test::class.';', $r);
         $this->assertRegExp('/@var FTest[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param FTest $test', $r);
 
-        $this->assertStringContainsString(Fixtures\SubFolder\Test::class . ' as TestAlias;', $r);
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\SubFolder\Test::class.' as TestAlias;', $r);
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class.';', $r);
         $this->assertRegExp('/@var TestAlias[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param TestAlias $test2', $r);
 
-        $this->assertStringContainsString(Fixtures\ATest3::class . ' as ATest;', $r);
-        $this->assertStringNotContainsString(Fixtures\ATest3::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\ATest3::class.' as ATest;', $r);
+        $this->assertStringNotContainsString(Fixtures\ATest3::class.';', $r);
         $this->assertRegExp('/@var ATest[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param ATest $test3', $r);
     }
@@ -93,7 +93,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/ChildClass.php';
+        $filename = __DIR__.'/Fixtures/ChildClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -106,17 +106,17 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Test::class.';', $r);
         $this->assertRegExp('/@var Test[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param Test $test', $r);
 
-        $this->assertStringContainsString(Fixtures\SubFolder\Test::class . ' as Test2;', $r);
-        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\SubFolder\Test::class.' as Test2;', $r);
+        $this->assertStringNotContainsString(Fixtures\SubFolder\Test::class.';', $r);
         $this->assertRegExp('/@var Test2[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param Test2 $test2', $r);
 
-        $this->assertStringContainsString(Fixtures\ATest3::class . ' as ATestAlias;', $r);
-        $this->assertStringNotContainsString(Fixtures\ATest3::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\ATest3::class.' as ATestAlias;', $r);
+        $this->assertStringNotContainsString(Fixtures\ATest3::class.';', $r);
         $this->assertRegExp('/@var ATestAlias[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param ATestAlias $test3', $r);
     }
@@ -125,7 +125,7 @@ 
     {
         $i = new Injector();
 
-        $filename = __DIR__ . '/Fixtures/DuplicatePropertyClass.php';
+        $filename = __DIR__.'/Fixtures/DuplicatePropertyClass.php';
         $r = $i->injectDependencies(
             file_get_contents($filename),
             $this->getDefinition(
@@ -136,7 +136,7 @@ 
             )
         );
 
-        $this->assertStringContainsString(Fixtures\Test::class . ';', $r);
+        $this->assertStringContainsString(Fixtures\Test::class.';', $r);
         $this->assertRegExp('/@var Test[\s|\r\n]/', $r);
         $this->assertStringContainsString('@param Test $test', $r);
         $this->assertStringContainsString('__construct(Test $test)', $r);

src/Bridge/Stempler/src/ExceptionMapper.php

Braces +10 added lines, -5 removed lines

@@ -45,9 +45,12 @@ 
         $classFilename = (new \ReflectionClass($class))->getFileName();
 
         // looking for proper view error location
-        if ($e->getFile() !== $classFilename) {
-            foreach ($e->getTrace() as $trace) {
-                if (isset($trace['file']) && $trace['file'] === $classFilename) {
+        if ($e->getFile() !== $classFilename)
+        {
+            foreach ($e->getTrace() as $trace)
+            {
+                if (isset($trace['file']) && $trace['file'] === $classFilename)
+                {
                     $line = $trace['line'];
                     break;
                 }
@@ -55,7 +58,8 @@ 
         }
 
         $userStack = [];
-        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack) {
+        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack)
+        {
             $userStack[] = [
                 'file'     => $stack['file'],
                 'line'     => $stack['line'],
@@ -65,7 +69,8 @@ 
                 'args'     => [$data],
             ];
 
-            if ($stack['file'] === $filename) {
+            if ($stack['file'] === $filename)
+            {
                 // no need to jump over root template
                 break;
             }

Spacing +6 added lines, -6 removed lines

@@ -12,7 +12,7 @@ 
     public function __construct(
         private readonly SourceMap $sourcemap,
         private readonly int $lineOffset
-    ) {
+    ){
     }
 
     public function mapException(\Throwable $e, string $class, string $filename, array $data): \Throwable
@@ -21,9 +21,9 @@ 
         $classFilename = (new \ReflectionClass($class))->getFileName();
 
         // looking for proper view error location
-        if ($e->getFile() !== $classFilename) {
-            foreach ($e->getTrace() as $trace) {
-                if (isset($trace['file']) && $trace['file'] === $classFilename) {
+        if ($e->getFile() !== $classFilename){
+            foreach ($e->getTrace() as $trace){
+                if (isset($trace['file']) && $trace['file'] === $classFilename){
                     $line = $trace['line'];
                     break;
                 }
@@ -31,7 +31,7 @@ 
         }
 
         $userStack = [];
-        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack) {
+        foreach ($this->sourcemap->getStack($line - $this->lineOffset) as $stack){
             $userStack[] = [
                 'file'     => $stack['file'],
                 'line'     => $stack['line'],
@@ -41,7 +41,7 @@ 
                 'args'     => [$data],
             ];
 
-            if ($stack['file'] === $filename) {
+            if ($stack['file'] === $filename){
                 // no need to jump over root template
                 break;
             }

src/Bridge/Stempler/src/StemplerLoader.php

Braces +2 added lines, -1 removed lines

@@ -56,7 +56,8 @@
      */
     public function load(string $path): Source
     {
-        if ($this->context === null) {
+        if ($this->context === null)
+        {
             throw new EngineException('Unable to use StemplerLoader without given context.');
         }
 

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
     public function __construct(
         private readonly LoaderInterface $loader,
         array $processors
-    ) {
+    ){
         $this->processors = $processors;
     }
 
@@ -40,7 +40,7 @@ 
      */
     public function load(string $path): Source
     {
-        if ($this->context === null) {
+        if ($this->context === null){
             throw new EngineException('Unable to use StemplerLoader without given context.');
         }
 

src/Bridge/Stempler/src/StemplerView.php

Braces +2 added lines, -1 removed lines

@@ -59,7 +59,8 @@
             $this->context
         );
 
-        if ($sourcemap === null) {
+        if ($sourcemap === null)
+        {
             return $e;
         }
 

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         protected StemplerEngine $engine,
         protected ViewSource $view,
         protected ContextInterface $context
-    ) {
+    ){
         $this->container = $engine->getContainer();
     }
 
@@ -34,7 +34,7 @@ 
             $this->context
         );
 
-        if ($sourcemap === null) {
+        if ($sourcemap === null){
             return $e;
         }
 

src/Bridge/Stempler/src/StemplerCache.php

Braces +12 added lines, -6 removed lines

@@ -67,15 +67,18 @@ 
     public function isFresh(string $key): bool
     {
         $mapFilename = $this->mapFilename($key);
-        if (!$this->files->exists($mapFilename)) {
+        if (!$this->files->exists($mapFilename))
+        {
             return false;
         }
 
         $time = $this->files->time($this->filename($key));
 
         $files = (array)include $mapFilename;
-        foreach ($files as $file) {
-            if ($this->files->time($file) > $time) {
+        foreach ($files as $file)
+        {
+            if ($this->files->time($file) > $time)
+            {
                 // some partial has changed
                 return false;
             }
@@ -92,12 +95,14 @@ 
     public function delete(string $key): void
     {
         $filename = $this->filename($key);
-        if ($this->files->exists($filename)) {
+        if ($this->files->exists($filename))
+        {
             $this->files->delete($filename);
         }
 
         $mapFilename = $this->mapFilename($key);
-        if ($this->files->exists($mapFilename)) {
+        if ($this->files->exists($mapFilename))
+        {
             $this->files->delete($mapFilename);
         }
     }
@@ -110,7 +115,8 @@ 
     public function load(string $key): void
     {
         $filename = $this->filename($key);
-        if ($this->files->exists($filename)) {
+        if ($this->files->exists($filename))
+        {
             include_once $filename;
         }
     }

Spacing +7 added lines, -7 removed lines

@@ -12,7 +12,7 @@ 
     public function __construct(
         private readonly string $directory,
         private readonly FilesInterface $files = new Files()
-    ) {
+    ){
     }
 
     /**
@@ -43,15 +43,15 @@ 
     public function isFresh(string $key): bool
     {
         $mapFilename = $this->mapFilename($key);
-        if (!$this->files->exists($mapFilename)) {
+        if (!$this->files->exists($mapFilename)){
             return false;
         }
 
         $time = $this->files->time($this->filename($key));
 
         $files = (array)include $mapFilename;
-        foreach ($files as $file) {
-            if ($this->files->time($file) > $time) {
+        foreach ($files as $file){
+            if ($this->files->time($file) > $time){
                 // some partial has changed
                 return false;
             }
@@ -66,12 +66,12 @@ 
     public function delete(string $key): void
     {
         $filename = $this->filename($key);
-        if ($this->files->exists($filename)) {
+        if ($this->files->exists($filename)){
             $this->files->delete($filename);
         }
 
         $mapFilename = $this->mapFilename($key);
-        if ($this->files->exists($mapFilename)) {
+        if ($this->files->exists($mapFilename)){
             $this->files->delete($mapFilename);
         }
     }
@@ -82,7 +82,7 @@ 
     public function load(string $key): void
     {
         $filename = $this->filename($key);
-        if ($this->files->exists($filename)) {
+        if ($this->files->exists($filename)){
             include_once $filename;
         }
     }

src/Bridge/Stempler/tests/EngineTest.php

Spacing +6 added lines, -6 removed lines

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

Braces +15 added lines, -6 removed lines

@@ -57,9 +57,12 @@ 
     {
         $s = $this->getStempler();
 
-        try {
+        try
+        {
             $s->get('echo', new ViewContext())->render();
-        } catch (RenderException $e) {
+        }
+        catch (RenderException $e)
+        {
             $t = $e->getUserTrace()[0];
 
             $this->assertSame(2, $t['line']);
@@ -71,9 +74,12 @@ 
     {
         $s = $this->getStempler();
 
-        try {
+        try
+        {
             $s->get('other:echo-in', new ViewContext())->render();
-        } catch (RenderException $e) {
+        }
+        catch (RenderException $e)
+        {
             $t = $e->getUserTrace();
             $this->assertCount(2, $t);
 
@@ -89,9 +95,12 @@ 
     {
         $twig = $this->getStempler();
 
-        try {
+        try
+        {
             $twig->get('other:bad', new ViewContext());
-        } catch (CompileException $e) {
+        }
+        catch (CompileException $e)
+        {
             $this->assertStringContainsString('bad.dark.php', $e->getFile());
         }
     }

src/Bridge/Stempler/tests/CacheTest.php

Spacing +3 added lines, -3 removed lines

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