spiral / framework

src/Hmvc/src/Core/AbstractCore.php

Spacing +6 added lines, -6 removed lines

@@ -28,7 +28,7 @@ 
     public function __construct(
         /** @internal */
         protected ContainerInterface $container
-    ) {
+    ){
         // resolver is usually the container itself
         /** @psalm-suppress MixedAssignment */
         $this->resolver = $container->get(ResolverInterface::class);
@@ -59,13 +59,13 @@ 
 
     protected function resolveArguments(\ReflectionMethod $method, array $parameters): array
     {
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter){
             $name = $parameter->getName();
             if (
                 \array_key_exists($name, $parameters) &&
                 $parameters[$name] === null &&
                 $parameter->isDefaultValueAvailable()
-            ) {
+            ){
                 /** @psalm-suppress MixedAssignment */
                 $parameters[$name] = $parameter->getDefaultValue();
             }
@@ -80,9 +80,9 @@ 
      */
     private function invoke(\ReflectionMethod $method, array $arguments): mixed
     {
-        try {
+        try{
             $args = $this->resolveArguments($method, $arguments);
-        } catch (ArgumentResolvingException | InvalidArgumentException $e) {
+        }catch (ArgumentResolvingException | InvalidArgumentException $e){
             throw new ControllerException(
                 \sprintf(
                     'Missing/invalid parameter %s of `%s`->`%s`',
@@ -93,7 +93,7 @@ 
                 ControllerException::BAD_ARGUMENT,
                 $e,
             );
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

Braces +10 added lines, -4 removed lines

@@ -59,7 +59,8 @@ 
 
     protected function resolveArguments(\ReflectionMethod $method, array $parameters): array
     {
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter)
+        {
             $name = $parameter->getName();
             if (
                 \array_key_exists($name, $parameters) &&
@@ -80,9 +81,12 @@ 
      */
     private function invoke(\ReflectionMethod $method, array $arguments): mixed
     {
-        try {
+        try
+        {
             $args = $this->resolveArguments($method, $arguments);
-        } catch (ArgumentResolvingException | InvalidArgumentException $e) {
+        }
+        catch (ArgumentResolvingException | InvalidArgumentException $e)
+        {
             throw new ControllerException(
                 \sprintf(
                     'Missing/invalid parameter %s of `%s`->`%s`',
@@ -93,7 +97,9 @@ 
                 ControllerException::BAD_ARGUMENT,
                 $e,
             );
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

src/Router/src/CoreHandler.php

Spacing +23 added lines, -23 removed lines

@@ -44,11 +44,11 @@ 
     private bool $isLegacyPipeline;
 
     public function __construct(
-        private readonly HandlerInterface|CoreInterface $core,
+        private readonly HandlerInterface | CoreInterface $core,
         private readonly ScopeInterface $scope,
         private readonly ResponseFactoryInterface $responseFactory,
         ?TracerInterface $tracer = null
-    ) {
+    ){
         $this->tracer = $tracer ?? new NullTracer($scope);
         $this->isLegacyPipeline = !$core instanceof HandlerInterface;
     }
@@ -96,9 +96,9 @@ 
         $output = '';
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try{
             $action = $this->verbActions
-                ? \strtolower($request->getMethod()) . \ucfirst($this->action)
+                ? \strtolower($request->getMethod()).\ucfirst($this->action)
                 : $this->action;
 
             // run the core withing PSR-7 Request/Response scope
@@ -108,42 +108,42 @@ 
                     Response::class => $response,
                 ],
                 fn (): mixed => $this->tracer->trace(
-                    name: 'Controller [' . $controller . ':' . $action . ']',
+                    name: 'Controller ['.$controller.':'.$action.']',
                     callback: $this->isLegacyPipeline
-                        ? fn (): mixed => $this->core->callAction(
-                            controller: $controller,
-                            action: $action,
-                            parameters: $parameters,
+                        ? fn () : mixed => $this->core->callAction(
+                            controller : $controller,
+                            action : $action,
+                            parameters : $parameters,
                         )
-                        : fn (): mixed => $this->core->handle(
+                        : fn () : mixed => $this->core->handle(
                             new CallContext(
                                 Target::fromPair($controller, $action),
                                 $parameters,
                             ),
                         ),
-                    attributes: [
+                    attributes : [
                         'route.controller' => $this->controller,
                         'route.action' => $action,
                         'route.parameters' => \array_keys($parameters),
                     ]
                 )
             );
-        } catch (TargetCallException $e) {
+        }catch (TargetCallException $e){
             \ob_get_clean();
             throw $this->mapException($e);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             \ob_get_clean();
             throw $e;
-        } finally {
-            while (\ob_get_level() > $outputLevel + 1) {
-                $output = \ob_get_clean() . $output;
+        }finally{
+            while (\ob_get_level() > $outputLevel + 1){
+                $output = \ob_get_clean().$output;
             }
         }
 
         return $this->wrapResponse(
             $response,
             $result,
-            \ob_get_clean() . $output,
+            \ob_get_clean().$output,
         );
     }
 
@@ -156,21 +156,21 @@ 
      */
     private function wrapResponse(Response $response, mixed $result = null, string $output = ''): Response
     {
-        if ($result instanceof Response) {
-            if ($output !== '' && $result->getBody()->isWritable()) {
+        if ($result instanceof Response){
+            if ($output !== '' && $result->getBody()->isWritable()){
                 $result->getBody()->write($output);
             }
 
             return $result;
         }
 
-        if ($result instanceof \Generator) {
+        if ($result instanceof \Generator){
             return $response->withBody(new GeneratorStream($result));
         }
 
-        if (\is_array($result) || $result instanceof \JsonSerializable) {
+        if (\is_array($result) || $result instanceof \JsonSerializable){
             $response = $this->writeJson($response, $result);
-        } else {
+        }else{
             $response->getBody()->write((string)$result);
         }
 
@@ -203,7 +203,7 @@ 
      */
     private function checkValues(): void
     {
-        if ($this->controller === null) {
+        if ($this->controller === null){
             throw new HandlerException('Controller and action pair are not set.');
         }
     }

Braces +26 added lines, -11 removed lines

@@ -96,7 +96,8 @@ 
         $output = '';
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try
+        {
             $action = $this->verbActions
                 ? \strtolower($request->getMethod()) . \ucfirst($this->action)
                 : $this->action;
@@ -128,14 +129,21 @@ 
                     ]
                 )
             );
-        } catch (TargetCallException $e) {
+        }
+        catch (TargetCallException $e)
+        {
             \ob_get_clean();
             throw $this->mapException($e);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             \ob_get_clean();
             throw $e;
-        } finally {
-            while (\ob_get_level() > $outputLevel + 1) {
+        }
+        finally
+        {
+            while (\ob_get_level() > $outputLevel + 1)
+            {
                 $output = \ob_get_clean() . $output;
             }
         }
@@ -156,21 +164,27 @@ 
      */
     private function wrapResponse(Response $response, mixed $result = null, string $output = ''): Response
     {
-        if ($result instanceof Response) {
-            if ($output !== '' && $result->getBody()->isWritable()) {
+        if ($result instanceof Response)
+        {
+            if ($output !== '' && $result->getBody()->isWritable())
+            {
                 $result->getBody()->write($output);
             }
 
             return $result;
         }
 
-        if ($result instanceof \Generator) {
+        if ($result instanceof \Generator)
+        {
             return $response->withBody(new GeneratorStream($result));
         }
 
-        if (\is_array($result) || $result instanceof \JsonSerializable) {
+        if (\is_array($result) || $result instanceof \JsonSerializable)
+        {
             $response = $this->writeJson($response, $result);
-        } else {
+        }
+        else
+        {
             $response->getBody()->write((string)$result);
         }
 
@@ -203,7 +217,8 @@ 
      */
     private function checkValues(): void
     {
-        if ($this->controller === null) {
+        if ($this->controller === null)
+        {
             throw new HandlerException('Controller and action pair are not set.');
         }
     }

src/Router/src/Loader/Configurator/RouteConfigurator.php

Spacing +11 added lines, -11 removed lines

@@ -22,7 +22,7 @@ 
     private ?string $group = null;
     private ?array $methods = null;
     private string $prefix = '';
-    private HandlerInterface|CoreInterface|null $core = null;
+    private HandlerInterface | CoreInterface | null $core = null;
     private ?array $middleware = null;
 
     /** @var null|string|callable|RequestHandlerInterface|TargetInterface */
@@ -36,12 +36,12 @@ 
         private readonly string $name,
         private readonly string $pattern,
         private readonly RouteCollection $collection
-    ) {
+    ){
     }
 
     public function __destruct()
     {
-        if ($this->target === null) {
+        if ($this->target === null){
             throw new TargetException(
                 \sprintf('The [%s] route has no defined target. Call one of: `controller`, `action`,
                     `namespaced`, `groupControllers`, `callable`, `handler` methods.', $this->name)
@@ -92,21 +92,21 @@ 
         return $this;
     }
 
-    public function action(string $controller, string|array $action, int $options = 0): self
+    public function action(string $controller, string | array $action, int $options = 0): self
     {
         $this->target = new Action($controller, $action, $options);
 
         return $this;
     }
 
-    public function callable(array|\Closure $callable): self
+    public function callable(array | \Closure $callable): self
     {
         $this->target = $callable;
 
         return $this;
     }
 
-    public function handler(TargetInterface|string $target): self
+    public function handler(TargetInterface | string $target): self
     {
         $this->target = $target;
 
@@ -134,16 +134,16 @@ 
         return $this;
     }
 
-    public function core(HandlerInterface|CoreInterface $core): self
+    public function core(HandlerInterface | CoreInterface $core): self
     {
         $this->core = $core;
 
         return $this;
     }
 
-    public function middleware(MiddlewareInterface|string|array $middleware): self
+    public function middleware(MiddlewareInterface | string | array $middleware): self
     {
-        if (!\is_array($middleware)) {
+        if (!\is_array($middleware)){
             $middleware = [$middleware];
         }
 
@@ -152,9 +152,9 @@ 
         return $this;
     }
 
-    public function methods(string|array $methods): self
+    public function methods(string | array $methods): self
     {
-        $this->methods = (array) $methods;
+        $this->methods = (array)$methods;
 
         return $this;
     }

src/Router/src/Loader/Configurator/ImportConfigurator.php

Spacing +12 added lines, -12 removed lines

@@ -14,7 +14,7 @@ 
     public function __construct(
         private readonly RouteCollection $parent,
         private readonly RouteCollection $routes
-    ) {
+    ){
     }
 
     public function __destruct()
@@ -24,17 +24,17 @@ 
 
     public function __sleep(): array
     {
-        throw new \BadMethodCallException('Cannot unserialize ' . self::class);
+        throw new \BadMethodCallException('Cannot unserialize '.self::class);
     }
 
     public function __wakeup()
     {
-        throw new \BadMethodCallException('Cannot unserialize ' . self::class);
+        throw new \BadMethodCallException('Cannot unserialize '.self::class);
     }
 
     public function defaults(array $defaults): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator){
             $configurator->defaults($defaults);
         }
 
@@ -46,7 +46,7 @@ 
      */
     public function group(string $group): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator){
             $configurator->group($group);
         }
 
@@ -58,7 +58,7 @@ 
      */
     public function prefix(string $prefix): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator){
             $configurator->prefix($prefix);
         }
 
@@ -70,26 +70,26 @@ 
      */
     public function namePrefix(string $prefix): self
     {
-        foreach ($this->routes->all() as $name => $configurator) {
-            $this->routes->add($prefix . $name, $configurator);
+        foreach ($this->routes->all() as $name => $configurator){
+            $this->routes->add($prefix.$name, $configurator);
             $this->routes->remove($name);
         }
 
         return $this;
     }
 
-    public function core(HandlerInterface|CoreInterface $core): self
+    public function core(HandlerInterface | CoreInterface $core): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator){
             $configurator->core($core);
         }
 
         return $this;
     }
 
-    public function middleware(MiddlewareInterface|string|array $middleware): self
+    public function middleware(MiddlewareInterface | string | array $middleware): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator){
             $configurator->middleware($middleware);
         }
 

Braces +12 added lines, -6 removed lines

@@ -34,7 +34,8 @@ 
 
     public function defaults(array $defaults): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator)
+        {
             $configurator->defaults($defaults);
         }
 
@@ -46,7 +47,8 @@ 
      */
     public function group(string $group): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator)
+        {
             $configurator->group($group);
         }
 
@@ -58,7 +60,8 @@ 
      */
     public function prefix(string $prefix): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator)
+        {
             $configurator->prefix($prefix);
         }
 
@@ -70,7 +73,8 @@ 
      */
     public function namePrefix(string $prefix): self
     {
-        foreach ($this->routes->all() as $name => $configurator) {
+        foreach ($this->routes->all() as $name => $configurator)
+        {
             $this->routes->add($prefix . $name, $configurator);
             $this->routes->remove($name);
         }
@@ -80,7 +84,8 @@ 
 
     public function core(HandlerInterface|CoreInterface $core): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator)
+        {
             $configurator->core($core);
         }
 
@@ -89,7 +94,8 @@ 
 
     public function middleware(MiddlewareInterface|string|array $middleware): self
     {
-        foreach ($this->routes->all() as $configurator) {
+        foreach ($this->routes->all() as $configurator)
+        {
             $configurator->middleware($middleware);
         }
 

src/Router/src/RouteGroup.php

Spacing +11 added lines, -11 removed lines

@@ -28,7 +28,7 @@ 
     /** @var array<MiddlewareType> */
     private array $middleware = [];
 
-    private Autowire|HandlerInterface|CoreInterface|string|null $core = null;
+    private Autowire | HandlerInterface | CoreInterface | string | null $core = null;
 
     public function __construct(
         /** @deprecated since v3.3.0 */
@@ -37,7 +37,7 @@ 
         private readonly ?RouterInterface $router = null,
         /** @deprecated since v3.3.0 */
         private readonly ?UriHandler $handler = null
-    ) {
+    ){
     }
 
     /**
@@ -68,7 +68,7 @@ 
         return $this;
     }
 
-    public function setCore(Autowire|CoreInterface|HandlerInterface|string $core): self
+    public function setCore(Autowire | CoreInterface | HandlerInterface | string $core): self
     {
         $this->core = $core;
 
@@ -78,7 +78,7 @@ 
     /**
      * @param MiddlewareType $middleware
      */
-    public function addMiddleware(MiddlewareInterface|Autowire|string $middleware): self
+    public function addMiddleware(MiddlewareInterface | Autowire | string $middleware): self
     {
         $this->middleware[] = $middleware;
 
@@ -92,21 +92,21 @@ 
      */
     public function register(RouterInterface $router, FactoryInterface $factory): void
     {
-        foreach ($this->routes as $name => $route) {
-            if ($this->core !== null) {
-                if (!$this->core instanceof CoreInterface && !$this->core instanceof HandlerInterface) {
+        foreach ($this->routes as $name => $route){
+            if ($this->core !== null){
+                if (!$this->core instanceof CoreInterface && !$this->core instanceof HandlerInterface){
                     $this->core = $factory->make($this->core);
                 }
 
                 $target = $route->getTarget();
-                if ($target instanceof AbstractTarget) {
+                if ($target instanceof AbstractTarget){
                     $route = $route->withTarget($target->withCore($this->core));
                 }
             }
 
-            try {
+            try{
                 $uriHandler = $route->getUriHandler();
-            } catch (\Throwable) {
+            }catch (\Throwable){
                 $uriHandler = $factory->make(UriHandler::class);
             }
 
@@ -130,7 +130,7 @@ 
     {
         \assert($route instanceof Route);
 
-        $this->routes[$this->namePrefix . $name] = $route;
+        $this->routes[$this->namePrefix.$name] = $route;
 
         return $this;
     }

Braces +13 added lines, -6 removed lines

@@ -92,21 +92,28 @@
      */
     public function register(RouterInterface $router, FactoryInterface $factory): void
     {
-        foreach ($this->routes as $name => $route) {
-            if ($this->core !== null) {
-                if (!$this->core instanceof CoreInterface && !$this->core instanceof HandlerInterface) {
+        foreach ($this->routes as $name => $route)
+        {
+            if ($this->core !== null)
+            {
+                if (!$this->core instanceof CoreInterface && !$this->core instanceof HandlerInterface)
+                {
                     $this->core = $factory->make($this->core);
                 }
 
                 $target = $route->getTarget();
-                if ($target instanceof AbstractTarget) {
+                if ($target instanceof AbstractTarget)
+                {
                     $route = $route->withTarget($target->withCore($this->core));
                 }
             }
 
-            try {
+            try
+            {
                 $uriHandler = $route->getUriHandler();
-            } catch (\Throwable) {
+            }
+            catch (\Throwable)
+            {
                 $uriHandler = $factory->make(UriHandler::class);
             }
 

src/Router/src/Target/AbstractTarget.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
     // Automatically prepend HTTP verb to all action names.
     public const RESTFUL = 1;
 
-    private HandlerInterface|CoreInterface|null $pipeline = null;
+    private HandlerInterface | CoreInterface | null $pipeline = null;
     private ?CoreHandler $handler = null;
     private bool $verbActions;
 
@@ -34,7 +34,7 @@ 
         private array $constrains,
         int $options = 0,
         private string $defaultAction = 'index'
-    ) {
+    ){
         $this->verbActions = ($options & self::RESTFUL) === self::RESTFUL;
     }
 
@@ -52,7 +52,7 @@ 
      * @mutation-free
      * @deprecated Use {@see withHandler()} instead.
      */
-    public function withCore(HandlerInterface|CoreInterface $core): TargetInterface
+    public function withCore(HandlerInterface | CoreInterface $core): TargetInterface
     {
         $target = clone $this;
         $target->pipeline = $core;
@@ -84,11 +84,11 @@ 
 
     protected function coreHandler(ContainerInterface $container): CoreHandler
     {
-        if ($this->handler !== null) {
+        if ($this->handler !== null){
             return $this->handler;
         }
 
-        try {
+        try{
             // construct on demand
             $this->handler = new CoreHandler(
                 $this->pipeline ?? $container->get(HandlerInterface::class),
@@ -98,7 +98,7 @@ 
             );
 
             return $this->handler;
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new TargetException($e->getMessage(), $e->getCode(), $e);
         }
     }

Braces +7 added lines, -3 removed lines

@@ -84,11 +84,13 @@ 
 
     protected function coreHandler(ContainerInterface $container): CoreHandler
     {
-        if ($this->handler !== null) {
+        if ($this->handler !== null)
+        {
             return $this->handler;
         }
 
-        try {
+        try
+        {
             // construct on demand
             $this->handler = new CoreHandler(
                 $this->pipeline ?? $container->get(HandlerInterface::class),
@@ -98,7 +100,9 @@ 
             );
 
             return $this->handler;
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new TargetException($e->getMessage(), $e->getCode(), $e);
         }
     }

src/Scaffolder/src/Declaration/BootloaderDeclaration.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
         ?string $comment = null,
         ?string $namespace = null,
         private readonly bool $isDomain = false,
-    ) {
+    ){
         parent::__construct($config, $name, $comment, $namespace);
     }
 
@@ -43,7 +43,7 @@ 
         $this->class->addConstant('SINGLETONS', [])->setProtected();
         $this->class->addConstant('DEPENDENCIES', [])->setProtected();
 
-        if ($this->isDomain) {
+        if ($this->isDomain){
             $this->class->addConstant('INTERCEPTORS', [])->setProtected();
             $this->namespace->addUse(HandlerInterface::class);
             $this->class->getConstant('SINGLETONS')->setValue([

Braces +2 added lines, -1 removed lines

@@ -43,7 +43,8 @@
         $this->class->addConstant('SINGLETONS', [])->setProtected();
         $this->class->addConstant('DEPENDENCIES', [])->setProtected();
 
-        if ($this->isDomain) {
+        if ($this->isDomain)
+        {
             $this->class->addConstant('INTERCEPTORS', [])->setProtected();
             $this->namespace->addUse(HandlerInterface::class);
             $this->class->getConstant('SINGLETONS')->setValue([

src/Events/src/EventDispatcher.php

Spacing +2 added lines, -2 removed lines

@@ -14,8 +14,8 @@
 {
     private readonly bool $isLegacy;
     public function __construct(
-        private readonly HandlerInterface|CoreInterface $core
-    ) {
+        private readonly HandlerInterface | CoreInterface $core
+    ){
         $this->isLegacy = !$core instanceof HandlerInterface;
     }
 

src/Framework/Bootloader/DomainBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -31,8 +31,8 @@
     ): CoreInterface&HandlerInterface {
         $pipeline = (new InterceptorPipeline($dispatcher))->withCore($core);
 
-        foreach (static::defineInterceptors() as $interceptor) {
-            if (!$interceptor instanceof CoreInterceptorInterface && !$interceptor instanceof InterceptorInterface) {
+        foreach (static::defineInterceptors() as $interceptor){
+            if (!$interceptor instanceof CoreInterceptorInterface && !$interceptor instanceof InterceptorInterface){
                 $interceptor = $container->get($interceptor);
             }
 

Braces +4 added lines, -2 removed lines

@@ -31,8 +31,10 @@
     ): CoreInterface&HandlerInterface {
         $pipeline = (new InterceptorPipeline($dispatcher))->withCore($core);
 
-        foreach (static::defineInterceptors() as $interceptor) {
-            if (!$interceptor instanceof CoreInterceptorInterface && !$interceptor instanceof InterceptorInterface) {
+        foreach (static::defineInterceptors() as $interceptor)
+        {
+            if (!$interceptor instanceof CoreInterceptorInterface && !$interceptor instanceof InterceptorInterface)
+            {
                 $interceptor = $container->get($interceptor);
             }