spiral / framework

src/Core/tests/Internal/Resolver/CommonCasesTest.php

Spacing +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
         $this->expectExceptionMessage('Enum `Spiral\Tests\Core\Stub\EnumObject` can not be constructed.');
 
         $this->resolveClosure(
-            static function (EnumObject $enum) {},
+            static function (EnumObject $enum){},
         );
     }
 
@@ -75,7 +75,7 @@ 
         );
 
         $this->resolveClosure(
-            static function (TestTrait $enum) {},
+            static function (TestTrait $enum){},
         );
     }
 }

Braces +6 added lines, -2 removed lines

@@ -63,7 +63,9 @@ 
         $this->expectExceptionMessage('Enum `Spiral\Tests\Core\Stub\EnumObject` can not be constructed.');
 
         $this->resolveClosure(
-            static function (EnumObject $enum) {},
+            static function (EnumObject $enum)
+            {
+},
         );
     }
 
@@ -75,7 +77,9 @@ 
         );
 
         $this->resolveClosure(
-            static function (TestTrait $enum) {},
+            static function (TestTrait $enum)
+            {
+},
         );
     }
 }

src/Core/src/Config.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         public readonly string $binder = Binder::class,
         public readonly string $invoker = Invoker::class,
         public readonly string $tracer = Tracer::class,
-    ) {
+    ){
     }
 
     public function getIterator(): Traversable

src/Core/src/Exception/Resolver/InvalidArgumentException.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     public function __construct(
         \ReflectionFunctionAbstract $reflection,
         private readonly string $parameter
-    ) {
+    ){
         $pattern = "Invalid argument value type for the `$parameter` parameter when validating arguments for `%s`.";
         parent::__construct($this->renderFunctionAndParameter($reflection, $pattern));
     }

src/Core/src/Internal/Trace.php

Spacing +3 added lines, -3 removed lines

@@ -18,13 +18,13 @@ 
     public function __construct(
         public readonly string $alias,
         public array $info,
-    ) {
+    ){
     }
 
     public function __toString(): string
     {
         $info = [$this->alias];
-        foreach ($this->info as $key => $item) {
+        foreach ($this->info as $key => $item){
             $info[] = "$key: {$this->stringifyValue($item)}";
         }
         return \implode("\n", $info);
@@ -36,7 +36,7 @@ 
             \is_string($item) => "'$item'",
             \is_scalar($item) => (string)$item,
             $item instanceof Closure => $this->renderClosureSignature(new ReflectionFunction($item)),
-            \is_object($item) => 'instance of ' . $item::class,
+            \is_object($item) => 'instance of '.$item::class,
             default => \gettype($item),
         };
     }

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
     public function __toString(): string
     {
         $info = [$this->alias];
-        foreach ($this->info as $key => $item) {
+        foreach ($this->info as $key => $item)
+        {
             $info[] = "$key: {$this->stringifyValue($item)}";
         }
         return \implode("\n", $info);

src/Core/src/Internal/Tracer.php

Spacing +9 added lines, -9 removed lines

@@ -18,7 +18,7 @@ 
 
     public function __toString(): string
     {
-        return $this->traces === [] ? '' : "Container trace list:\n" . $this->renderTraceList($this->traces);
+        return $this->traces === [] ? '' : "Container trace list:\n".$this->renderTraceList($this->traces);
     }
 
     /**
@@ -34,26 +34,26 @@ 
     public function push(string $alias, bool $nextLevel = false, mixed ...$details): void
     {
         $trace = new Trace($alias, $details);
-        if ($nextLevel || $this->traces === []) {
+        if ($nextLevel || $this->traces === []){
             $this->traces[] = [$trace];
-        } else {
+        }else{
             $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;
         }
         $key = \array_key_last($this->traces);
         $list = &$this->traces[$key];
         \array_pop($list);
-        if ($list === []) {
+        if ($list === []){
             unset($this->traces[$key]);
         }
     }
@@ -70,7 +70,7 @@ 
     {
         $result = [];
         $i = 0;
-        foreach ($blocks as $block) {
+        foreach ($blocks as $block){
             \array_push($result, ...$this->blockToStringList($block, $i++));
         }
         return \implode("\n", $result);
@@ -90,8 +90,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;
     }

Braces +15 added lines, -7 removed lines

@@ -34,26 +34,32 @@ 
     public function push(string $alias, bool $nextLevel = false, mixed ...$details): void
     {
         $trace = new Trace($alias, $details);
-        if ($nextLevel || $this->traces === []) {
+        if ($nextLevel || $this->traces === [])
+        {
             $this->traces[] = [$trace];
-        } else {
+        }
+        else
+        {
             $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;
         }
         $key = \array_key_last($this->traces);
         $list = &$this->traces[$key];
         \array_pop($list);
-        if ($list === []) {
+        if ($list === [])
+        {
             unset($this->traces[$key]);
         }
     }
@@ -70,7 +76,8 @@ 
     {
         $result = [];
         $i = 0;
-        foreach ($blocks as $block) {
+        foreach ($blocks as $block)
+        {
             \array_push($result, ...$this->blockToStringList($block, $i++));
         }
         return \implode("\n", $result);
@@ -90,7 +97,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/tests/ExceptionsTest.php

Spacing +2 added lines, -2 removed lines

@@ -143,9 +143,9 @@
     {
         $container = new Container();
 
-        try {
+        try{
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             $this->assertSame(
                 $e->getMessage(),
                 <<<MARKDOWN

Braces +5 added lines, -2 removed lines

@@ -143,9 +143,12 @@
     {
         $container = new Container();
 
-        try {
+        try
+        {
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             $this->assertSame(
                 $e->getMessage(),
                 <<<MARKDOWN

src/Core/src/Internal/Factory.php

Spacing +34 added lines, -34 removed lines

@@ -54,12 +54,12 @@ 
      */
     public function make(string $alias, array $parameters = [], string $context = null): mixed
     {
-        if (!isset($this->state->bindings[$alias])) {
+        if (!isset($this->state->bindings[$alias])){
             $this->tracer->push($alias, false, source: 'autowiring', context: $context);
-            try {
+            try{
                 //No direct instructions how to construct class, make is automatically
                 return $this->autowire($alias, $parameters, $context);
-            } finally {
+            }finally{
                 $this->tracer->pop(false);
             }
         }
@@ -67,20 +67,20 @@ 
         $binding = $this->state->bindings[$alias];
         $this->tracer->push($alias, false, source: 'binding', binding: $binding, context: $context);
 
-        if (\is_object($binding)) {
-            if ($binding::class === WeakReference::class) {
-                if ($binding->get() === null && \class_exists($alias)) {
-                    try {
+        if (\is_object($binding)){
+            if ($binding::class === WeakReference::class){
+                if ($binding->get() === null && \class_exists($alias)){
+                    try{
                         $this->tracer->push($alias, false, source: WeakReference::class, context: $context);
                         $object = $this->createInstance($alias, $parameters, $context);
                         $binding = $this->state->bindings[$alias] = WeakReference::create($object);
-                    } catch (\Throwable) {
+                    }catch (\Throwable){
                         throw new ContainerException($this->tracer->combineTraceMessage(\sprintf(
                             'Can\'t resolve `%s`: can\'t instantiate `%s` from WeakReference binding.',
                             $this->tracer->getRootAlias(),
                             $alias,
                         )));
-                    } finally {
+                    }finally{
                         $this->tracer->pop();
                     }
                 }
@@ -90,22 +90,22 @@ 
             return $binding;
         }
 
-        if (\is_string($binding)) {
+        if (\is_string($binding)){
             //Binding is pointing to something else
             return $this->make($binding, $parameters, $context);
         }
 
         unset($this->state->bindings[$alias]);
-        try {
+        try{
             $instance = $binding[0] === $alias
                 ? $this->autowire($alias, $parameters, $context)
                 : $this->evaluateBinding($alias, $binding[0], $parameters, $context);
-        } finally {
+        }finally{
             /** @psalm-var class-string $alias */
             $this->state->bindings[$alias] = $binding;
         }
 
-        if ($binding[1]) {
+        if ($binding[1]){
             // Indicates singleton
             /** @psalm-var class-string $alias */
             $this->state->bindings[$alias] = $instance;
@@ -124,7 +124,7 @@ 
      */
     private function autowire(string $class, array $parameters, string $context = null): object
     {
-        if (!\class_exists($class) && !isset($this->state->injectors[$class])) {
+        if (!\class_exists($class) && !isset($this->state->injectors[$class])){
             throw new NotFoundException($this->tracer->combineTraceMessage(\sprintf(
                 'Can\'t resolve `%s`: undefined class or binding `%s`.',
                 $this->tracer->getRootAlias(),
@@ -151,18 +151,18 @@ 
         array $parameters,
         string $context = null
     ): mixed {
-        if (\is_string($target)) {
+        if (\is_string($target)){
             // Reference
             return $this->make($target, $parameters, $context);
         }
 
-        if ($target instanceof Autowire) {
+        if ($target instanceof Autowire){
             return $target->resolve($this, $parameters);
         }
 
-        try {
+        try{
             return $this->invoker->invoke($target, $parameters);
-        } catch (NotCallableException $e) {
+        }catch (NotCallableException $e){
             throw new ContainerException(
                 $this->tracer->combineTraceMessage(\sprintf('Invalid binding for `%s`.', $alias)),
                 $e->getCode(),
@@ -186,20 +186,20 @@ 
      */
     private function createInstance(string $class, array $parameters, string $context = null): object
     {
-        try {
+        try{
             $reflection = new \ReflectionClass($class);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
         //We have to construct class using external injector when we know exact context
-        if ($parameters === [] && $this->binder->hasInjector($class)) {
+        if ($parameters === [] && $this->binder->hasInjector($class)){
             $injector = $this->state->injectors[$reflection->getName()];
 
-            try {
+            try{
                 $injectorInstance = $this->container->get($injector);
 
-                if (!$injectorInstance instanceof InjectorInterface) {
+                if (!$injectorInstance instanceof InjectorInterface){
                     throw new InjectionException(
                         \sprintf(
                             "Class '%s' must be an instance of InjectorInterface for '%s'.",
@@ -211,7 +211,7 @@ 
 
                 /** @var InjectorInterface<TObject> $injectorInstance */
                 $instance = $injectorInstance->createInjection($reflection, $context);
-                if (!$reflection->isInstance($instance)) {
+                if (!$reflection->isInstance($instance)){
                     throw new InjectionException(
                         \sprintf(
                             "Invalid injection response for '%s'.",
@@ -221,12 +221,12 @@ 
                 }
 
                 return $instance;
-            } finally {
+            }finally{
                 $this->state->injectors[$reflection->getName()] = $injector;
             }
         }
 
-        if (!$reflection->isInstantiable()) {
+        if (!$reflection->isInstantiable()){
             $itIs = match (true) {
                 $reflection->isEnum() => 'Enum',
                 $reflection->isAbstract() => 'Abstract class',
@@ -239,10 +239,10 @@ 
 
         $constructor = $reflection->getConstructor();
 
-        if ($constructor !== null) {
-            try {
+        if ($constructor !== null){
+            try{
                 $arguments = $this->resolver->resolveArguments($constructor, $parameters);
-            } catch (ValidationException $e) {
+            }catch (ValidationException $e){
                 throw new ContainerException(
                     $this->tracer->combineTraceMessage(
                         \sprintf(
@@ -253,13 +253,13 @@ 
                     ),
                 );
             }
-            try {
+            try{
                 // Using constructor with resolved arguments
                 $instance = new $class(...$arguments);
-            } catch (\TypeError $e) {
+            }catch (\TypeError $e){
                 throw new WrongTypeException($constructor, $e);
             }
-        } else {
+        }else{
             // No constructor specified
             $instance = $reflection->newInstance();
         }
@@ -277,9 +277,9 @@ 
     private function registerInstance(object $instance, array $parameters): object
     {
         //Declarative singletons (only when class received via direct get)
-        if ($parameters === [] && $instance instanceof SingletonInterface) {
+        if ($parameters === [] && $instance instanceof SingletonInterface){
             $alias = $instance::class;
-            if (!isset($this->state->bindings[$alias])) {
+            if (!isset($this->state->bindings[$alias])){
                 $this->state->bindings[$alias] = $instance;
             }
         }

Braces +78 added lines, -34 removed lines

@@ -54,12 +54,16 @@ 
      */
     public function make(string $alias, array $parameters = [], string $context = null): mixed
     {
-        if (!isset($this->state->bindings[$alias])) {
+        if (!isset($this->state->bindings[$alias]))
+        {
             $this->tracer->push($alias, false, source: 'autowiring', context: $context);
-            try {
+            try
+            {
                 //No direct instructions how to construct class, make is automatically
                 return $this->autowire($alias, $parameters, $context);
-            } finally {
+            }
+            finally
+            {
                 $this->tracer->pop(false);
             }
         }
@@ -67,20 +71,28 @@ 
         $binding = $this->state->bindings[$alias];
         $this->tracer->push($alias, false, source: 'binding', binding: $binding, context: $context);
 
-        if (\is_object($binding)) {
-            if ($binding::class === WeakReference::class) {
-                if ($binding->get() === null && \class_exists($alias)) {
-                    try {
+        if (\is_object($binding))
+        {
+            if ($binding::class === WeakReference::class)
+            {
+                if ($binding->get() === null && \class_exists($alias))
+                {
+                    try
+                    {
                         $this->tracer->push($alias, false, source: WeakReference::class, context: $context);
                         $object = $this->createInstance($alias, $parameters, $context);
                         $binding = $this->state->bindings[$alias] = WeakReference::create($object);
-                    } catch (\Throwable) {
+                    }
+                    catch (\Throwable)
+                    {
                         throw new ContainerException($this->tracer->combineTraceMessage(\sprintf(
                             'Can\'t resolve `%s`: can\'t instantiate `%s` from WeakReference binding.',
                             $this->tracer->getRootAlias(),
                             $alias,
                         )));
-                    } finally {
+                    }
+                    finally
+                    {
                         $this->tracer->pop();
                     }
                 }
@@ -90,22 +102,27 @@ 
             return $binding;
         }
 
-        if (\is_string($binding)) {
+        if (\is_string($binding))
+        {
             //Binding is pointing to something else
             return $this->make($binding, $parameters, $context);
         }
 
         unset($this->state->bindings[$alias]);
-        try {
+        try
+        {
             $instance = $binding[0] === $alias
                 ? $this->autowire($alias, $parameters, $context)
                 : $this->evaluateBinding($alias, $binding[0], $parameters, $context);
-        } finally {
+        }
+        finally
+        {
             /** @psalm-var class-string $alias */
             $this->state->bindings[$alias] = $binding;
         }
 
-        if ($binding[1]) {
+        if ($binding[1])
+        {
             // Indicates singleton
             /** @psalm-var class-string $alias */
             $this->state->bindings[$alias] = $instance;
@@ -124,7 +141,8 @@ 
      */
     private function autowire(string $class, array $parameters, string $context = null): object
     {
-        if (!\class_exists($class) && !isset($this->state->injectors[$class])) {
+        if (!\class_exists($class) && !isset($this->state->injectors[$class]))
+        {
             throw new NotFoundException($this->tracer->combineTraceMessage(\sprintf(
                 'Can\'t resolve `%s`: undefined class or binding `%s`.',
                 $this->tracer->getRootAlias(),
@@ -151,18 +169,23 @@ 
         array $parameters,
         string $context = null
     ): mixed {
-        if (\is_string($target)) {
+        if (\is_string($target))
+        {
             // Reference
             return $this->make($target, $parameters, $context);
         }
 
-        if ($target instanceof Autowire) {
+        if ($target instanceof Autowire)
+        {
             return $target->resolve($this, $parameters);
         }
 
-        try {
+        try
+        {
             return $this->invoker->invoke($target, $parameters);
-        } catch (NotCallableException $e) {
+        }
+        catch (NotCallableException $e)
+        {
             throw new ContainerException(
                 $this->tracer->combineTraceMessage(\sprintf('Invalid binding for `%s`.', $alias)),
                 $e->getCode(),
@@ -186,20 +209,26 @@ 
      */
     private function createInstance(string $class, array $parameters, string $context = null): object
     {
-        try {
+        try
+        {
             $reflection = new \ReflectionClass($class);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
         //We have to construct class using external injector when we know exact context
-        if ($parameters === [] && $this->binder->hasInjector($class)) {
+        if ($parameters === [] && $this->binder->hasInjector($class))
+        {
             $injector = $this->state->injectors[$reflection->getName()];
 
-            try {
+            try
+            {
                 $injectorInstance = $this->container->get($injector);
 
-                if (!$injectorInstance instanceof InjectorInterface) {
+                if (!$injectorInstance instanceof InjectorInterface)
+                {
                     throw new InjectionException(
                         \sprintf(
                             "Class '%s' must be an instance of InjectorInterface for '%s'.",
@@ -211,7 +240,8 @@ 
 
                 /** @var InjectorInterface<TObject> $injectorInstance */
                 $instance = $injectorInstance->createInjection($reflection, $context);
-                if (!$reflection->isInstance($instance)) {
+                if (!$reflection->isInstance($instance))
+                {
                     throw new InjectionException(
                         \sprintf(
                             "Invalid injection response for '%s'.",
@@ -221,12 +251,15 @@ 
                 }
 
                 return $instance;
-            } finally {
+            }
+            finally
+            {
                 $this->state->injectors[$reflection->getName()] = $injector;
             }
         }
 
-        if (!$reflection->isInstantiable()) {
+        if (!$reflection->isInstantiable())
+        {
             $itIs = match (true) {
                 $reflection->isEnum() => 'Enum',
                 $reflection->isAbstract() => 'Abstract class',
@@ -239,10 +272,14 @@ 
 
         $constructor = $reflection->getConstructor();
 
-        if ($constructor !== null) {
-            try {
+        if ($constructor !== null)
+        {
+            try
+            {
                 $arguments = $this->resolver->resolveArguments($constructor, $parameters);
-            } catch (ValidationException $e) {
+            }
+            catch (ValidationException $e)
+            {
                 throw new ContainerException(
                     $this->tracer->combineTraceMessage(
                         \sprintf(
@@ -253,13 +290,18 @@ 
                     ),
                 );
             }
-            try {
+            try
+            {
                 // Using constructor with resolved arguments
                 $instance = new $class(...$arguments);
-            } catch (\TypeError $e) {
+            }
+            catch (\TypeError $e)
+            {
                 throw new WrongTypeException($constructor, $e);
             }
-        } else {
+        }
+        else
+        {
             // No constructor specified
             $instance = $reflection->newInstance();
         }
@@ -277,9 +319,11 @@ 
     private function registerInstance(object $instance, array $parameters): object
     {
         //Declarative singletons (only when class received via direct get)
-        if ($parameters === [] && $instance instanceof SingletonInterface) {
+        if ($parameters === [] && $instance instanceof SingletonInterface)
+        {
             $alias = $instance::class;
-            if (!isset($this->state->bindings[$alias])) {
+            if (!isset($this->state->bindings[$alias]))
+            {
                 $this->state->bindings[$alias] = $instance;
             }
         }