spiral / framework

src/Framework/Bootloader/Http/RoutesBootloader.php

Braces +6 added lines, -3 removed lines

@@ -19,7 +19,8 @@ 
 {
     public function init(HttpBootloader $http): void
     {
-        foreach ($this->globalMiddleware() as $middleware) {
+        foreach ($this->globalMiddleware() as $middleware)
+        {
             $http->addMiddleware($middleware);
         }
     }
@@ -61,7 +62,8 @@ 
 
     private function registerMiddlewareGroups(BinderInterface $binder, array $groups): void
     {
-        foreach ($groups as $group => $middleware) {
+        foreach ($groups as $group => $middleware)
+        {
             $binder
                 ->getBinder('http')
                 ->bind(
@@ -75,7 +77,8 @@ 
 
     private function registerMiddlewareForRouteGroups(GroupRegistry $registry, array $groups): void
     {
-        foreach ($groups as $group) {
+        foreach ($groups as $group)
+        {
             $registry->getGroup($group)->addMiddleware('middleware:' . $group);
         }
     }

src/Http/src/LazyPipeline.php

Braces +9 added lines, -4 removed lines

@@ -92,9 +92,11 @@ 
 
         $previousRequest = $currentRequest->get();
         $currentRequest->set($request);
-        try {
+        try
+        {
             // There is no middleware to process, let's pass the request to the handler
-            if (!\array_key_exists($this->position, $this->middleware)) {
+            if (!\array_key_exists($this->position, $this->middleware))
+            {
                 return $this->handler->handle($request);
             }
 
@@ -108,7 +110,8 @@ 
                 ? \sprintf('%s=%s', $this->middleware[$this->position], $middleware::class)
                 : $middleware::class;
             // Init a tracing span when the pipeline starts
-            if ($span === null) {
+            if ($span === null)
+            {
                 /** @var TracerInterface $tracer */
                 $tracer = $this->container->get(TracerInterface::class);
                 return $tracer->trace(
@@ -126,7 +129,9 @@ 
             $span->setAttribute('http.middleware', $middlewares);
 
             return $middleware->process($request, $this->next($span));
-        } finally {
+        }
+        finally
+        {
             $currentRequest->set($previousRequest);
         }
     }

src/Http/src/Http.php

Braces +11 added lines, -5 removed lines

@@ -35,11 +35,15 @@ 
         ?TracerFactoryInterface $tracerFactory = null,
         private readonly ?EventDispatcherInterface $dispatcher = null,
     ) {
-        if ($pipeline instanceof Pipeline) {
-            foreach ($this->config->getMiddleware() as $middleware) {
+        if ($pipeline instanceof Pipeline)
+        {
+            foreach ($this->config->getMiddleware() as $middleware)
+            {
                 $pipeline->pushMiddleware($this->container->get($middleware));
             }
-        } else {
+        }
+        else
+        {
             $pipeline = $pipeline->withAddedMiddleware(
                 ...$this->config->getMiddleware()
             );
@@ -77,7 +81,8 @@ 
 
             $this->dispatcher?->dispatch(new RequestReceived($request));
 
-            if ($this->handler === null) {
+            if ($this->handler === null)
+            {
                 throw new HttpException('Unable to run HttpCore, no handler is set.');
             }
 
@@ -117,7 +122,8 @@ 
             traceKind: TraceKind::SERVER,
         );
 
-        foreach ($tracer->getContext() as $key => $value) {
+        foreach ($tracer->getContext() as $key => $value)
+        {
             $response = $response->withHeader($key, $value);
         }
 

src/Router/src/Traits/PipelineTrait.php

Braces +16 added lines, -7 removed lines

@@ -44,16 +44,19 @@ 
         $route = clone $this;
 
         // array fallback
-        if (\count($middleware) === 1 && \is_array($middleware[0])) {
+        if (\count($middleware) === 1 && \is_array($middleware[0]))
+        {
             $middleware = $middleware[0];
         }
 
         /** @var MiddlewareType[] $middleware */
-        foreach ($middleware as $item) {
+        foreach ($middleware as $item)
+        {
             $route->middleware[] = $item;
         }
 
-        if ($route->pipeline !== null) {
+        if ($route->pipeline !== null)
+        {
             $route->pipeline = $route->makeLazyPipeline();
         }
 
@@ -80,11 +83,14 @@ 
     protected function makePipeline(): Pipeline
     {
         \assert($this->container !== null);
-        try {
+        try
+        {
             return $this->container
                 ->get(PipelineFactory::class)
                 ->createWithMiddleware($this->middleware);
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }
@@ -97,11 +103,14 @@ 
     protected function makeLazyPipeline(): LazyPipeline
     {
         \assert($this->container !== null);
-        try {
+        try
+        {
             /** @var LazyPipeline $pipeline */
             $pipeline = $this->container->get(LazyPipeline::class);
             return $pipeline->withMiddleware(...$this->middleware);
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }

src/Router/src/Route.php

Braces +22 added lines, -10 removed lines

@@ -69,7 +69,8 @@ 
      */
     public function withUriHandler(UriHandler $uriHandler): static
     {
-        if ($this->target instanceof TargetInterface) {
+        if ($this->target instanceof TargetInterface)
+        {
             $uriHandler = $uriHandler->withConstrains(
                 $this->target->getConstrains(),
                 $this->defaults,
@@ -87,7 +88,8 @@ 
         $route = clone $this;
         $route->container = $container;
 
-        if ($route->target instanceof TargetInterface) {
+        if ($route->target instanceof TargetInterface)
+        {
             $route->target = clone $route->target;
         }
 
@@ -114,7 +116,8 @@ 
      */
     public function handle(ServerRequestInterface $request): ResponseInterface
     {
-        if (empty($this->requestHandler)) {
+        if (empty($this->requestHandler))
+        {
             $this->requestHandler = $this->requestHandler();
         }
 
@@ -134,25 +137,32 @@ 
             'Unable to configure route pipeline without associated container.',
         );
 
-        if ($this->target instanceof TargetInterface) {
-            try {
+        if ($this->target instanceof TargetInterface)
+        {
+            try
+            {
                 \assert($this->matches !== null);
                 return $this->target->getHandler($this->container, $this->matches);
-            } catch (TargetException $e) {
+            }
+            catch (TargetException $e)
+            {
                 throw new RouteException('Invalid target resolution', $e->getCode(), $e);
             }
         }
 
-        if ($this->target instanceof RequestHandlerInterface) {
+        if ($this->target instanceof RequestHandlerInterface)
+        {
             return $this->target;
         }
 
-        try {
+        try
+        {
             $target = \is_string($this->target)
                 ? $this->container->get($this->target)
                 : $this->target;
 
-            if ($target instanceof RequestHandlerInterface) {
+            if ($target instanceof RequestHandlerInterface)
+            {
                 return $target;
             }
 
@@ -160,7 +170,9 @@ 
                 $target,
                 $this->container->get(ResponseFactoryInterface::class)
             );
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
     }

src/Telemetry/src/AbstractTracer.php

Braces +17 added lines, -7 removed lines

@@ -28,33 +28,43 @@
     final protected function runScope(Span $span, callable $callback): mixed
     {
         $container = ContainerScope::getContainer();
-        if ($container === null) {
+        if ($container === null)
+        {
             return $this->scope->runScope([
                 SpanInterface::class => $span,
                 TracerInterface::class => $this,
             ], static fn (InvokerInterface $invoker): mixed => $invoker->invoke($callback));
         }
 
-        if ($container instanceof Container) {
+        if ($container instanceof Container)
+        {
             $invoker = $container;
             $binder = $container;
-        } else {
+        }
+        else
+        {
             /** @var InvokerInterface $invoker */
             $invoker = $container->get(InvokerInterface::class);
             /** @var BinderInterface $binder */
             $binder = $container->get(BinderInterface::class);
         }
 
-        try {
+        try
+        {
             $prevSpan = $container->get(SpanInterface::class);
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
             $prevSpan = null;
         }
 
         $binder->bindSingleton(SpanInterface::class, $span);
-        try {
+        try
+        {
             return $invoker->invoke($callback);
-        } finally {
+        }
+        finally
+        {
             $prevSpan === null
                 ? $binder->removeBinding(SpanInterface::class)
                 : $binder->bindSingleton(SpanInterface::class, $prevSpan);

src/Telemetry/tests/NullTracerTest.php

Braces +2 added lines, -1 removed lines

@@ -77,7 +77,8 @@
             ->with(SpanInterface::class);
         $scope->shouldNotReceive('runScope');
 
-        ContainerScope::runScope($container, function () use ($tracer, $callable) {
+        ContainerScope::runScope($container, function () use ($tracer, $callable)
+        {
             $this->assertSame(
                 'hello',
                 $tracer->trace('foo', $callable, ['foo' => 'bar'])