spiral / framework

src/Core/src/Internal/Config/StateBinder.php

Spacing +20 added lines, -20 removed lines

@@ -32,7 +32,7 @@ 
 {
     public function __construct(
         protected readonly State $state,
-    ) {
+    ){
     }
 
     /**
@@ -40,14 +40,14 @@ 
      */
     public function bind(string $alias, mixed $resolver): void
     {
-        if ($resolver instanceof Inflector && (\interface_exists($alias) || \class_exists($alias))) {
+        if ($resolver instanceof Inflector && (\interface_exists($alias) || \class_exists($alias))){
             $this->state->inflectors[$alias][] = $resolver;
             return;
         }
 
-        try {
+        try{
             $config = $this->makeConfig($resolver, false);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw $this->invalidBindingException($alias, $e);
         }
 
@@ -59,9 +59,9 @@ 
      */
     public function bindSingleton(string $alias, mixed $resolver): void
     {
-        try {
+        try{
             $config = $this->makeConfig($resolver, true);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw $this->invalidBindingException($alias, $e);
         }
 
@@ -73,13 +73,13 @@ 
         $bindings = &$this->state->bindings;
 
         $flags = [];
-        while ($binding = $bindings[$alias] ?? null and $binding::class === Alias::class) {
+        while ($binding = $bindings[$alias] ?? null and $binding::class === Alias::class){
             //Checking alias tree
-            if ($flags[$binding->alias] ?? false) {
+            if ($flags[$binding->alias] ?? false){
                 return $binding->alias === $alias ?: throw new Exception('Circular alias detected');
             }
 
-            if (\array_key_exists($alias, $this->state->singletons)) {
+            if (\array_key_exists($alias, $this->state->singletons)){
                 return true;
             }
 
@@ -104,7 +104,7 @@ 
     public function removeInjector(string $class): void
     {
         unset($this->state->injectors[$class]);
-        if (!isset($this->state->bindings[$class]) || $this->state->bindings[$class]::class !== Injectable::class) {
+        if (!isset($this->state->bindings[$class]) || $this->state->bindings[$class]::class !== Injectable::class){
             return;
         }
         unset($this->state->bindings[$class]);
@@ -112,20 +112,20 @@ 
 
     public function hasInjector(string $class): bool
     {
-        try {
+        try{
             $reflection = new \ReflectionClass($class);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (\array_key_exists($class, $this->state->injectors)) {
+        if (\array_key_exists($class, $this->state->injectors)){
             return true;
         }
 
         if (
             $reflection->implementsInterface(InjectableInterface::class)
             && $reflection->hasConstant('INJECTOR')
-        ) {
+        ){
             $const = $reflection->getConstant('INJECTOR');
             $this->bindInjector($class, $const);
 
@@ -133,12 +133,12 @@ 
         }
 
         // check interfaces
-        foreach ($this->state->injectors as $target => $injector) {
+        foreach ($this->state->injectors as $target => $injector){
             if (
                 (\class_exists($target, true) && $reflection->isSubclassOf($target))
                 ||
                 (\interface_exists($target, true) && $reflection->implementsInterface($target))
-            ) {
+            ){
                 $this->state->bindings[$class] = new Injectable($injector);
 
                 return true;
@@ -165,15 +165,15 @@ 
 
     private function makeConfigFromArray(array $resolver, bool $singleton): Binding
     {
-        if (\is_callable($resolver)) {
+        if (\is_callable($resolver)){
             return new Factory($resolver, $singleton);
         }
 
         // Validate lazy invokable array
-        if (!isset($resolver[0]) || !isset($resolver[1]) || !\is_string($resolver[1]) || $resolver[1] === '') {
+        if (!isset($resolver[0]) || !isset($resolver[1]) || !\is_string($resolver[1]) || $resolver[1] === ''){
             throw new InvalidArgumentException('Incompatible array declaration for resolver.');
         }
-        if ((!\is_string($resolver[0]) && !\is_object($resolver[0])) || $resolver[0] === '') {
+        if ((!\is_string($resolver[0]) && !\is_object($resolver[0])) || $resolver[0] === ''){
             throw new InvalidArgumentException('Incompatible array declaration for resolver.');
         }
 
@@ -191,7 +191,7 @@ 
 
     private function setBinding(string $alias, Binding $config): void
     {
-        if (isset($this->state->singletons[$alias])) {
+        if (isset($this->state->singletons[$alias])){
             throw new SingletonOverloadException($alias);
         }
 

Braces +37 added lines, -17 removed lines

@@ -40,14 +40,18 @@ 
      */
     public function bind(string $alias, mixed $resolver): void
     {
-        if ($resolver instanceof Inflector && (\interface_exists($alias) || \class_exists($alias))) {
+        if ($resolver instanceof Inflector && (\interface_exists($alias) || \class_exists($alias)))
+        {
             $this->state->inflectors[$alias][] = $resolver;
             return;
         }
 
-        try {
+        try
+        {
             $config = $this->makeConfig($resolver, false);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw $this->invalidBindingException($alias, $e);
         }
 
@@ -59,9 +63,12 @@ 
      */
     public function bindSingleton(string $alias, mixed $resolver): void
     {
-        try {
+        try
+        {
             $config = $this->makeConfig($resolver, true);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw $this->invalidBindingException($alias, $e);
         }
 
@@ -73,13 +80,16 @@ 
         $bindings = &$this->state->bindings;
 
         $flags = [];
-        while ($binding = $bindings[$alias] ?? null and $binding::class === Alias::class) {
+        while ($binding = $bindings[$alias] ?? null and $binding::class === Alias::class)
+        {
             //Checking alias tree
-            if ($flags[$binding->alias] ?? false) {
+            if ($flags[$binding->alias] ?? false)
+            {
                 return $binding->alias === $alias ?: throw new Exception('Circular alias detected');
             }
 
-            if (\array_key_exists($alias, $this->state->singletons)) {
+            if (\array_key_exists($alias, $this->state->singletons))
+            {
                 return true;
             }
 
@@ -104,7 +114,8 @@ 
     public function removeInjector(string $class): void
     {
         unset($this->state->injectors[$class]);
-        if (!isset($this->state->bindings[$class]) || $this->state->bindings[$class]::class !== Injectable::class) {
+        if (!isset($this->state->bindings[$class]) || $this->state->bindings[$class]::class !== Injectable::class)
+        {
             return;
         }
         unset($this->state->bindings[$class]);
@@ -112,13 +123,17 @@ 
 
     public function hasInjector(string $class): bool
     {
-        try {
+        try
+        {
             $reflection = new \ReflectionClass($class);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new ContainerException($e->getMessage(), $e->getCode(), $e);
         }
 
-        if (\array_key_exists($class, $this->state->injectors)) {
+        if (\array_key_exists($class, $this->state->injectors))
+        {
             return true;
         }
 
@@ -133,7 +148,8 @@ 
         }
 
         // check interfaces
-        foreach ($this->state->injectors as $target => $injector) {
+        foreach ($this->state->injectors as $target => $injector)
+        {
             if (
                 (\class_exists($target, true) && $reflection->isSubclassOf($target))
                 ||
@@ -165,15 +181,18 @@ 
 
     private function makeConfigFromArray(array $resolver, bool $singleton): Binding
     {
-        if (\is_callable($resolver)) {
+        if (\is_callable($resolver))
+        {
             return new Factory($resolver, $singleton);
         }
 
         // Validate lazy invokable array
-        if (!isset($resolver[0]) || !isset($resolver[1]) || !\is_string($resolver[1]) || $resolver[1] === '') {
+        if (!isset($resolver[0]) || !isset($resolver[1]) || !\is_string($resolver[1]) || $resolver[1] === '')
+        {
             throw new InvalidArgumentException('Incompatible array declaration for resolver.');
         }
-        if ((!\is_string($resolver[0]) && !\is_object($resolver[0])) || $resolver[0] === '') {
+        if ((!\is_string($resolver[0]) && !\is_object($resolver[0])) || $resolver[0] === '')
+        {
             throw new InvalidArgumentException('Incompatible array declaration for resolver.');
         }
 
@@ -191,7 +210,8 @@ 
 
     private function setBinding(string $alias, Binding $config): void
     {
-        if (isset($this->state->singletons[$alias])) {
+        if (isset($this->state->singletons[$alias]))
+        {
             throw new SingletonOverloadException($alias);
         }
 

src/Core/tests/Internal/Config/StateBinderTest.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $binder = $this->constructor->get('binder', BinderInterface::class);
         $factory = $this->constructor->get('factory', FactoryInterface::class);
 
-        $this->bindSingleton('test', new class implements SingletonInterface {});
+        $this->bindSingleton('test', new class implements SingletonInterface{});
         $factory->make('test');
 
         $this->assertTrue($binder->hasInstance('test'));

Braces +3 added lines, -1 removed lines

@@ -46,7 +46,9 @@
         $binder = $this->constructor->get('binder', BinderInterface::class);
         $factory = $this->constructor->get('factory', FactoryInterface::class);
 
-        $this->bindSingleton('test', new class implements SingletonInterface {});
+        $this->bindSingleton('test', new class implements SingletonInterface
+        {
+});
         $factory->make('test');
 
         $this->assertTrue($binder->hasInstance('test'));