spiral / framework

src/Core/src/Internal/Introspector.php

Spacing +4 added lines, -4 removed lines

@@ -28,10 +28,10 @@ 
     {
         $scope = self::getAccessor($container)->scope;
         $result = [];
-        do {
+        do{
             $result[] = $scope->getScopeName();
             $scope = $scope->getParentScope();
-        } while ($scope !== null);
+        }while ($scope !== null);
 
         return $result;
     }
@@ -46,12 +46,12 @@ 
             Proxy::isProxy($container) => ContainerScope::getContainer() ?? throw new \RuntimeException(
                 'Container Proxy is out of scope.',
             ),
-            default => throw new \InvalidArgumentException('Container must be an instance of ' . Container::class),
+            default => throw new \InvalidArgumentException('Container must be an instance of '.Container::class),
         };
 
         $container ??= ContainerScope::getContainer();
 
-        if (!$container instanceof Container) {
+        if (!$container instanceof Container){
             throw new \RuntimeException('Container is not available.');
         }
 

Braces +4 added lines, -2 removed lines

@@ -28,7 +28,8 @@ 
     {
         $scope = self::getAccessor($container)->scope;
         $result = [];
-        do {
+        do
+        {
             $result[] = $scope->getScopeName();
             $scope = $scope->getParentScope();
         } while ($scope !== null);
@@ -51,7 +52,8 @@ 
 
         $container ??= ContainerScope::getContainer();
 
-        if (!$container instanceof Container) {
+        if (!$container instanceof Container)
+        {
             throw new \RuntimeException('Container is not available.');
         }
 

src/Http/src/Request/InputManager.php

Spacing +23 added lines, -23 removed lines

@@ -122,7 +122,7 @@ 
         #[Proxy] private readonly ContainerInterface $container,
         /** @invisible */
         HttpConfig $config = new HttpConfig()
-    ) {
+    ){
         $this->bagAssociations = \array_merge($this->bagAssociations, $config->getInputBags());
     }
 
@@ -151,10 +151,10 @@ 
     {
         $input = clone $this;
 
-        if ($add) {
-            $input->prefix .= '.' . $prefix;
+        if ($add){
+            $input->prefix .= '.'.$prefix;
             $input->prefix = \trim($input->prefix, '.');
-        } else {
+        }else{
             $input->prefix = $prefix;
         }
 
@@ -170,7 +170,7 @@ 
 
         return match (true) {
             empty($path) => '/',
-            $path[0] !== '/' => '/' . $path,
+            $path[0] !== '/' => '/'.$path,
             default => $path
         };
     }
@@ -190,9 +190,9 @@ 
      */
     public function request(): Request
     {
-        try {
+        try{
             $request = $this->container->get(Request::class);
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             $scope = implode('.', \array_reverse(Introspector::scopeNames($this->container)));
             throw new ScopeException(
                 "Unable to get `ServerRequestInterface` in the `$scope` container scope",
@@ -202,7 +202,7 @@ 
         }
 
         // Flushing input state
-        if ($this->request !== $request) {
+        if ($this->request !== $request){
             $this->bags = [];
             $this->request = $request;
         }
@@ -215,11 +215,11 @@ 
      */
     public function bearerToken(): ?string
     {
-        $header = (string) $this->header('Authorization', '');
+        $header = (string)$this->header('Authorization', '');
 
         $position = \strrpos($header, 'Bearer ');
 
-        if ($position !== false) {
+        if ($position !== false){
             $header = \substr($header, $position + 7);
 
             return \str_contains($header, ',')
@@ -273,16 +273,16 @@ 
     public function isJsonExpected(bool $softMatch = false): bool
     {
         $acceptHeader = AcceptHeader::fromString($this->request()->getHeaderLine('Accept'));
-        foreach ($this->jsonTypes as $jsonType) {
-            if ($acceptHeader->has($jsonType)) {
+        foreach ($this->jsonTypes as $jsonType){
+            if ($acceptHeader->has($jsonType)){
                 return true;
             }
         }
 
-        if ($softMatch) {
-            foreach ($acceptHeader->getAll() as $item) {
-                $itemValue = \strtolower((string) $item->getValue());
-                if (\str_ends_with($itemValue, '/json') || \str_ends_with($itemValue, '+json')) {
+        if ($softMatch){
+            foreach ($acceptHeader->getAll() as $item){
+                $itemValue = \strtolower((string)$item->getValue());
+                if (\str_ends_with($itemValue, '/json') || \str_ends_with($itemValue, '+json')){
                     return true;
                 }
             }
@@ -321,19 +321,19 @@ 
         // ensure proper request association
         $this->request();
 
-        if (isset($this->bags[$name])) {
+        if (isset($this->bags[$name])){
             return $this->bags[$name];
         }
 
         $definition = $this->findBagDefinition($name);
-        if (!$definition) {
+        if (!$definition){
             throw new InputException(\sprintf("Undefined input bag '%s'", $name));
         }
 
         $class = $definition['class'];
         $data = \call_user_func([$this->request(), $definition['source']]);
 
-        if (!\is_array($data)) {
+        if (!\is_array($data)){
             $data = (array)$data;
         }
 
@@ -342,7 +342,7 @@ 
 
     public function hasBag(string $name): bool
     {
-        if (isset($this->bags[$name])) {
+        if (isset($this->bags[$name])){
             return true;
         }
 
@@ -362,12 +362,12 @@ 
      */
     private function findBagDefinition(string $name): ?array
     {
-        if (isset($this->bagAssociations[$name])) {
+        if (isset($this->bagAssociations[$name])){
             return $this->bagAssociations[$name];
         }
 
-        foreach ($this->bagAssociations as $bag) {
-            if (isset($bag['alias']) && $bag['alias'] === $name) {
+        foreach ($this->bagAssociations as $bag){
+            if (isset($bag['alias']) && $bag['alias'] === $name){
                 return $bag;
             }
         }

Braces +38 added lines, -18 removed lines

@@ -151,10 +151,13 @@ 
     {
         $input = clone $this;
 
-        if ($add) {
+        if ($add)
+        {
             $input->prefix .= '.' . $prefix;
             $input->prefix = \trim($input->prefix, '.');
-        } else {
+        }
+        else
+        {
             $input->prefix = $prefix;
         }
 
@@ -190,9 +193,12 @@ 
      */
     public function request(): Request
     {
-        try {
+        try
+        {
             $request = $this->container->get(Request::class);
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             $scope = implode('.', \array_reverse(Introspector::scopeNames($this->container)));
             throw new ScopeException(
                 "Unable to get `ServerRequestInterface` in the `$scope` container scope",
@@ -202,7 +208,8 @@ 
         }
 
         // Flushing input state
-        if ($this->request !== $request) {
+        if ($this->request !== $request)
+        {
             $this->bags = [];
             $this->request = $request;
         }
@@ -219,7 +226,8 @@ 
 
         $position = \strrpos($header, 'Bearer ');
 
-        if ($position !== false) {
+        if ($position !== false)
+        {
             $header = \substr($header, $position + 7);
 
             return \str_contains($header, ',')
@@ -273,16 +281,21 @@ 
     public function isJsonExpected(bool $softMatch = false): bool
     {
         $acceptHeader = AcceptHeader::fromString($this->request()->getHeaderLine('Accept'));
-        foreach ($this->jsonTypes as $jsonType) {
-            if ($acceptHeader->has($jsonType)) {
+        foreach ($this->jsonTypes as $jsonType)
+        {
+            if ($acceptHeader->has($jsonType))
+            {
                 return true;
             }
         }
 
-        if ($softMatch) {
-            foreach ($acceptHeader->getAll() as $item) {
+        if ($softMatch)
+        {
+            foreach ($acceptHeader->getAll() as $item)
+            {
                 $itemValue = \strtolower((string) $item->getValue());
-                if (\str_ends_with($itemValue, '/json') || \str_ends_with($itemValue, '+json')) {
+                if (\str_ends_with($itemValue, '/json') || \str_ends_with($itemValue, '+json'))
+                {
                     return true;
                 }
             }
@@ -321,19 +334,22 @@ 
         // ensure proper request association
         $this->request();
 
-        if (isset($this->bags[$name])) {
+        if (isset($this->bags[$name]))
+        {
             return $this->bags[$name];
         }
 
         $definition = $this->findBagDefinition($name);
-        if (!$definition) {
+        if (!$definition)
+        {
             throw new InputException(\sprintf("Undefined input bag '%s'", $name));
         }
 
         $class = $definition['class'];
         $data = \call_user_func([$this->request(), $definition['source']]);
 
-        if (!\is_array($data)) {
+        if (!\is_array($data))
+        {
             $data = (array)$data;
         }
 
@@ -342,7 +358,8 @@ 
 
     public function hasBag(string $name): bool
     {
-        if (isset($this->bags[$name])) {
+        if (isset($this->bags[$name]))
+        {
             return true;
         }
 
@@ -362,12 +379,15 @@ 
      */
     private function findBagDefinition(string $name): ?array
     {
-        if (isset($this->bagAssociations[$name])) {
+        if (isset($this->bagAssociations[$name]))
+        {
             return $this->bagAssociations[$name];
         }
 
-        foreach ($this->bagAssociations as $bag) {
-            if (isset($bag['alias']) && $bag['alias'] === $name) {
+        foreach ($this->bagAssociations as $bag)
+        {
+            if (isset($bag['alias']) && $bag['alias'] === $name)
+            {
                 return $bag;
             }
         }

src/Bridge/Stempler/src/StemplerEngine.php

Spacing +27 added lines, -27 removed lines

@@ -46,7 +46,7 @@ 
         #[Proxy] private readonly ContainerInterface $container,
         private readonly StemplerConfig $config,
         private readonly ?StemplerCache $cache = null
-    ) {
+    ){
     }
 
     public function getContainer(): ContainerInterface
@@ -65,7 +65,7 @@ 
 
     public function getLoader(): LoaderInterface
     {
-        if ($this->loader === null) {
+        if ($this->loader === null){
             throw new EngineException('No associated loader found');
         }
 
@@ -77,13 +77,13 @@ 
      */
     public function getBuilder(ContextInterface $context): Builder
     {
-        if ($this->builder === null) {
+        if ($this->builder === null){
             throw new EngineException('No associated builder found');
         }
 
         // since view source support pre-processing we must ensure that context is always set
         $loader = $this->builder->getLoader();
-        if ($loader instanceof StemplerLoader) {
+        if ($loader instanceof StemplerLoader){
             $loader->setContext($context);
         }
 
@@ -101,20 +101,20 @@ 
         // cache key
         $key = $this->cacheKey($view, $context);
 
-        if ($this->cache !== null && $this->cache->isFresh($key)) {
+        if ($this->cache !== null && $this->cache->isFresh($key)){
             $this->cache->load($key);
-        } elseif (!\class_exists($class)) {
-            try {
+        } elseif (!\class_exists($class)){
+            try{
                 $builder = $this->getBuilder($context);
 
                 $result = $builder->compile($path);
-            } catch (Throwable $e) {
+            }catch (Throwable $e){
                 throw new CompileException($e);
             }
 
             $compiled = $this->compileClass($class, $result);
 
-            if ($this->cache !== null) {
+            if ($this->cache !== null){
                 $this->cache->write(
                     $key,
                     $compiled,
@@ -127,13 +127,13 @@ 
                 $this->cache->load($key);
             }
 
-            if (!\class_exists($class)) {
+            if (!\class_exists($class)){
                 // runtime initialization
-                eval('?>' . $compiled);
+                eval('?>'.$compiled);
             }
         }
 
-        if (!\class_exists($class) || !\is_subclass_of($class, ViewInterface::class)) {
+        if (!\class_exists($class) || !\is_subclass_of($class, ViewInterface::class)){
             throw new EngineException(\sprintf('Unable to load `%s`, cache might be corrupted.', $path));
         }
 
@@ -142,7 +142,7 @@ 
 
     public function reset(string $path, ContextInterface $context): void
     {
-        if ($this->cache === null) {
+        if ($this->cache === null){
             return;
         }
 
@@ -161,12 +161,12 @@ 
      */
     public function makeSourceMap(string $path, ContextInterface $context): ?SourceMap
     {
-        try {
+        try{
             $builder = $this->getBuilder($context);
 
             // there is no need to cache sourcemaps since they are used during the exception only
             return $builder->compile($path)->getSourceMap($builder->getLoader());
-        } catch (Throwable) {
+        }catch (Throwable){
             return null;
         }
     }
@@ -202,7 +202,7 @@ 
      */
     private function className(ViewSource $source, ContextInterface $context): string
     {
-        return $this->classPrefix . $this->cacheKey($source, $context);
+        return $this->classPrefix.$this->cacheKey($source, $context);
     }
 
     private function cacheKey(ViewSource $source, ContextInterface $context): string
@@ -222,7 +222,7 @@ 
         $builder = new Builder($loader);
 
         $directivesGroup = new DirectiveGroup();
-        foreach ($this->getDirectives() as $directive) {
+        foreach ($this->getDirectives() as $directive){
             $directivesGroup->addDirective($directive);
         }
 
@@ -253,7 +253,7 @@ 
         $builder->getCompiler()->addRenderer(new DynamicRenderer(new DirectiveGroup($this->getDirectives())));
 
         // ATS modifications
-        foreach ($this->getVisitors(Builder::STAGE_PREPARE) as $visitor) {
+        foreach ($this->getVisitors(Builder::STAGE_PREPARE) as $visitor){
             $builder->addVisitor($visitor, Builder::STAGE_PREPARE);
         }
 
@@ -266,15 +266,15 @@ 
         $builder->addVisitor(new ResolveImports($builder), Builder::STAGE_TRANSFORM);
         $builder->addVisitor(new ExtendsParent($builder), Builder::STAGE_TRANSFORM);
 
-        foreach ($this->getVisitors(Builder::STAGE_TRANSFORM) as $visitor) {
+        foreach ($this->getVisitors(Builder::STAGE_TRANSFORM) as $visitor){
             $builder->addVisitor($visitor, Builder::STAGE_TRANSFORM);
         }
 
-        foreach ($this->getVisitors(Builder::STAGE_FINALIZE) as $visitor) {
+        foreach ($this->getVisitors(Builder::STAGE_FINALIZE) as $visitor){
             $builder->addVisitor($visitor, Builder::STAGE_FINALIZE);
         }
 
-        foreach ($this->getVisitors(Builder::STAGE_COMPILE) as $visitor) {
+        foreach ($this->getVisitors(Builder::STAGE_COMPILE) as $visitor){
             $builder->addVisitor($visitor, Builder::STAGE_COMPILE);
         }
 
@@ -287,8 +287,8 @@ 
     private function getVisitors(int $stage): iterable
     {
         $result = [];
-        foreach ($this->config->getVisitors($stage) as $visitor) {
-            if ($visitor instanceof Autowire) {
+        foreach ($this->config->getVisitors($stage) as $visitor){
+            if ($visitor instanceof Autowire){
                 $result[] = $visitor->resolve($this->container->get(FactoryInterface::class));
                 continue;
             }
@@ -305,8 +305,8 @@ 
     private function getProcessors(): iterable
     {
         $result = [];
-        foreach ($this->config->getProcessors() as $processor) {
-            if ($processor instanceof Autowire) {
+        foreach ($this->config->getProcessors() as $processor){
+            if ($processor instanceof Autowire){
                 $result[] = $processor->resolve($this->container->get(FactoryInterface::class));
                 continue;
             }
@@ -323,8 +323,8 @@ 
     private function getDirectives(): iterable
     {
         $result = [];
-        foreach ($this->config->getDirectives() as $directive) {
-            if ($directive instanceof Autowire) {
+        foreach ($this->config->getDirectives() as $directive){
+            if ($directive instanceof Autowire){
                 $result[] = $directive->resolve($this->container->get(FactoryInterface::class));
                 continue;
             }

src/Bridge/Stempler/tests/BaseTestCase.php

Spacing +3 added lines, -3 removed lines

@@ -40,9 +40,9 @@
     public function defineDirectories(string $root): array
     {
         return [
-            'app'   => __DIR__ . '/fixtures',
-            'cache' => __DIR__ . '/cache',
-            'config' => __DIR__ . '/config',
+            'app'   => __DIR__.'/fixtures',
+            'cache' => __DIR__.'/cache',
+            'config' => __DIR__.'/config',
         ] + parent::defineDirectories($root);
     }
 

src/Bridge/Stempler/tests/CacheTest.php

Spacing +3 added lines, -3 removed lines

@@ -30,13 +30,13 @@
     #[TestScope("http")]
     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'));
     }
 }

src/SendIt/tests/RenderTest.php

Spacing +3 added lines, -3 removed lines

@@ -33,14 +33,14 @@
     public function defineDirectories(string $root): array
     {
         return [
-                'root' => __DIR__ . '/App',
-                'app'  => __DIR__ . '/App'
+                'root' => __DIR__.'/App',
+                'app'  => __DIR__.'/App'
             ] + parent::defineDirectories($root);
     }
 
     public function tearDown(): void
     {
-        foreach (glob(__DIR__ . '/App/runtime/cache/views/*.php') as $file) {
+        foreach (glob(__DIR__.'/App/runtime/cache/views/*.php') as $file){
             @unlink($file);
         }
     }

src/Logger/src/Traits/LoggerTrait.php

Spacing +3 added lines, -3 removed lines

@@ -29,11 +29,11 @@ 
      */
     protected function getLogger(string $channel = null): LoggerInterface
     {
-        if ($channel !== null) {
+        if ($channel !== null){
             return $this->allocateLogger($channel);
         }
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null){
             return $this->logger;
         }
 
@@ -47,7 +47,7 @@ 
     private function allocateLogger(string $channel): LoggerInterface
     {
         $container = ContainerScope::getContainer();
-        if ($container === null || !$container->has(LogsInterface::class)) {
+        if ($container === null || !$container->has(LogsInterface::class)){
             return $this->logger ?? new NullLogger();
         }
 

Braces +6 added lines, -3 removed lines

@@ -29,11 +29,13 @@ 
      */
     protected function getLogger(string $channel = null): LoggerInterface
     {
-        if ($channel !== null) {
+        if ($channel !== null)
+        {
             return $this->allocateLogger($channel);
         }
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null)
+        {
             return $this->logger;
         }
 
@@ -47,7 +49,8 @@ 
     private function allocateLogger(string $channel): LoggerInterface
     {
         $container = ContainerScope::getContainer();
-        if ($container === null || !$container->has(LogsInterface::class)) {
+        if ($container === null || !$container->has(LogsInterface::class))
+        {
             return $this->logger ?? new NullLogger();
         }
 

src/Bridge/Monolog/tests/TraitTest.php

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
     public function testNoScope(): void
     {
         $c = new Container();
-        $c->runScope(new Scope(), function () {
+        $c->runScope(new Scope(), function (){
             $logger = $this->getLogger();
             $this->assertInstanceOf(NullLogger::class, $this->getLogger());
             $this->assertSame($logger, $this->getLogger());
@@ -51,7 +51,7 @@ 
 
         $c->bind(LogsInterface::class, $mock);
 
-        $c->runScope(new Scope(), function () {
+        $c->runScope(new Scope(), function (){
             $logger = $this->getLogger();
             $this->assertInstanceOf(\Spiral\Logger\NullLogger::class, $this->getLogger());
             $this->assertSame($logger, $this->getLogger());
@@ -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 +6 added lines, -3 removed lines

@@ -35,7 +35,8 @@ 
     public function testNoScope(): void
     {
         $c = new Container();
-        $c->runScope(new Scope(), function () {
+        $c->runScope(new Scope(), function ()
+        {
             $logger = $this->getLogger();
             $this->assertInstanceOf(NullLogger::class, $this->getLogger());
             $this->assertSame($logger, $this->getLogger());
@@ -51,7 +52,8 @@ 
 
         $c->bind(LogsInterface::class, $mock);
 
-        $c->runScope(new Scope(), function () {
+        $c->runScope(new Scope(), function ()
+        {
             $logger = $this->getLogger();
             $this->assertInstanceOf(\Spiral\Logger\NullLogger::class, $this->getLogger());
             $this->assertSame($logger, $this->getLogger());
@@ -73,7 +75,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/Stempler/tests/EngineTest.php

Spacing +6 added lines, -6 removed lines

@@ -52,10 +52,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];
 
             $this->assertSame(2, $t['line']);
@@ -67,10 +67,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();
             $this->assertCount(2, $t);
 
@@ -86,9 +86,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

@@ -52,10 +52,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];
 
             $this->assertSame(2, $t['line']);
@@ -67,10 +70,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();
             $this->assertCount(2, $t);
 
@@ -86,9 +92,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());
         }
     }