spiral / framework

src/Hmvc/tests/Core/Unit/Stub/MultipleCallNextInterceptor.php

Spacing +2 added lines, -2 removed lines

@@ -14,13 +14,13 @@
 
     public function __construct(
         private readonly int $counter,
-    ) {
+    ){
     }
 
     public function intercept(CallContextInterface $context, HandlerInterface $handler): mixed
     {
         $this->result = [];
-        for ($i = 0; $i < $this->counter; ++$i) {
+        for ($i = 0; $i < $this->counter; ++$i){
             $this->result[] = $handler->handle($context);
         }
 

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
     public function intercept(CallContextInterface $context, HandlerInterface $handler): mixed
     {
         $this->result = [];
-        for ($i = 0; $i < $this->counter; ++$i) {
+        for ($i = 0; $i < $this->counter; ++$i)
+        {
             $this->result[] = $handler->handle($context);
         }
 

src/Hmvc/src/Interceptors/Internal/ActionResolver.php

Spacing +5 added lines, -5 removed lines

@@ -19,10 +19,10 @@ 
      */
     public static function pathToReflection(string $controller, string $action): \ReflectionMethod
     {
-        try {
+        try{
             /** @psalm-suppress ArgumentTypeCoercion */
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             throw new TargetCallException(
                 \sprintf('Invalid action `%s`->`%s`', $controller, $action),
                 TargetCallException::BAD_ACTION,
@@ -39,7 +39,7 @@ 
      */
     public static function validateControllerMethod(\ReflectionMethod $method, mixed $controller = null): void
     {
-        if ($method->isStatic() || !$method->isPublic()) {
+        if ($method->isStatic() || !$method->isPublic()){
             throw new TargetCallException(
                 \sprintf(
                     'Invalid action `%s`->`%s`',
@@ -50,11 +50,11 @@ 
             );
         }
 
-        if ($controller === null) {
+        if ($controller === null){
             return;
         }
 
-        if (!\is_object($controller) || !$method->getDeclaringClass()->isInstance($controller)) {
+        if (!\is_object($controller) || !$method->getDeclaringClass()->isInstance($controller)){
             throw new TargetCallException(
                 \sprintf(
                     'Invalid controller. Expected instance of `%s`, got `%s`.',

Braces +11 added lines, -5 removed lines

@@ -19,10 +19,13 @@ 
      */
     public static function pathToReflection(string $controller, string $action): \ReflectionMethod
     {
-        try {
+        try
+        {
             /** @psalm-suppress ArgumentTypeCoercion */
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             throw new TargetCallException(
                 \sprintf('Invalid action `%s`->`%s`', $controller, $action),
                 TargetCallException::BAD_ACTION,
@@ -39,7 +42,8 @@ 
      */
     public static function validateControllerMethod(\ReflectionMethod $method, mixed $controller = null): void
     {
-        if ($method->isStatic() || !$method->isPublic()) {
+        if ($method->isStatic() || !$method->isPublic())
+        {
             throw new TargetCallException(
                 \sprintf(
                     'Invalid action `%s`->`%s`',
@@ -50,11 +54,13 @@ 
             );
         }
 
-        if ($controller === null) {
+        if ($controller === null)
+        {
             return;
         }
 
-        if (!\is_object($controller) || !$method->getDeclaringClass()->isInstance($controller)) {
+        if (!\is_object($controller) || !$method->getDeclaringClass()->isInstance($controller))
+        {
             throw new TargetCallException(
                 \sprintf(
                     'Invalid controller. Expected instance of `%s`, got `%s`.',

src/Queue/src/QueueManager.php

Spacing +6 added lines, -6 removed lines

@@ -25,7 +25,7 @@ 
         private readonly ContainerInterface $container,
         private readonly FactoryInterface $factory,
         private readonly ?EventDispatcherInterface $dispatcher = null
-    ) {
+    ){
     }
 
     public function getConnection(?string $name = null): QueueInterface
@@ -34,7 +34,7 @@ 
         // Replaces alias with real pipeline name
         $name = $this->config->getAliases()[$name] ?? $name;
 
-        if (!isset($this->pipelines[$name])) {
+        if (!isset($this->pipelines[$name])){
             $this->pipelines[$name] = $this->resolveConnection($name);
         }
 
@@ -49,12 +49,12 @@ 
     {
         $config = $this->config->getConnection($name);
 
-        try {
+        try{
             $driver = $this->factory->make($config['driver'], $config);
             $pipeline = (new InterceptorPipeline($this->dispatcher))->withHandler(new PushCore($driver));
 
-            foreach ($this->config->getPushInterceptors() as $interceptor) {
-                if (\is_string($interceptor) || $interceptor instanceof Autowire) {
+            foreach ($this->config->getPushInterceptors() as $interceptor){
+                if (\is_string($interceptor) || $interceptor instanceof Autowire){
                     $interceptor = $this->container->get($interceptor);
                 }
 
@@ -63,7 +63,7 @@ 
             }
 
             return new Queue($pipeline);
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             throw new Exception\NotSupportedDriverException(
                 \sprintf(
                     'Driver `%s` is not supported. Connection `%s` cannot be created. Reason: `%s`',

Braces +11 added lines, -5 removed lines

@@ -34,7 +34,8 @@ 
         // Replaces alias with real pipeline name
         $name = $this->config->getAliases()[$name] ?? $name;
 
-        if (!isset($this->pipelines[$name])) {
+        if (!isset($this->pipelines[$name]))
+        {
             $this->pipelines[$name] = $this->resolveConnection($name);
         }
 
@@ -49,12 +50,15 @@ 
     {
         $config = $this->config->getConnection($name);
 
-        try {
+        try
+        {
             $driver = $this->factory->make($config['driver'], $config);
             $pipeline = (new InterceptorPipeline($this->dispatcher))->withHandler(new PushCore($driver));
 
-            foreach ($this->config->getPushInterceptors() as $interceptor) {
-                if (\is_string($interceptor) || $interceptor instanceof Autowire) {
+            foreach ($this->config->getPushInterceptors() as $interceptor)
+            {
+                if (\is_string($interceptor) || $interceptor instanceof Autowire)
+                {
                     $interceptor = $this->container->get($interceptor);
                 }
 
@@ -63,7 +67,9 @@ 
             }
 
             return new Queue($pipeline);
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             throw new Exception\NotSupportedDriverException(
                 \sprintf(
                     'Driver `%s` is not supported. Connection `%s` cannot be created. Reason: `%s`',

src/Queue/src/Bootloader/QueueBootloader.php

Spacing +9 added lines, -9 removed lines

@@ -55,7 +55,7 @@ 
 
     public function __construct(
         private readonly ConfiguratorInterface $config,
-    ) {
+    ){
     }
 
     public function init(
@@ -73,11 +73,11 @@ 
             $registry = $container->get(QueueRegistry::class);
             $config = $container->get(QueueConfig::class);
 
-            foreach ($config->getRegistryHandlers() as $jobType => $handler) {
+            foreach ($config->getRegistryHandlers() as $jobType => $handler){
                 $registry->setHandler($jobType, $handler);
             }
 
-            foreach ($config->getRegistrySerializers() as $jobType => $serializer) {
+            foreach ($config->getRegistrySerializers() as $jobType => $serializer){
                 $registry->setSerializer($jobType, $serializer);
             }
         });
@@ -95,7 +95,7 @@ 
     /**
      * @param class-string<CoreInterceptorInterface>|CoreInterceptorInterface|Autowire $interceptor
      */
-    public function addConsumeInterceptor(string|CoreInterceptorInterface|Autowire $interceptor): void
+    public function addConsumeInterceptor(string | CoreInterceptorInterface | Autowire $interceptor): void
     {
         $this->config->modify(
             QueueConfig::CONFIG,
@@ -106,7 +106,7 @@ 
     /**
      * @param class-string<CoreInterceptorInterface>|CoreInterceptorInterface|Autowire $interceptor
      */
-    public function addPushInterceptor(string|CoreInterceptorInterface|Autowire $interceptor): void
+    public function addPushInterceptor(string | CoreInterceptorInterface | Autowire $interceptor): void
     {
         $this->config->modify(
             QueueConfig::CONFIG,
@@ -131,7 +131,7 @@ 
         ContainerInterface $container,
         FactoryInterface $factory,
         ContainerRegistry $registry,
-    ) {
+    ){
         return new QueueRegistry($container, $factory, $registry);
     }
 
@@ -145,10 +145,10 @@ 
     ): Handler {
         $pipeline = (new InterceptorPipeline($dispatcher))->withHandler($core);
 
-        foreach ($config->getConsumeInterceptors() as $interceptor) {
-            if (\is_string($interceptor)) {
+        foreach ($config->getConsumeInterceptors() as $interceptor){
+            if (\is_string($interceptor)){
                 $interceptor = $container->get($interceptor);
-            } elseif ($interceptor instanceof Autowire) {
+            } elseif ($interceptor instanceof Autowire){
                 $interceptor = $interceptor->resolve($factory);
             }
 

Braces +11 added lines, -5 removed lines

@@ -73,11 +73,13 @@ 
             $registry = $container->get(QueueRegistry::class);
             $config = $container->get(QueueConfig::class);
 
-            foreach ($config->getRegistryHandlers() as $jobType => $handler) {
+            foreach ($config->getRegistryHandlers() as $jobType => $handler)
+            {
                 $registry->setHandler($jobType, $handler);
             }
 
-            foreach ($config->getRegistrySerializers() as $jobType => $serializer) {
+            foreach ($config->getRegistrySerializers() as $jobType => $serializer)
+            {
                 $registry->setSerializer($jobType, $serializer);
             }
         });
@@ -145,10 +147,14 @@ 
     ): Handler {
         $pipeline = (new InterceptorPipeline($dispatcher))->withHandler($core);
 
-        foreach ($config->getConsumeInterceptors() as $interceptor) {
-            if (\is_string($interceptor)) {
+        foreach ($config->getConsumeInterceptors() as $interceptor)
+        {
+            if (\is_string($interceptor))
+            {
                 $interceptor = $container->get($interceptor);
-            } elseif ($interceptor instanceof Autowire) {
+            }
+            elseif ($interceptor instanceof Autowire)
+            {
                 $interceptor = $interceptor->resolve($factory);
             }
 

src/Queue/src/Queue.php

Spacing +2 added lines, -2 removed lines

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

src/Framework/Bootloader/Security/FiltersBootloader.php

Braces +2 added lines, -1 removed lines

@@ -96,7 +96,8 @@
     ): FilterProvider {
         $pipeline = (new InterceptorPipeline($dispatcher))->withHandler(new Core());
 
-        foreach ($config->getInterceptors() as $interceptor) {
+        foreach ($config->getInterceptors() as $interceptor)
+        {
             $pipeline->addInterceptor($container->get($interceptor));
         }
 

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         private readonly ContainerInterface $container,
         private readonly BinderInterface $binder,
         private readonly ConfiguratorInterface $config
-    ) {
+    ){
     }
 
     public function defineSingletons(): array
@@ -113,7 +113,7 @@ 
     ): FilterProvider {
         $pipeline = (new InterceptorPipeline($dispatcher))->withHandler(new Core());
 
-        foreach ($config->getInterceptors() as $interceptor) {
+        foreach ($config->getInterceptors() as $interceptor){
             $pipeline->addInterceptor($container->get($interceptor));
         }
 

src/Events/src/Bootloader/EventsBootloader.php

Braces +8 added lines, -4 removed lines

@@ -72,11 +72,13 @@ 
         FinalizerInterface $finalizer,
         ?EventDispatcherInterface $eventDispatcher = null
     ): void {
-        if ($eventDispatcher !== null) {
+        if ($eventDispatcher !== null)
+        {
             $this->initEventDispatcher(new Core($eventDispatcher), $config, $container, $factory);
         }
 
-        foreach ($config->getProcessors() as $processor) {
+        foreach ($config->getProcessors() as $processor)
+        {
             $processor = $this->autowire($processor, $container, $factory);
 
             \assert($processor instanceof ProcessorInterface);
@@ -87,7 +89,8 @@ 
             $registry->process();
         });
 
-        if ($finalizer instanceof EventDispatcherAwareInterface && $eventDispatcher !== null) {
+        if ($finalizer instanceof EventDispatcherAwareInterface && $eventDispatcher !== null)
+        {
             $finalizer->setEventDispatcher($eventDispatcher);
         }
     }
@@ -108,7 +111,8 @@ 
     ): void {
         $pipeline = (new InterceptorPipeline())->withCore($core);
 
-        foreach ($config->getInterceptors() as $interceptor) {
+        foreach ($config->getInterceptors() as $interceptor)
+        {
             $interceptor = $this->autowire($interceptor, $container, $factory);
 
             \assert($interceptor instanceof CoreInterceptorInterface || $interceptor instanceof InterceptorInterface);

Spacing +7 added lines, -7 removed lines

@@ -48,7 +48,7 @@ 
 
     public function __construct(
         private readonly ConfiguratorInterface $configs
-    ) {
+    ){
     }
 
     public function init(): void
@@ -72,11 +72,11 @@ 
         FinalizerInterface $finalizer,
         ?EventDispatcherInterface $eventDispatcher = null
     ): void {
-        if ($eventDispatcher !== null) {
+        if ($eventDispatcher !== null){
             $this->initEventDispatcher(new Core($eventDispatcher), $config, $container, $factory);
         }
 
-        foreach ($config->getProcessors() as $processor) {
+        foreach ($config->getProcessors() as $processor){
             $processor = $this->autowire($processor, $container, $factory);
 
             \assert($processor instanceof ProcessorInterface);
@@ -87,7 +87,7 @@ 
             $registry->process();
         });
 
-        if ($finalizer instanceof EventDispatcherAwareInterface && $eventDispatcher !== null) {
+        if ($finalizer instanceof EventDispatcherAwareInterface && $eventDispatcher !== null){
             $finalizer->setEventDispatcher($eventDispatcher);
         }
     }
@@ -96,7 +96,7 @@ 
      * @param TInterceptor $interceptor
      */
     public function addInterceptor(
-        string|InterceptorInterface|CoreInterceptorInterface|Container\Autowire $interceptor,
+        string | InterceptorInterface | CoreInterceptorInterface | Container\Autowire $interceptor,
     ): void {
         $this->configs->modify(EventsConfig::CONFIG, new Append('interceptors', null, $interceptor));
     }
@@ -109,7 +109,7 @@ 
     ): void {
         $pipeline = (new InterceptorPipeline())->withCore($core);
 
-        foreach ($config->getInterceptors() as $interceptor) {
+        foreach ($config->getInterceptors() as $interceptor){
             $interceptor = $this->autowire($interceptor, $container, $factory);
 
             \assert($interceptor instanceof CoreInterceptorInterface || $interceptor instanceof InterceptorInterface);
@@ -129,7 +129,7 @@ 
      *
      * @throws ContainerExceptionInterface
      */
-    private function autowire(string|object $id, ContainerInterface $container, FactoryInterface $factory): object
+    private function autowire(string | object $id, ContainerInterface $container, FactoryInterface $factory): object
     {
         return match (true) {
             \is_string($id) => $container->get($id),

src/Router/src/CoreHandler.php

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.');
         }
     }

Spacing +23 added lines, -23 removed lines

@@ -46,11 +46,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;
     }
@@ -98,9 +98,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
@@ -114,42 +114,42 @@ 
                     bindings: [Request::class => $request, Response::class => $response, $controller => $controller],
                 ),
                 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,
         );
     }
 
@@ -162,21 +162,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);
         }
 
@@ -209,7 +209,7 @@ 
      */
     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;
     }