spiral / framework

src/Boot/tests/KernelTest.php

Spacing +10 added lines, -10 removed lines

@@ -47,7 +47,7 @@ 
     {
         $kernel = TestCore::create(['root' => __DIR__])->run();
 
-        $d = new class() implements DispatcherInterface {
+        $d = new class() implements DispatcherInterface{
             public static function canServe(EnvironmentInterface $env): bool
             {
                 return true;
@@ -67,7 +67,7 @@ 
     {
         $kernel = TestCore::create(['root' => __DIR__])->run();
 
-        $d = new class() implements DispatcherInterface {
+        $d = new class() implements DispatcherInterface{
             public function canServe(): bool
             {
                 return true;
@@ -90,7 +90,7 @@ 
     {
         $kernel = TestCore::create(['root' => __DIR__])->run();
 
-        $d = new class() implements DispatcherInterface {
+        $d = new class() implements DispatcherInterface{
             public static function canServe(EnvironmentInterface $env): bool
             {
                 return true;
@@ -124,19 +124,19 @@ 
     {
         $kernel = TestCore::create(['root' => __DIR__]);
 
-        $kernel->booting(static function (TestCore $core) {
+        $kernel->booting(static function (TestCore $core){
             $core->getContainer()->bind('abc', 'foo');
         });
 
-        $kernel->booting(static function (TestCore $core) {
+        $kernel->booting(static function (TestCore $core){
             $core->getContainer()->bind('bcd', 'foo');
         });
 
-        $kernel->booted( static function (TestCore $core) {
+        $kernel->booted(static function (TestCore $core){
             $core->getContainer()->bind('cde', 'foo');
         });
 
-        $kernel->booted( static function (TestCore $core) {
+        $kernel->booted(static function (TestCore $core){
             $core->getContainer()->bind('def', 'foo');
         });
 
@@ -159,7 +159,7 @@ 
 
     public function testEventsShouldBeDispatched(): void
     {
-        $testDispatcher = new class implements DispatcherInterface {
+        $testDispatcher = new class implements DispatcherInterface{
             public static function canServe(EnvironmentInterface $env): bool
             {
                 return true;
@@ -171,7 +171,7 @@ 
         };
 
         $container = new Container();
-        $kernel = TestCore::create(directories: ['root' => __DIR__,], container: $container)
+        $kernel = TestCore::create(directories: ['root' => __DIR__, ], container: $container)
             ->addDispatcher($testDispatcher);
 
         $dispatcher = $this->createMock(EventDispatcherInterface::class);
@@ -192,7 +192,7 @@ 
     public function testDispatcherNotFoundEventShouldBeDispatched(): void
     {
         $container = new Container();
-        $kernel = TestCore::create(directories: ['root' => __DIR__,], container: $container);
+        $kernel = TestCore::create(directories: ['root' => __DIR__, ], container: $container);
 
         $dispatcher = $this->createMock(EventDispatcherInterface::class);
         $dispatcher

src/Boot/src/AbstractKernel.php

Spacing +24 added lines, -24 removed lines

@@ -77,7 +77,7 @@ 
         protected readonly ExceptionHandlerInterface $exceptionHandler,
         protected readonly BootloadManagerInterface $bootloader,
         array $directories
-    ) {
+    ){
         $container->bindSingleton(ExceptionHandlerInterface::class, $exceptionHandler);
         $container->bindSingleton(ExceptionRendererInterface::class, $exceptionHandler);
         $container->bindSingleton(ExceptionReporterInterface::class, $exceptionHandler);
@@ -114,35 +114,35 @@ 
     final public static function create(
         array $directories,
         bool $handleErrors = true,
-        ExceptionHandlerInterface|string|null $exceptionHandler = null,
+        ExceptionHandlerInterface | string | null $exceptionHandler = null,
         Container $container = new Container(),
-        BootloadManagerInterface|Autowire|null $bootloadManager = null
+        BootloadManagerInterface | Autowire | null $bootloadManager = null
     ): static {
         $exceptionHandler ??= ExceptionHandler::class;
 
-        if (\is_string($exceptionHandler)) {
+        if (\is_string($exceptionHandler)){
             $exceptionHandler = $container->make($exceptionHandler);
         }
 
-        if ($handleErrors) {
+        if ($handleErrors){
             $exceptionHandler->register();
         }
 
-        if (!$container->has(InitializerInterface::class)) {
+        if (!$container->has(InitializerInterface::class)){
             $container->bind(InitializerInterface::class, Initializer::class);
         }
-        if (!$container->has(InvokerStrategyInterface::class)) {
+        if (!$container->has(InvokerStrategyInterface::class)){
             $container->bind(InvokerStrategyInterface::class, DefaultInvokerStrategy::class);
         }
 
-        if ($bootloadManager instanceof Autowire) {
+        if ($bootloadManager instanceof Autowire){
             $bootloadManager = $bootloadManager->resolve($container);
         }
         $bootloadManager ??= $container->make(StrategyBasedBootloadManager::class);
         \assert($bootloadManager instanceof BootloadManagerInterface);
         $container->bind(BootloadManagerInterface::class, $bootloadManager);
 
-        if (!$container->has(BootloaderRegistryInterface::class)) {
+        if (!$container->has(BootloaderRegistryInterface::class)){
             $container->bindSingleton(BootloaderRegistryInterface::class, [self::class, 'initBootloaderRegistry']);
         }
 
@@ -171,7 +171,7 @@ 
         $environment ??= new Environment();
         $this->container->bindSingleton(EnvironmentInterface::class, $environment);
 
-        try {
+        try{
             // will protect any against env overwrite action
             $this->container->runScope(
                 [EnvironmentInterface::class => $environment],
@@ -188,7 +188,7 @@ 
                     $this->fireCallbacks($this->bootstrappedCallbacks);
                 }
             );
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->exceptionHandler->handleGlobalException($e);
 
             return null;
@@ -209,7 +209,7 @@ 
      */
     public function running(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->runningCallbacks[] = $callback;
         }
     }
@@ -224,7 +224,7 @@ 
      */
     public function booting(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->bootingCallbacks[] = $callback;
         }
     }
@@ -239,7 +239,7 @@ 
      */
     public function booted(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->bootedCallbacks[] = $callback;
         }
     }
@@ -255,7 +255,7 @@ 
      */
     public function bootstrapped(Closure ...$callbacks): void
     {
-        foreach ($callbacks as $callback) {
+        foreach ($callbacks as $callback){
             $this->bootstrappedCallbacks[] = $callback;
         }
     }
@@ -267,9 +267,9 @@ 
      * @param class-string<DispatcherInterface>|DispatcherInterface $dispatcher The class name or instance
      * of the dispatcher. Since v4.0, it will only accept the class name.
      */
-    public function addDispatcher(string|DispatcherInterface $dispatcher): self
+    public function addDispatcher(string | DispatcherInterface $dispatcher): self
     {
-        if (\is_object($dispatcher)) {
+        if (\is_object($dispatcher)){
             $dispatcher = $dispatcher::class;
         }
 
@@ -291,19 +291,19 @@ 
         $eventDispatcher?->dispatch(new Serving());
 
         $serving = $servingScope = null;
-        foreach ($this->dispatchers as $dispatcher) {
+        foreach ($this->dispatchers as $dispatcher){
             $reflection = new \ReflectionClass($dispatcher);
 
             $scope = ($reflection->getAttributes(DispatcherScope::class)[0] ?? null)?->newInstance()->scope;
             $this->container->getBinder($scope)->bind($dispatcher, $dispatcher);
 
-            if ($serving === null && $this->canServe($reflection)) {
+            if ($serving === null && $this->canServe($reflection)){
                 $serving = $dispatcher;
                 $servingScope = $scope;
             }
         }
 
-        if ($serving === null) {
+        if ($serving === null){
             $eventDispatcher?->dispatch(new DispatcherNotFound());
             throw new BootException('Unable to locate active dispatcher.');
         }
@@ -352,13 +352,13 @@ 
      */
     protected function fireCallbacks(array &$callbacks): void
     {
-        if ($callbacks === []) {
+        if ($callbacks === []){
             return;
         }
 
-        do {
+        do{
             $this->container->invoke(\current($callbacks));
-        } while (\next($callbacks));
+        }while (\next($callbacks));
 
         $callbacks = [];
     }
@@ -398,7 +398,7 @@ 
      */
     private function canServe(\ReflectionClass $reflection): bool
     {
-        if (!$reflection->hasMethod('canServe')) {
+        if (!$reflection->hasMethod('canServe')){
             throw new BootException('Dispatcher must implement static `canServe` method.');
         }
 

src/Core/tests/Attribute/ScopeTest.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 final class ScopeTest extends TestCase
 {
     #[DataProvider('scopeNameDataProvider')]
-    public function testScope(string|\BackedEnum $name, string $expected): void
+    public function testScope(string | \BackedEnum $name, string $expected): void
     {
         $scope = new Scope($name);
 

src/Core/tests/Scope/UseCaseTest.php

Spacing +16 added lines, -16 removed lines

@@ -34,7 +34,7 @@ 
         $root = new Container();
         $root->bind('foo', SampleClass::class);
 
-        $root->runScoped(function (ContainerInterface $c1) {
+        $root->runScoped(function (ContainerInterface $c1){
             $c1->get('foo');
         }, bindings: ['foo' => SampleClass::class]);
 
@@ -321,7 +321,7 @@ 
 
         $root->runScope(
             new Scope('foo'),
-            function (ContainerInterface $c) {
+            function (ContainerInterface $c){
                 self::assertTrue($c->get('isFoo'));
                 $c->get('foo');
             }
@@ -334,7 +334,7 @@ 
         $root = new Container();
         $root->getBinder($scope)->bindSingleton('foo', SampleClass::class);
 
-        $root->runScope(new Scope($scope), function (Container $container) {
+        $root->runScope(new Scope($scope), function (Container $container){
             $this->assertTrue($container->has('foo'));
             $this->assertInstanceOf(SampleClass::class, $container->get('foo'));
         });
@@ -346,20 +346,20 @@ 
         $root = new Container();
         $root->bindSingleton('sampleClass', SampleClass::class);
 
-        $root->runScope(new Scope('foo'), function (Container $container) {
+        $root->runScope(new Scope('foo'), function (Container $container){
             $this->assertTrue($container->has('sampleClass'));
         });
 
-        $root->runScope(new Scope('foo'), function (Container $container) {
-            $container->runScope(new Scope('bar'), function (Container $container) {
+        $root->runScope(new Scope('foo'), function (Container $container){
+            $container->runScope(new Scope('bar'), function (Container $container){
                 $this->assertTrue($container->has('sampleClass'));
             });
         });
 
-        $root->runScope(new Scope('foo'), function (Container $container) {
+        $root->runScope(new Scope('foo'), function (Container $container){
             $container->bindSingleton('otherClass', SampleClass::class);
 
-            $container->runScope(new Scope('bar'), function (Container $container) {
+            $container->runScope(new Scope('bar'), function (Container $container){
                 $this->assertTrue($container->has('sampleClass'));
                 $this->assertTrue($container->has('otherClass'));
             });
@@ -371,33 +371,33 @@ 
         $root = new Container();
         $root->getBinder('foo')->bindSingleton('sampleClass', AttrScopeFoo::class);
 
-        $root->runScope(new Scope('foo'), function (Container $container) {
+        $root->runScope(new Scope('foo'), function (Container $container){
             $this->assertTrue($container->has('sampleClass'));
         });
 
-        $root->runScope(new Scope('bar'), function (Container $container) {
+        $root->runScope(new Scope('bar'), function (Container $container){
             $this->assertFalse($container->has('sampleClass'));
         });
 
-        $root->runScope(new Scope('foo'), function (Container $container) {
-            $container->runScope(new Scope('bar'), function (Container $container) {
+        $root->runScope(new Scope('foo'), function (Container $container){
+            $container->runScope(new Scope('bar'), function (Container $container){
                 $this->assertTrue($container->has('sampleClass'));
             });
         });
 
-        $root->runScope(new Scope('foo'), function (Container $container) {
+        $root->runScope(new Scope('foo'), function (Container $container){
             $container->bindSingleton('otherClass', AttrScopeFoo::class);
 
-            $container->runScope(new Scope('bar'), function (Container $container) {
+            $container->runScope(new Scope('bar'), function (Container $container){
                 $this->assertTrue($container->has('sampleClass'));
                 $this->assertTrue($container->has('otherClass'));
             });
         });
 
-        $root->runScope(new Scope('baz'), function (Container $container) {
+        $root->runScope(new Scope('baz'), function (Container $container){
             $container->getBinder('foo')->bindSingleton('otherClass', AttrScopeFoo::class);
 
-            $container->runScope(new Scope('bar'), function (Container $container) {
+            $container->runScope(new Scope('bar'), function (Container $container){
                 $this->assertFalse($container->has('sampleClass'));
                 $this->assertFalse($container->has('otherClass'));
             });

src/Core/src/Attribute/Scope.php

Spacing +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
 {
     public readonly string $name;
 
-    public function __construct(string|\BackedEnum $name)
+    public function __construct(string | \BackedEnum $name)
     {
-        $this->name = \is_object($name) ? (string) $name->value : $name;
+        $this->name = \is_object($name) ? (string)$name->value : $name;
     }
 }

src/Core/src/Scope.php

Spacing +2 added lines, -2 removed lines

@@ -20,9 +20,9 @@
      *        as the first argument. Otherwise, {@see InvokerInterface::invoke()} will be used to invoke the closure.
      */
     public function __construct(
-        public readonly string|\BackedEnum|null $name = null,
+        public readonly string | \BackedEnum | null $name = null,
         public readonly array $bindings = [],
         public readonly bool $autowire = true,
-    ) {
+    ){
     }
 }

src/Core/src/Container.php

Spacing +36 added lines, -36 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;
 
     /**
@@ -59,11 +59,11 @@ 
      */
     public function __construct(
         private Config $config = new Config(),
-        string|\BackedEnum|null $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
+        string | \BackedEnum | null $scopeName = self::DEFAULT_ROOT_SCOPE_NAME,
         private Options $options = new Options(),
-    ) {
-        if (\is_object($scopeName)) {
-            $scopeName = (string) $scopeName->value;
+    ){
+        if (\is_object($scopeName)){
+            $scopeName = (string)$scopeName->value;
         }
 
         $this->initServices($this, $scopeName);
@@ -117,7 +117,7 @@ 
      * @throws \Throwable
      * @psalm-suppress TooManyArguments
      */
-    public function make(string $alias, array $parameters = [], \Stringable|string|null $context = null): mixed
+    public function make(string $alias, array $parameters = [], \Stringable | string | null $context = null): mixed
     {
         return ContainerScope::getContainer() === $this
             ? $this->factory->make($alias, $parameters, $context)
@@ -143,7 +143,7 @@ 
      * @throws \Throwable
      * @psalm-suppress TooManyArguments
      */
-    public function get(string|Autowire $id, \Stringable|string|null $context = null): mixed
+    public function get(string | Autowire $id, \Stringable | string | null $context = null): mixed
     {
         return ContainerScope::getContainer() === $this
             ? $this->container->get($id, $context)
@@ -163,9 +163,9 @@ 
      *        If {@see string}, the default binder for the given scope will be returned. Default bindings won't affect
      *        already created Container instances except the case with the root one.
      */
-    public function getBinder(string|\BackedEnum|null $scope = null): BinderInterface
+    public function getBinder(string | \BackedEnum | null $scope = null): BinderInterface
     {
-        $scope = \is_object($scope) ? (string) $scope->value : $scope;
+        $scope = \is_object($scope) ? (string)$scope->value : $scope;
 
         return $scope === null
             ? $this->binder
@@ -175,25 +175,25 @@ 
     /**
      * @throws \Throwable
      */
-    public function runScope(Scope|array $bindings, callable $scope): mixed
+    public function runScope(Scope | array $bindings, callable $scope): mixed
     {
-        if (!\is_array($bindings)) {
+        if (!\is_array($bindings)){
             return $this->runIsolatedScope($bindings, $scope);
         }
 
         $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]);
             }
@@ -201,21 +201,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;
             }
         }
@@ -261,9 +261,9 @@ 
      * @param bool $force If the value is false, an exception will be thrown when attempting
      *  to bind an already constructed singleton.
      */
-    public function bindSingleton(string $alias, string|array|callable|object $resolver, bool $force = true): void
+    public function bindSingleton(string $alias, string | array | callable | object $resolver, bool $force = true): void
     {
-        if ($force) {
+        if ($force){
             $this->binder->removeBinding($alias);
         }
 
@@ -334,8 +334,8 @@ 
         ], $this->options);
 
         // 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);
             }
         }
@@ -349,7 +349,7 @@ 
     private function closeScope(): void
     {
         /** @psalm-suppress RedundantPropertyInitializationCheck */
-        if (!isset($this->scope)) {
+        if (!isset($this->scope)){
             $this->destruct();
             return;
         }
@@ -358,10 +358,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;
             }
         }
@@ -370,7 +370,7 @@ 
         $this->destruct();
 
         // Throw collected errors
-        if ($errors !== []) {
+        if ($errors !== []){
             throw new FinalizersException($scopeName, $errors);
         }
     }
@@ -392,18 +392,18 @@ 
         $container->scope->setParent($this, $this->scope, $this->factory);
 
         // Add specific bindings
-        foreach ($config->bindings as $alias => $resolver) {
+        foreach ($config->bindings as $alias => $resolver){
             $container->binder->bind($alias, $resolver);
         }
 
         return ContainerScope::runScope(
             $container,
             static function (self $container) use ($config, $closure): mixed {
-                try {
+                try{
                     return $config->autowire
                         ? $container->invoke($closure)
                         : $closure($container);
-                } finally {
+                }finally{
                     $container->closeScope();
                 }
             }

src/Framework/Attribute/DispatcherScope.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@
 final class DispatcherScope
 {
     public function __construct(
-        public readonly string|\BackedEnum $scope
-    ) {
+        public readonly string | \BackedEnum $scope
+    ){
     }
 }

src/Framework/Console/ConsoleDispatcher.php

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
         private readonly FinalizerInterface $finalizer,
         private readonly ContainerInterface $container,
         private readonly ExceptionHandlerInterface $errorHandler,
-    ) {
+    ){
     }
 
     public static function canServe(EnvironmentInterface $env): bool
@@ -51,13 +51,13 @@ 
         /** @var Console $console */
         $console = $this->container->get(Console::class);
 
-        try {
+        try{
             return $console->start($input ?? new ArgvInput(), $output);
-        } catch (Throwable $e) {
+        }catch (Throwable $e){
             $this->handleException($e, $output);
 
             return 255;
-        } finally {
+        }finally{
             $listener->disable();
             $this->finalizer->finalize(false);
         }