spiral / framework

src/Core/tests/ExceptionsTest.php

Spacing +6 added lines, -6 removed lines

@@ -208,10 +208,10 @@ 
                 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 +234,9 @@ 
     {
         $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 +267,9 @@ 
     {
         $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

Spacing +5 added lines, -5 removed lines

@@ -166,7 +166,7 @@ 
 
         $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 +174,7 @@ 
                     }, name: 'root');
                 });
             });
-        } catch (NamedScopeDuplicationException $e) {
+        }catch (NamedScopeDuplicationException $e){
             self::assertSame('root', $e->getScope());
             throw $e;
         }
@@ -187,7 +187,7 @@ 
     #[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 {
@@ -195,8 +195,8 @@ 
                 });
             }, name: 'bar');
 
-            self::fail(BadScopeException::class . ' should be thrown');
-        } catch (BadScopeException $e) {
+            self::fail(BadScopeException::class.' should be thrown');
+        }catch (BadScopeException $e){
             self::assertSame('foo', $e->getScope());
         }
     }

src/Core/tests/Fixtures/PrivateConstructor.php

Spacing +1 added lines, -1 removed lines

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

src/Core/src/Internal/Scope.php

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
 
     public function __construct(
         private readonly ?string $scopeName = null,
-    ) {}
+    ){}
 
     public function getScopeName(): ?string
     {
@@ -39,9 +39,9 @@ 
         $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 +64,7 @@ 
         $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

Spacing +10 added lines, -10 removed lines

@@ -42,26 +42,26 @@ 
      */
     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 +72,14 @@ 
             );
         }
 
-        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

Spacing +8 added lines, -8 removed lines

@@ -34,19 +34,19 @@ 
     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;
         }
@@ -62,7 +62,7 @@ 
 
     public function __toString(): string
     {
-        return $this->traces === [] ? '' : "Resolving trace:\n" . self::renderTraceList($this->traces);
+        return $this->traces === [] ? '' : "Resolving trace:\n".self::renderTraceList($this->traces);
     }
 
     /**
@@ -72,7 +72,7 @@ 
     {
         $result = [];
         $i = 0;
-        foreach ($blocks as $block) {
+        foreach ($blocks as $block){
             \array_push($result, ...self::blockToStringList($block, $i++));
         }
         return \implode("\n", $result);
@@ -92,8 +92,8 @@ 
         // Separator
         $s = "\n";
         $nexPrefix = "$s$padding  ";
-        foreach ($items as $item) {
-            $result[] = $firstPrefix . \str_replace($s, $nexPrefix, (string) $item);
+        foreach ($items as $item){
+            $result[] = $firstPrefix.\str_replace($s, $nexPrefix, (string)$item);
         }
         return $result;
     }

src/Core/src/Container.php

Spacing +38 added lines, -38 removed lines

@@ -48,11 +48,11 @@ 
     public const DEFAULT_ROOT_SCOPE_NAME = 'root';
 
     private Internal\State $state;
-    private ResolverInterface|Internal\Resolver $resolver;
-    private FactoryInterface|Internal\Factory $factory;
-    private ContainerInterface|Internal\Container $container;
-    private BinderInterface|Internal\Binder $binder;
-    private InvokerInterface|Internal\Invoker $invoker;
+    private ResolverInterface | Internal\Resolver $resolver;
+    private FactoryInterface | Internal\Factory $factory;
+    private ContainerInterface | Internal\Container $container;
+    private BinderInterface | Internal\Binder $binder;
+    private InvokerInterface | Internal\Invoker $invoker;
     private Internal\Scope $scope;
     private Internal\Actor $actor;
 
@@ -61,11 +61,11 @@ 
      */
     public function __construct(
         private Config $config = new Config(),
-        string|\BackedEnum|null $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
+        string | \BackedEnum | null $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
         private Options $options = new Options(),
-    ) {
-        if (\is_object($scopeName)) {
-            $scopeName = (string) $scopeName->value;
+    ){
+        if (\is_object($scopeName)){
+            $scopeName = (string)$scopeName->value;
         }
 
         $this->initServices($this, $scopeName);
@@ -106,11 +106,11 @@ 
      * @throws \Throwable
      * @psalm-suppress TooManyArguments
      */
-    public function make(string $alias, array $parameters = [], \Stringable|string|null $context = null): mixed
+    public function make(string $alias, array $parameters = [], \Stringable | string | null $context = null): mixed
     {
         return ContainerScope::getContainer() === $this
             ? $this->factory->make($alias, $parameters, $context)
-            : ContainerScope::runScope($this, fn(): mixed => $this->factory->make($alias, $parameters, $context));
+            : ContainerScope::runScope($this, fn() : mixed => $this->factory->make($alias, $parameters, $context));
     }
 
     /**
@@ -132,7 +132,7 @@ 
      * @throws \Throwable
      * @psalm-suppress TooManyArguments
      */
-    public function get(string|Autowire $id, \Stringable|string|null $context = null): mixed
+    public function get(string | Autowire $id, \Stringable | string | null $context = null): mixed
     {
         return ContainerScope::getContainer() === $this
             ? $this->container->get($id, $context)
@@ -152,9 +152,9 @@ 
      *        If {@see string}, the default binder for the given scope will be returned. Default bindings won't affect
      *        already created Container instances except the case with the root one.
      */
-    public function getBinder(string|\BackedEnum|null $scope = null): BinderInterface
+    public function getBinder(string | \BackedEnum | null $scope = null): BinderInterface
     {
-        $scope = \is_object($scope) ? (string) $scope->value : $scope;
+        $scope = \is_object($scope) ? (string)$scope->value : $scope;
 
         return $scope === null
             ? $this->binder
@@ -164,25 +164,25 @@ 
     /**
      * @throws \Throwable
      */
-    public function runScope(Scope|array $bindings, callable $scope): mixed
+    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 +190,21 @@ 
             $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;
             }
         }
@@ -252,9 +252,9 @@ 
      *        If the value is null, option {@see Options::$allowSingletonsRebinding} will be used.
      * @throws SingletonOverloadException
      */
-    public function bindSingleton(string $alias, string|array|callable|object $resolver, ?bool $force = null): void
+    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);
         }
 
@@ -281,7 +281,7 @@ 
     {
         return ContainerScope::getContainer() === $this
             ? $this->invoker->invoke($target, $parameters)
-            : ContainerScope::runScope($this, fn(): mixed => $this->invoker->invoke($target, $parameters));
+            : ContainerScope::runScope($this, fn() : mixed => $this->invoker->invoke($target, $parameters));
     }
 
     /**
@@ -338,8 +338,8 @@ 
         ], $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 +353,7 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope)){
             $this->destruct();
             return;
         }
@@ -362,10 +362,10 @@ 
 
         // 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 +374,7 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== []){
             throw new FinalizersException($scopeName, $errors);
         }
     }
@@ -396,18 +396,18 @@ 
         $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/Config.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
         public readonly string $container = Container::class,
         public readonly string $binder = Binder::class,
         public readonly string $invoker = Invoker::class,
-    ) {
+    ){
         $this->scope = Scope::class;
         $this->scopedBindings = new Internal\Config\StateStorage();
         $this->actor = Actor::class;
@@ -75,9 +75,9 @@ 
      */
     public function lockRoot(): bool
     {
-        try {
+        try{
             return $this->rootLocked;
-        } finally {
+        }finally{
             $this->rootLocked = false;
         }
     }

src/Core/src/Config/Factory.php

Spacing +3 added lines, -3 removed lines

@@ -22,13 +22,13 @@ 
     public function __construct(
         callable $callable,
         public readonly bool $singleton = false,
-    ) {
+    ){
         $this->factory = $callable(...);
         $reflection = new \ReflectionFunction($this->factory);
         $this->parametersCount = $reflection->getNumberOfParameters();
 
         // Detect the return type of the factory
-        $returnType = (string) $reflection->getReturnType();
+        $returnType = (string)$reflection->getReturnType();
         $this->returnClass = \class_exists($returnType) ? $returnType : null;
 
         /** @psalm-suppress TypeDoesNotContainType */
@@ -39,7 +39,7 @@ 
                 \is_object($callable[0]) ? $callable[0]::class : $callable[0],
                 $callable[1],
             ),
-            \is_object($callable) && $callable::class !== \Closure::class => 'object ' . $callable::class,
+            \is_object($callable) && $callable::class !== \Closure::class => 'object '.$callable::class,
             default => null,
         };
     }