spiral / framework

src/Core/src/Config/WeakReference.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function __construct(
         public \WeakReference $reference,
-    ) {
+    ){
     }
 
     public function __toString(): string

src/Core/src/Config/Shared.php

Spacing +2 added lines, -2 removed lines

@@ -11,11 +11,11 @@
 {
     public function __construct(
         public readonly object $value,
-    ) {
+    ){
     }
 
     public function __toString(): string
     {
-        return 'Shared object of class ' . $this->value::class;
+        return 'Shared object of class '.$this->value::class;
     }
 }

src/Core/src/Config/Alias.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     public function __construct(
         public readonly string $alias,
         public readonly bool $singleton = false,
-    ) {
+    ){
     }
 
     public function __toString(): string

src/Core/src/Config/DeferredFactory.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function __construct(
         public readonly array $factory,
         public readonly bool $singleton = false,
-    ) {
+    ){
     }
 
     public function __toString(): string

src/Core/src/Config/Autowire.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     public function __construct(
         public readonly AutowireAlias $autowire,
         public readonly bool $singleton = false,
-    ) {
+    ){
     }
 
     public function __toString(): string

src/Core/src/Container.php

Spacing +30 added lines, -30 removed lines

@@ -49,11 +49,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;
 
     /**
@@ -62,7 +62,7 @@ 
     public function __construct(
         private Config $config = new Config(),
         ?string $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
-    ) {
+    ){
         $this->initServices($this, $scopeName);
 
         /** @psalm-suppress RedundantPropertyInitializationCheck */
@@ -140,7 +140,7 @@ 
      *
      * @psalm-suppress InvalidArgument, InvalidCast
      */
-    public function get(string|Autowire $id, string $context = null): mixed
+    public function get(string | Autowire $id, string $context = null): mixed
     {
         /** @psalm-suppress TooManyArguments */
         return $this->container->get($id, $context);
@@ -168,16 +168,16 @@ 
         $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]);
             }
@@ -185,21 +185,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;
             }
         }
@@ -213,32 +213,32 @@ 
         // Open scope
         $container = new self($this->config, $name);
 
-        try {
+        try{
             // Configure scope
             $container->scope->setParent($this, $this->scope);
 
             // Add specific bindings
-            foreach ($bindings as $alias => $resolver) {
+            foreach ($bindings as $alias => $resolver){
                 $container->binder->bind($alias, $resolver);
             }
 
             return ContainerScope::runScope(
                 $container,
                 static function (self $container) use ($autowire, $closure): mixed {
-                    try {
+                    try{
                         return $autowire
                             ? $container->invoke($closure)
                             : $closure($container);
-                    } finally {
+                    }finally{
                         $container->closeScope();
                     }
                 }
             );
-        } finally {
+        }finally{
             // Check the container has not been leaked
             $link = \WeakReference::create($container);
             unset($container);
-            if ($link->get() !== null) {
+            if ($link->get() !== null){
                 throw new ScopeContainerLeakedException($name, $this->scope->getParentScopeNames());
             }
         }
@@ -270,7 +270,7 @@ 
      *
      * @psalm-param TResolver $resolver
      */
-    public function bindSingleton(string $alias, string|array|callable|object $resolver): void
+    public function bindSingleton(string $alias, string | array | callable | object $resolver): void
     {
         $this->binder->bindSingleton($alias, $resolver);
     }
@@ -340,8 +340,8 @@ 
         ]);
 
         // 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);
             }
         }
@@ -355,7 +355,7 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope)){
             $this->destruct();
             return;
         }
@@ -364,10 +364,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;
             }
         }
@@ -376,7 +376,7 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== []){
             throw new FinalizersException($scopeName, $errors);
         }
     }

Braces +49 added lines, -22 removed lines

@@ -168,16 +168,21 @@ 
         $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]);
             }
@@ -185,21 +190,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;
             }
         }
@@ -213,32 +224,40 @@ 
         // Open scope
         $container = new self($this->config, $name);
 
-        try {
+        try
+        {
             // Configure scope
             $container->scope->setParent($this, $this->scope);
 
             // Add specific bindings
-            foreach ($bindings as $alias => $resolver) {
+            foreach ($bindings as $alias => $resolver)
+            {
                 $container->binder->bind($alias, $resolver);
             }
 
             return ContainerScope::runScope(
                 $container,
                 static function (self $container) use ($autowire, $closure): mixed {
-                    try {
+                    try
+                    {
                         return $autowire
                             ? $container->invoke($closure)
                             : $closure($container);
-                    } finally {
+                    }
+                    finally
+                    {
                         $container->closeScope();
                     }
                 }
             );
-        } finally {
+        }
+        finally
+        {
             // Check the container has not been leaked
             $link = \WeakReference::create($container);
             unset($container);
-            if ($link->get() !== null) {
+            if ($link->get() !== null)
+            {
                 throw new ScopeContainerLeakedException($name, $this->scope->getParentScopeNames());
             }
         }
@@ -340,8 +359,10 @@ 
         ]);
 
         // 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);
             }
         }
@@ -355,7 +376,8 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope))
+        {
             $this->destruct();
             return;
         }
@@ -364,10 +386,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;
             }
         }
@@ -376,7 +402,8 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== [])
+        {
             throw new FinalizersException($scopeName, $errors);
         }
     }

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

Spacing +18 added lines, -18 removed lines

@@ -31,7 +31,7 @@ 
 {
     public function __construct(
         protected readonly State $state,
-    ) {
+    ){
     }
 
     /**
@@ -39,14 +39,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);
         }
 
@@ -58,9 +58,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);
         }
 
@@ -72,9 +72,9 @@ 
         $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');
             }
 
@@ -99,7 +99,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]);
@@ -107,20 +107,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);
 
@@ -128,12 +128,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;
@@ -160,15 +160,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.');
         }
 

Braces +33 added lines, -15 removed lines

@@ -39,14 +39,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);
         }
 
@@ -58,9 +62,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);
         }
 
@@ -72,9 +79,11 @@ 
         $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');
             }
 
@@ -99,7 +108,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]);
@@ -107,13 +117,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;
         }
 
@@ -128,7 +142,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))
                 ||
@@ -160,15 +175,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.');
         }
 

src/Core/src/Config/Factory.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
     public function __construct(
         callable $callable,
         public readonly bool $singleton = false,
-    ) {
+    ){
         $this->factory = $callable(...);
         $this->parametersCount = (new \ReflectionFunction($this->factory))->getNumberOfParameters();
         /** @psalm-suppress TypeDoesNotContainType */
@@ -29,7 +29,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,
         };
     }