spiral / framework

src/Core/src/Config.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,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();
     }
@@ -65,9 +65,9 @@ 
 
     public function lockRoot(): bool
     {
-        try {
+        try{
             return $this->rootLocked;
-        } finally {
+        }finally{
             $this->rootLocked = false;
         }
     }

Braces +5 added lines, -2 removed lines

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

src/Core/src/Attribute/Finalize.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 {
     public function __construct(
         public string $method,
-    ) {
+    ){
     }
 }

src/Core/src/Attribute/Scope.php

Spacing +1 added lines, -1 removed lines

@@ -12,6 +12,6 @@
 {
     public function __construct(
         public string $name,
-    ) {
+    ){
     }
 }

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     public function __construct(
         protected ?string $scope,
         protected array $exceptions,
-    ) {
+    ){
         $count = \count($exceptions);
         parent::__construct(
             \sprintf(
@@ -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/Exception/Scope/NamedScopeDuplicationException.php

Spacing +1 added lines, -1 removed lines

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

src/Core/tests/Scope/UseCaseTest.php

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
         $root = new Container();
         $root->bind('foo', SampleClass::class);
 
-        $root->scope(function (ContainerInterface $c1) {
+        $root->scope(function (ContainerInterface $c1){
             $c1->get('foo');
         }, bindings: ['foo' => SampleClass::class]);
 
@@ -35,15 +35,15 @@ 
         $root = new Container();
         $root->bind('foo', SampleClass::class);
 
-        try {
+        try{
             $leak = $root->scope(function (ContainerInterface $c1): callable {
                 return fn() => $c1->get('foo');
             });
             $leak();
-        } catch (\AssertionError $e) {
+        }catch (\AssertionError $e){
             // Assertions are enabled
             self::assertStringContainsString("Scope Container shouldn't be leaked", $e->getMessage());
-        }catch (\Error $e) {
+        }catch (\Error $e){
             // Call to destroyed container
             self::assertStringContainsString('must not be accessed before initialization', $e->getMessage());
         }

Braces +20 added lines, -9 removed lines

@@ -20,7 +20,8 @@ 
         $root = new Container();
         $root->bind('foo', SampleClass::class);
 
-        $root->scope(function (ContainerInterface $c1) {
+        $root->scope(function (ContainerInterface $c1)
+        {
             $c1->get('foo');
         }, bindings: ['foo' => SampleClass::class]);
 
@@ -35,15 +36,20 @@ 
         $root = new Container();
         $root->bind('foo', SampleClass::class);
 
-        try {
+        try
+        {
             $leak = $root->scope(function (ContainerInterface $c1): callable {
                 return fn() => $c1->get('foo');
             });
             $leak();
-        } catch (\AssertionError $e) {
+        }
+        catch (\AssertionError $e)
+        {
             // Assertions are enabled
             self::assertStringContainsString("Scope Container shouldn't be leaked", $e->getMessage());
-        }catch (\Error $e) {
+        }
+        catch (\Error $e)
+        {
             // Call to destroyed container
             self::assertStringContainsString('must not be accessed before initialization', $e->getMessage());
         }
@@ -58,7 +64,8 @@ 
     {
         $root = new Container();
 
-        $root->scope(function (ContainerInterface $c1) use ($theSame, $alias) {
+        $root->scope(function (ContainerInterface $c1) use ($theSame, $alias)
+        {
             $obj1 = $c1->get($alias);
             $obj2 = $c1->get($alias);
 
@@ -82,14 +89,16 @@ 
     {
         $root = new Container();
 
-        $root->scope(function (ContainerInterface $c1) use ($root) {
+        $root->scope(function (ContainerInterface $c1) use ($root)
+        {
             $obj1 = $c1->get('foo');
             $this->weakMap->offsetSet($obj1, true);
 
             self::assertNotSame($root, $c1);
             self::assertInstanceOf(stdClass::class, $obj1);
 
-            $c1->scope(function (ContainerInterface $c2) use ($root, $c1, $obj1) {
+            $c1->scope(function (ContainerInterface $c2) use ($root, $c1, $obj1)
+            {
                 $obj2 = $c2->get('foo');
                 $this->weakMap->offsetSet($obj2, true);
 
@@ -118,14 +127,16 @@ 
         $root->bindSingleton('bar', [self::class, 'makeStdClass']);
         $root->bind(stdClass::class, new stdClass());
 
-        $root->scope(function (ContainerInterface $c1) use ($root) {
+        $root->scope(function (ContainerInterface $c1) use ($root)
+        {
             $obj1 = $c1->get('foo');
             $this->weakMap->offsetSet($obj1, true);
 
             self::assertInstanceOf(stdClass::class, $obj1);
             // Singleton must be the same
             self::assertSame($c1->get('bar'), $root->get('bar'));
-            $c1->scope(function (ContainerInterface $c2) use ($root, $obj1) {
+            $c1->scope(function (ContainerInterface $c2) use ($root, $obj1)
+            {
                 $obj2 = $c2->get('foo');
 
                 self::assertInstanceOf(stdClass::class, $obj2);

src/Core/tests/Scope/ContainerScopeFibersTest.php

Spacing +16 added lines, -16 removed lines

@@ -30,7 +30,7 @@ 
 
         self::runInFiber(
             self::functionScopedTestDataIterator(),
-            static function (mixed $suspendValue) {
+            static function (mixed $suspendValue){
                 self::assertNull(ContainerScope::getContainer());
                 self::assertTrue(\in_array($suspendValue, self::TEST_DATA, true));
             },
@@ -63,7 +63,7 @@ 
         );
 
         self::assertCount(5, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue){
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -81,7 +81,7 @@ 
         );
 
         self::assertCount(2, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue){
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -98,7 +98,7 @@ 
         ?callable $load = null,
         ?Container $container = null,
     ): callable {
-        return static function () use ($load, $container): array {
+        return static function () use ($load, $container) : array {
             // The function should be called in a fiber
             self::assertNotNull(Fiber::getCurrent());
 
@@ -119,7 +119,7 @@ 
                         $resource = $c3->get('resource');
                         self::assertInstanceOf(DateTimeImmutable::class, $c3->get('bar'));
                         self::assertInstanceOf(stdClass::class, $resource);
-                        foreach (self::TEST_DATA as $key => $value) {
+                        foreach (self::TEST_DATA as $key => $value){
                             $resource->$key = $value;
                             $load === null or $load();
                             Fiber::suspend($value);
@@ -132,8 +132,8 @@ 
             }, ['foo' => new DateTime()]);
             self::assertFalse($c1->has('foo'));
 
-            self::assertSame(self::TEST_DATA, (array) $result);
-            return (array) $result;
+            self::assertSame(self::TEST_DATA, (array)$result);
+            return (array)$result;
         };
     }
 
@@ -150,8 +150,8 @@ 
     {
         $fiber = new Fiber($callable);
         $value = $fiber->start();
-        while (!$fiber->isTerminated()) {
-            if ($check !== null) {
+        while (!$fiber->isTerminated()){
+            if ($check !== null){
                 $check($value);
             }
             $value = $fiber->resume($value);
@@ -173,14 +173,14 @@ 
         /** Values that were suspended by the fiber. */
         $suspends = [];
         $results = [];
-        foreach ($callables as $key => $callable) {
+        foreach ($callables as $key => $callable){
             $fiberGenerators[$key] = (static function () use ($callable) {
                 $fiber = new Fiber($callable);
                 // Get ready
                 yield null;
 
                 $value = yield $fiber->start();
-                while (!$fiber->isTerminated()) {
+                while (!$fiber->isTerminated()){
                     $value = yield $fiber->resume($value);
                 }
                 return $fiber->getReturn();
@@ -190,15 +190,15 @@ 
         }
 
 
-        while ($fiberGenerators !== []) {
-            foreach ($fiberGenerators as $key => $generator) {
-                try {
+        while ($fiberGenerators !== []){
+            foreach ($fiberGenerators as $key => $generator){
+                try{
                     $suspends[$key] = $generator->send($suspends[$key]);
-                    if (!$generator->valid()) {
+                    if (!$generator->valid()){
                         $results[$key] = $generator->getReturn();
                         unset($fiberGenerators[$key]);
                     }
-                } catch (\Throwable $e) {
+                }catch (\Throwable $e){
                     unset($fiberGenerators[$key]);
                     $results[$key] = $e;
                 }

Braces +33 added lines, -16 removed lines

@@ -30,7 +30,8 @@ 
 
         self::runInFiber(
             self::functionScopedTestDataIterator(),
-            static function (mixed $suspendValue) {
+            static function (mixed $suspendValue)
+            {
                 self::assertNull(ContainerScope::getContainer());
                 self::assertTrue(\in_array($suspendValue, self::TEST_DATA, true));
             },
@@ -63,7 +64,8 @@ 
         );
 
         self::assertCount(5, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue)
+        {
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -81,7 +83,8 @@ 
         );
 
         self::assertCount(2, $result);
-        foreach ($result as $suspendValue) {
+        foreach ($result as $suspendValue)
+        {
             self::assertSame(self::TEST_DATA, $suspendValue);
         }
     }
@@ -106,20 +109,23 @@ 
             $c1 = $container ?? new Container();
             $c1->bindSingleton('resource', new stdClass());
 
-            $result = $c1->scope(static function (Container $c2) use ($load) {
+            $result = $c1->scope(static function (Container $c2) use ($load)
+            {
                 // check local binding
                 self::assertTrue($c2->has('foo'));
                 self::assertInstanceOf(DateTime::class, $c2->get('foo'));
 
                 return $c2->scope(
-                    static function (ContainerInterface $c3) use ($load) {
+                    static function (ContainerInterface $c3) use ($load)
+                    {
                         // check local binding
                         self::assertTrue($c3->has('bar'));
 
                         $resource = $c3->get('resource');
                         self::assertInstanceOf(DateTimeImmutable::class, $c3->get('bar'));
                         self::assertInstanceOf(stdClass::class, $resource);
-                        foreach (self::TEST_DATA as $key => $value) {
+                        foreach (self::TEST_DATA as $key => $value)
+                        {
                             $resource->$key = $value;
                             $load === null or $load();
                             Fiber::suspend($value);
@@ -150,8 +156,10 @@ 
     {
         $fiber = new Fiber($callable);
         $value = $fiber->start();
-        while (!$fiber->isTerminated()) {
-            if ($check !== null) {
+        while (!$fiber->isTerminated())
+        {
+            if ($check !== null)
+            {
                 $check($value);
             }
             $value = $fiber->resume($value);
@@ -173,14 +181,17 @@ 
         /** Values that were suspended by the fiber. */
         $suspends = [];
         $results = [];
-        foreach ($callables as $key => $callable) {
-            $fiberGenerators[$key] = (static function () use ($callable) {
+        foreach ($callables as $key => $callable)
+        {
+            $fiberGenerators[$key] = (static function () use ($callable)
+            {
                 $fiber = new Fiber($callable);
                 // Get ready
                 yield null;
 
                 $value = yield $fiber->start();
-                while (!$fiber->isTerminated()) {
+                while (!$fiber->isTerminated())
+                {
                     $value = yield $fiber->resume($value);
                 }
                 return $fiber->getReturn();
@@ -190,15 +201,21 @@ 
         }
 
 
-        while ($fiberGenerators !== []) {
-            foreach ($fiberGenerators as $key => $generator) {
-                try {
+        while ($fiberGenerators !== [])
+        {
+            foreach ($fiberGenerators as $key => $generator)
+            {
+                try
+                {
                     $suspends[$key] = $generator->send($suspends[$key]);
-                    if (!$generator->valid()) {
+                    if (!$generator->valid())
+                    {
                         $results[$key] = $generator->getReturn();
                         unset($fiberGenerators[$key]);
                     }
-                } catch (\Throwable $e) {
+                }
+                catch (\Throwable $e)
+                {
                     unset($fiberGenerators[$key]);
                     $results[$key] = $e;
                 }

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

Spacing +1 added lines, -1 removed lines

@@ -10,6 +10,6 @@
 {
     public function __construct(
         public DateTimeInterface $logger,
-    ) {
+    ){
     }
 }