spiral / framework

src/Core/tests/ExceptionsTest.php

Braces +15 added lines, -6 removed lines

@@ -208,10 +208,13 @@ 
                 context: Parameter #1 [ <required> Spiral\Tests\Core\Fixtures\InvalidClass $class ]
             MARKDOWN;
 
-        try {
+        try
+        {
             $container->get(InvalidWithContainerInside::class);
             self::fail('Exception `ContainerException` expected');
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame($expectedMessage, $e->getMessage());
         }
     }
@@ -234,9 +237,12 @@ 
     {
         $container = new Container();
 
-        try {
+        try
+        {
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame(<<<MARKDOWN
                 Can't autowire `invalid`: class or injector not found.
                 Resolving trace:
@@ -267,9 +273,12 @@ 
     {
         $this->expectException(ContainerException::class);
 
-        try {
+        try
+        {
             $container->get(ClassWithUndefinedDependency::class);
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame($message, $e->getMessage());
 
             throw $e;

src/Core/tests/Scope/ScopeAttributeTest.php

Braces +10 added lines, -4 removed lines

@@ -166,7 +166,8 @@ 
 
         $root = self::makeContainer();
 
-        try {
+        try
+        {
             $root->runScoped(static function (Container $c1): void {
                 $c1->runScoped(static function (Container $c2): void {
                     $c2->runScoped(static function (Container $c3): void {
@@ -174,7 +175,9 @@ 
                     }, name: 'root');
                 });
             });
-        } catch (NamedScopeDuplicationException $e) {
+        }
+        catch (NamedScopeDuplicationException $e)
+        {
             self::assertSame('root', $e->getScope());
             throw $e;
         }
@@ -187,7 +190,8 @@ 
     #[Group('scrutinizer-ignore')]
     public function testBadScopeException(): void
     {
-        try {
+        try
+        {
             $root = self::makeContainer();
             $root->runScoped(static function (Container $c1): void {
                 $c1->runScoped(static function (Container $c2): void {
@@ -196,7 +200,9 @@ 
             }, name: 'bar');
 
             self::fail(BadScopeException::class . ' should be thrown');
-        } catch (BadScopeException $e) {
+        }
+        catch (BadScopeException $e)
+        {
             self::assertSame('foo', $e->getScope());
         }
     }

src/Core/tests/Fixtures/PrivateConstructor.php

Braces +3 added lines, -1 removed lines

@@ -6,7 +6,9 @@
 
 final class PrivateConstructor
 {
-    private function __construct() {}
+    private function __construct()
+    {
+}
 
     private static function privateMethod(int $result): int
     {

src/Core/src/Internal/Scope.php

Braces +8 added lines, -4 removed lines

@@ -17,7 +17,8 @@ 
 
     public function __construct(
         private readonly ?string $scopeName = null,
-    ) {}
+    ) {
+}
 
     public function getScopeName(): ?string
     {
@@ -39,9 +40,11 @@ 
         $this->parentActor = $actor;
 
         // Check a scope with the same name is not already registered
-        if ($this->scopeName !== null) {
+        if ($this->scopeName !== null)
+        {
             $tmp = $this;
-            while ($tmp->parentScope !== null) {
+            while ($tmp->parentScope !== null)
+            {
                 $tmp = $tmp->parentScope;
                 $tmp->scopeName !== $this->scopeName ?: throw new NamedScopeDuplicationException($this->scopeName);
             }
@@ -64,7 +67,8 @@ 
         $result = [$this->scopeName];
 
         $parent = $this;
-        while ($parent->parentScope !== null) {
+        while ($parent->parentScope !== null)
+        {
             $parent = $parent->parentScope;
             $result[] = $parent->scopeName;
         }

src/Core/src/Internal/Invoker.php

Braces +23 added lines, -10 removed lines

@@ -42,26 +42,34 @@ 
      */
     public function invoke(mixed $target, array $parameters = []): mixed
     {
-        if (\is_array($target) && isset($target[1])) {
+        if (\is_array($target) && isset($target[1]))
+        {
             // In a form of resolver and method
             [$resolver, $method] = $target;
 
             // Resolver instance or class name if the method is static (i.e. [ClassName::class, 'method'])
-            if (\is_string($resolver)) {
+            if (\is_string($resolver))
+            {
                 // Detect return type
                 $type = $this->actor->resolveType($resolver, $binding, $singleton, $injector);
 
-                if ($singleton === null) {
+                if ($singleton === null)
+                {
                     $type ??= $injector === null && $binding === null ? $resolver : null;
                     $resolver = \is_callable([$type, $method]) ? $type : $this->container->get($resolver);
-                } else {
+                }
+                else
+                {
                     $resolver = $singleton;
                 }
             }
 
-            try {
+            try
+            {
                 $method = new \ReflectionMethod($resolver, $method);
-            } catch (\ReflectionException $e) {
+            }
+            catch (\ReflectionException $e)
+            {
                 throw new ContainerException($e->getMessage(), $e->getCode(), $e);
             }
 
@@ -72,14 +80,19 @@ 
             );
         }
 
-        if (\is_string($target) && \is_callable($target)) {
+        if (\is_string($target) && \is_callable($target))
+        {
             $target = $target(...);
         }
 
-        if ($target instanceof \Closure) {
-            try {
+        if ($target instanceof \Closure)
+        {
+            try
+            {
                 $reflection = new \ReflectionFunction($target);
-            } catch (\ReflectionException $e) {
+            }
+            catch (\ReflectionException $e)
+            {
                 throw new ContainerException($e->getMessage(), $e->getCode(), $e);
             }
 

src/Core/src/Internal/Tracer.php

Braces +13 added lines, -6 removed lines

@@ -34,19 +34,24 @@ 
     public function push(bool $nextLevel, mixed ...$details): void
     {
         $trace = $details === [] ? null : new Trace($details);
-        if ($nextLevel || $this->traces === []) {
+        if ($nextLevel || $this->traces === [])
+        {
             $this->traces[] = $trace === null ? [] : [$trace];
-        } elseif ($trace !== null) {
+        }
+        elseif ($trace !== null)
+        {
             $this->traces[\array_key_last($this->traces)][] = $trace;
         }
     }
 
     public function pop(bool $previousLevel = false): void
     {
-        if ($this->traces === []) {
+        if ($this->traces === [])
+        {
             return;
         }
-        if ($previousLevel) {
+        if ($previousLevel)
+        {
             \array_pop($this->traces);
             return;
         }
@@ -72,7 +77,8 @@ 
     {
         $result = [];
         $i = 0;
-        foreach ($blocks as $block) {
+        foreach ($blocks as $block)
+        {
             \array_push($result, ...self::blockToStringList($block, $i++));
         }
         return \implode("\n", $result);
@@ -92,7 +98,8 @@ 
         // Separator
         $s = "\n";
         $nexPrefix = "$s$padding  ";
-        foreach ($items as $item) {
+        foreach ($items as $item)
+        {
             $result[] = $firstPrefix . \str_replace($s, $nexPrefix, (string) $item);
         }
         return $result;

src/Core/src/Container.php

Braces +48 added lines, -22 removed lines

@@ -64,7 +64,8 @@ 
         string|\BackedEnum|null $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
         private Options $options = new Options(),
     ) {
-        if (\is_object($scopeName)) {
+        if (\is_object($scopeName))
+        {
             $scopeName = (string) $scopeName->value;
         }
 
@@ -166,23 +167,29 @@ 
      */
     public function runScope(Scope|array $bindings, callable $scope): mixed
     {
-        if (!\is_array($bindings)) {
+        if (!\is_array($bindings))
+        {
             return $this->runIsolatedScope($bindings, $scope);
         }
 
         $binds = &$this->state->bindings;
         $singletons = &$this->state->singletons;
         $cleanup = $previous = $prevSin = [];
-        foreach ($bindings as $alias => $resolver) {
+        foreach ($bindings as $alias => $resolver)
+        {
             // Store previous bindings
-            if (isset($binds[$alias])) {
+            if (isset($binds[$alias]))
+            {
                 $previous[$alias] = $binds[$alias];
-            } else {
+            }
+            else
+            {
                 // Store bindings to be removed
                 $cleanup[] = $alias;
             }
             // Store previous singletons
-            if (isset($singletons[$alias])) {
+            if (isset($singletons[$alias]))
+            {
                 $prevSin[$alias] = $singletons[$alias];
                 unset($singletons[$alias]);
             }
@@ -190,21 +197,27 @@ 
             $this->binder->bind($alias, $resolver);
         }
 
-        try {
+        try
+        {
             return ContainerScope::getContainer() !== $this
                 ? ContainerScope::runScope($this, $scope)
                 : $scope($this);
-        } finally {
+        }
+        finally
+        {
             // Remove new bindings
-            foreach ($cleanup as $alias) {
+            foreach ($cleanup as $alias)
+            {
                 unset($binds[$alias], $singletons[$alias]);
             }
             // Restore previous bindings
-            foreach ($previous as $alias => $resolver) {
+            foreach ($previous as $alias => $resolver)
+            {
                 $binds[$alias] = $resolver;
             }
             // Restore singletons
-            foreach ($prevSin as $alias => $instance) {
+            foreach ($prevSin as $alias => $instance)
+            {
                 $singletons[$alias] = $instance;
             }
         }
@@ -254,7 +267,8 @@ 
      */
     public function bindSingleton(string $alias, string|array|callable|object $resolver, ?bool $force = null): void
     {
-        if ($force ?? $this->options->allowSingletonsRebinding) {
+        if ($force ?? $this->options->allowSingletonsRebinding)
+        {
             $this->binder->removeBinding($alias);
         }
 
@@ -338,8 +352,10 @@ 
         ], $this->options);
 
         // Create container services
-        foreach ($container->config as $property => $class) {
-            if (\property_exists($container, $property)) {
+        foreach ($container->config as $property => $class)
+        {
+            if (\property_exists($container, $property))
+            {
                 $container->$property = $constructor->get($property, $class);
             }
         }
@@ -353,7 +369,8 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope))
+        {
             $this->destruct();
             return;
         }
@@ -362,10 +379,14 @@ 
 
         // Run finalizers
         $errors = [];
-        foreach ($this->state->finalizers as $finalizer) {
-            try {
+        foreach ($this->state->finalizers as $finalizer)
+        {
+            try
+            {
                 $this->invoker->invoke($finalizer);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 $errors[] = $e;
             }
         }
@@ -374,7 +395,8 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== [])
+        {
             throw new FinalizersException($scopeName, $errors);
         }
     }
@@ -396,18 +418,22 @@ 
         $container->scope->setParent($this, $this->scope, $this->actor);
 
         // Add specific bindings
-        foreach ($config->bindings as $alias => $resolver) {
+        foreach ($config->bindings as $alias => $resolver)
+        {
             $container->binder->bind($alias, $resolver);
         }
 
         return ContainerScope::runScope(
             $container,
             static function (self $container) use ($config, $closure): mixed {
-                try {
+                try
+                {
                     return $config->autowire
                         ? $container->invoke($closure)
                         : $closure($container);
-                } finally {
+                }
+                finally
+                {
                     $container->closeScope();
                 }
             },

src/Core/src/Exception/Container/TracedContainerException.php

Braces +2 added lines, -1 removed lines

@@ -37,7 +37,8 @@
     public static function extendTracedException(string $message, array $traces, self $exception): static
     {
         $merge = $exception->containerTrace;
-        if ($traces !== [] && $merge !== []) {
+        if ($traces !== [] && $merge !== [])
+        {
             // merge lat element of $traces with first element of $merge
             \array_push($traces[\count($traces) - 1], ...$merge[0]);
             unset($merge[0]);

src/Core/src/Exception/Container/AutowireException.php

Braces +3 added lines, -1 removed lines

@@ -7,4 +7,6 @@
 /**
  * When class can not be created.
  */
-class AutowireException extends TracedContainerException {}
+class AutowireException extends TracedContainerException
+{
+}