spiral / framework

src/Core/src/Internal/Scope.php

Spacing +4 added lines, -4 removed lines

@@ -16,7 +16,7 @@ 
 
     public function __construct(
         private readonly ?string $scopeName = null,
-    ) {
+    ){
     }
 
     public function getScopeName(): ?string
@@ -35,9 +35,9 @@ 
         $this->parentScope = $parentScope;
 
         // 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);
             }
@@ -60,7 +60,7 @@ 
         $result = [$this->scopeName];
 
         $parent = $this;
-        while ($parent->parentScope !== null) {
+        while ($parent->parentScope !== null){
             $parent = $parent->parentScope;
             $result[] = $parent->scopeName;
         }

Braces +6 added lines, -3 removed lines

@@ -35,9 +35,11 @@ 
         $this->parentScope = $parentScope;
 
         // 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);
             }
@@ -60,7 +62,8 @@ 
         $result = [$this->scopeName];
 
         $parent = $this;
-        while ($parent->parentScope !== null) {
+        while ($parent->parentScope !== null)
+        {
             $parent = $parent->parentScope;
             $result[] = $parent->scopeName;
         }

src/Core/src/Exception/Scope/NamedScopeDuplicationException.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 {
     public function __construct(
         string $scope,
-    ) {
+    ){
         parent::__construct(
             $scope,
             "Error on a scope allocation with the name `{$scope}`. A scope with the same name already exists.",

src/Core/src/Exception/Scope/BadScopeException.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
     public function __construct(
         string $scope,
         protected string $className,
-    ) {
+    ){
         parent::__construct(
             $scope,
             \sprintf('Class `%s` can be resolved only in `%s` scope.', $className, $scope),

src/Core/src/Exception/Scope/ScopeException.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         string $message = '',
         int $code = 0,
         ?\Throwable $previous = null,
-    ) {
+    ){
         parent::__construct(
             $message,
             $code,

src/Core/src/Exception/Scope/FinalizersException.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     public function __construct(
         ?string $scope,
         protected array $exceptions,
-    ) {
+    ){
         $count = \count($exceptions);
         parent::__construct(
             $scope,
@@ -24,7 +24,7 @@ 
                 $count === 1 ? 'An exception has been' : "$count exceptions have been",
                 $scope === null ? 'an unnamed scope' : "the scope `$scope`",
                 \implode("\n\n", \array_map(
-                    static fn (Exception $e): string => \sprintf(
+                    static fn (Exception $e) : string => \sprintf(
                         "# %s\n%s",
                         $e::class,
                         $e->getMessage(),

src/Core/src/Config.php

Spacing +3 added lines, -3 removed lines

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

Braces +5 added lines, -2 removed lines

@@ -74,9 +74,12 @@
      */
     public function lockRoot(): bool
     {
-        try {
+        try
+        {
             return $this->rootLocked;
-        } finally {
+        }
+        finally
+        {
             $this->rootLocked = false;
         }
     }

src/Core/src/Exception/Scope/ScopeContainerLeakedException.php

Spacing +2 added lines, -2 removed lines

@@ -13,9 +13,9 @@
     public function __construct(
         ?string $scope,
         array $parents,
-    ) {
+    ){
         $scopes = \implode('->', \array_map(
-            static fn (?string $scope): string => $scope === null ? 'null' : "\"$scope\"",
+            static fn (?string $scope) : string => $scope === null ? 'null' : "\"$scope\"",
             [...\array_reverse($parents), $scope],
         ));
         parent::__construct(

src/Core/tests/Scope/Stub/AttrFinalize.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     {
         $this->logger = $logger;
         $this->finalized = true;
-        if ($this->throwException) {
+        if ($this->throwException){
             throw new RuntimeException('Test exception from finalize method.');
         }
     }

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
     {
         $this->logger = $logger;
         $this->finalized = true;
-        if ($this->throwException) {
+        if ($this->throwException)
+        {
             throw new RuntimeException('Test exception from finalize method.');
         }
     }

src/Core/src/Container.php

Spacing +29 added lines, -29 removed lines

@@ -47,11 +47,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;
 
     /**
@@ -60,7 +60,7 @@ 
     public function __construct(
         private Config $config = new Config(),
         ?string $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
-    ) {
+    ){
         $this->initServices($this, $scopeName);
 
         // Bind himself
@@ -130,7 +130,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
     {
         return $this->container->get($id, $context);
     }
@@ -158,26 +158,26 @@ 
     {
         $binds = &$this->state->bindings;
         $cleanup = $previous = [];
-        foreach ($bindings as $alias => $resolver) {
-            if (isset($binds[$alias])) {
+        foreach ($bindings as $alias => $resolver){
+            if (isset($binds[$alias])){
                 $previous[$alias] = $binds[$alias];
-            } else {
+            }else{
                 $cleanup[] = $alias;
             }
 
             $this->binder->bind($alias, $resolver);
         }
 
-        try {
+        try{
             return ContainerScope::getContainer() !== $this
                 ? ContainerScope::runScope($this, $scope)
                 : $scope($this);
-        } finally {
-            foreach ($previous as $alias => $resolver) {
+        }finally{
+            foreach ($previous as $alias => $resolver){
                 $binds[$alias] = $resolver;
             }
 
-            foreach ($cleanup as $alias) {
+            foreach ($cleanup as $alias){
                 unset($binds[$alias]);
             }
         }
@@ -202,32 +202,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());
             }
         }
@@ -238,7 +238,7 @@ 
      * for each method call), function array or Closure (executed every call). Only object resolvers
      * supported by this method.
      */
-    public function bind(string $alias, string|array|callable|object $resolver): void
+    public function bind(string $alias, string | array | callable | object $resolver): void
     {
         $this->binder->bind($alias, $resolver);
     }
@@ -249,7 +249,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);
     }
@@ -319,8 +319,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);
             }
         }
@@ -334,7 +334,7 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope)){
             $this->destruct();
             return;
         }
@@ -343,10 +343,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;
             }
         }
@@ -355,7 +355,7 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== []){
             throw new FinalizersException($scopeName, $errors);
         }
     }

Braces +45 added lines, -20 removed lines

@@ -158,26 +158,35 @@ 
     {
         $binds = &$this->state->bindings;
         $cleanup = $previous = [];
-        foreach ($bindings as $alias => $resolver) {
-            if (isset($binds[$alias])) {
+        foreach ($bindings as $alias => $resolver)
+        {
+            if (isset($binds[$alias]))
+            {
                 $previous[$alias] = $binds[$alias];
-            } else {
+            }
+            else
+            {
                 $cleanup[] = $alias;
             }
 
             $this->binder->bind($alias, $resolver);
         }
 
-        try {
+        try
+        {
             return ContainerScope::getContainer() !== $this
                 ? ContainerScope::runScope($this, $scope)
                 : $scope($this);
-        } finally {
-            foreach ($previous as $alias => $resolver) {
+        }
+        finally
+        {
+            foreach ($previous as $alias => $resolver)
+            {
                 $binds[$alias] = $resolver;
             }
 
-            foreach ($cleanup as $alias) {
+            foreach ($cleanup as $alias)
+            {
                 unset($binds[$alias]);
             }
         }
@@ -202,32 +211,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());
             }
         }
@@ -319,8 +336,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);
             }
         }
@@ -334,7 +353,8 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope))
+        {
             $this->destruct();
             return;
         }
@@ -343,10 +363,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;
             }
         }
@@ -355,7 +379,8 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== [])
+        {
             throw new FinalizersException($scopeName, $errors);
         }
     }