spiral / framework

src/Telemetry/src/SpanInterface.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      * @param non-empty-string|int $code
      * @param non-empty-string|null $description
      */
-    public function setStatus(string|int $code, ?string $description = null): self;
+    public function setStatus(string | int $code, ?string $description = null): self;
 
     /**
      * Get the current span status.

src/Storage/src/Storage/ReadableTrait.php

Spacing +8 added lines, -8 removed lines

@@ -18,21 +18,21 @@ 
      */
     abstract public function bucket(?string $name = null): BucketInterface;
 
-    public function getContents(string|\Stringable $id): string
+    public function getContents(string | \Stringable $id): string
     {
         [$name, $pathname] = $this->parseUri($id);
 
         return $this->bucket($name)->getContents($pathname);
     }
 
-    public function getStream(string|\Stringable $id)
+    public function getStream(string | \Stringable $id)
     {
         [$name, $pathname] = $this->parseUri($id);
 
         return $this->bucket($name)->getStream($pathname);
     }
 
-    public function exists(string|\Stringable $id): bool
+    public function exists(string | \Stringable $id): bool
     {
         [$name, $pathname] = $this->parseUri($id);
 
@@ -42,7 +42,7 @@ 
     /**
      * @return int<0, max>
      */
-    public function getLastModified(string|\Stringable $id): int
+    public function getLastModified(string | \Stringable $id): int
     {
         [$name, $pathname] = $this->parseUri($id);
 
@@ -52,14 +52,14 @@ 
     /**
      * @return int<0, max>
      */
-    public function getSize(string|\Stringable $id): int
+    public function getSize(string | \Stringable $id): int
     {
         [$name, $pathname] = $this->parseUri($id);
 
         return $this->bucket($name)->getSize($pathname);
     }
 
-    public function getMimeType(string|\Stringable $id): string
+    public function getMimeType(string | \Stringable $id): string
     {
         [$name, $pathname] = $this->parseUri($id);
 
@@ -70,7 +70,7 @@ 
      * @return Visibility::VISIBILITY_*
      */
     #[ExpectedValues(valuesFromClass: Visibility::class)]
-    public function getVisibility(string|\Stringable $id): string
+    public function getVisibility(string | \Stringable $id): string
     {
         [$name, $pathname] = $this->parseUri($id);
 
@@ -80,5 +80,5 @@ 
     /**
      * {@see Storage::parseUri()}
      */
-    abstract protected function parseUri(string|\Stringable $uri, bool $withScheme = true): array;
+    abstract protected function parseUri(string | \Stringable $uri, bool $withScheme = true): array;
 }

src/Core/tests/Scope/FinalizeAttributeTest.php

Spacing +6 added lines, -6 removed lines

@@ -21,8 +21,8 @@ 
     {
         $root = self::makeContainer();
 
-        $obj = $root->runScoped(static function (Container $c1) {
-            $obj = $c1->runScoped(static function (Container $c2) {
+        $obj = $root->runScoped(static function (Container $c1){
+            $obj = $c1->runScoped(static function (Container $c2){
                 $obj = $c2->get(AttrScopeFooFinalize::class);
 
                 self::assertFalse($obj->finalized);
@@ -124,12 +124,12 @@ 
         self::expectException(FinalizersException::class);
         self::expectExceptionMessage('An exception has been thrown during finalization of the scope `foo`');
 
-        try {
+        try{
             $root->runScoped(static function (Container $c1): void {
                 $obj = $c1->get(AttrScopeFooFinalize::class);
                 $obj->throwException = true;
             }, name: 'foo');
-        } catch (FinalizersException $e) {
+        }catch (FinalizersException $e){
             self::assertSame('foo', $e->getScope());
             self::assertCount(1, $e->getExceptions());
             // Contains the message from the inner exception.
@@ -153,13 +153,13 @@ 
         self::expectException(FinalizersException::class);
         self::expectExceptionMessage('3 exceptions have been thrown during finalization of the scope `foo`');
 
-        try {
+        try{
             $root->runScoped(static function (Container $c1): void {
                 $c1->get(AttrScopeFooFinalize::class)->throwException = true;
                 $c1->get(AttrScopeFooFinalize::class)->throwException = true;
                 $c1->get(AttrScopeFooFinalize::class)->throwException = true;
             }, name: 'foo');
-        } catch (FinalizersException $e) {
+        }catch (FinalizersException $e){
             self::assertSame('foo', $e->getScope());
             self::assertCount(3, $e->getExceptions());
             // Contains the message from the inner exception.

Braces +22 added lines, -10 removed lines

@@ -21,8 +21,10 @@ 
     {
         $root = self::makeContainer();
 
-        $obj = $root->runScoped(static function (Container $c1) {
-            $obj = $c1->runScoped(static function (Container $c2) {
+        $obj = $root->runScoped(static function (Container $c1)
+        {
+            $obj = $c1->runScoped(static function (Container $c2)
+            {
                 $obj = $c2->get(AttrScopeFooFinalize::class);
 
                 self::assertFalse($obj->finalized);
@@ -46,8 +48,10 @@ 
         $root->bindSingleton(LoggerInterface::class, FileLogger::class);
 
         $obj2 = null;
-        $obj = $root->runScoped(static function (Container $c1) use (&$obj2) {
-            $obj = $c1->runScoped(static function (Container $c2) use (&$obj2) {
+        $obj = $root->runScoped(static function (Container $c1) use (&$obj2)
+        {
+            $obj = $c1->runScoped(static function (Container $c2) use (&$obj2)
+            {
                 $obj = $c2->get(AttrScopeFooFinalize::class);
                 $obj2 = $c2->get(AttrScopeFooFinalize::class);
 
@@ -77,8 +81,10 @@ 
         $root->bindSingleton(LoggerInterface::class, FileLogger::class);
 
         $obj2 = null;
-        $obj = $root->runScoped(static function (Container $c1) use (&$obj2) {
-            $obj = $c1->runScoped(static function (Container $c2) use (&$obj2) {
+        $obj = $root->runScoped(static function (Container $c1) use (&$obj2)
+        {
+            $obj = $c1->runScoped(static function (Container $c2) use (&$obj2)
+            {
                 $obj = $c2->get(AttrFinalize::class);
                 $obj2 = $c2->get(AttrFinalize::class);
 
@@ -124,12 +130,15 @@ 
         self::expectException(FinalizersException::class);
         self::expectExceptionMessage('An exception has been thrown during finalization of the scope `foo`');
 
-        try {
+        try
+        {
             $root->runScoped(static function (Container $c1): void {
                 $obj = $c1->get(AttrScopeFooFinalize::class);
                 $obj->throwException = true;
             }, name: 'foo');
-        } catch (FinalizersException $e) {
+        }
+        catch (FinalizersException $e)
+        {
             self::assertSame('foo', $e->getScope());
             self::assertCount(1, $e->getExceptions());
             // Contains the message from the inner exception.
@@ -153,13 +162,16 @@ 
         self::expectException(FinalizersException::class);
         self::expectExceptionMessage('3 exceptions have been thrown during finalization of the scope `foo`');
 
-        try {
+        try
+        {
             $root->runScoped(static function (Container $c1): void {
                 $c1->get(AttrScopeFooFinalize::class)->throwException = true;
                 $c1->get(AttrScopeFooFinalize::class)->throwException = true;
                 $c1->get(AttrScopeFooFinalize::class)->throwException = true;
             }, name: 'foo');
-        } catch (FinalizersException $e) {
+        }
+        catch (FinalizersException $e)
+        {
             self::assertSame('foo', $e->getScope());
             self::assertCount(3, $e->getExceptions());
             // Contains the message from the inner exception.

src/Core/tests/Scope/ScopeAttributeTest.php

Spacing +4 added lines, -4 removed lines

@@ -165,7 +165,7 @@ 
 
         $root = self::makeContainer();
 
-        try {
+        try{
             $root->runScoped(static function (Container $c1): void {
                 $c1->runScoped(static function (Container $c2): void {
                     $c2->runScoped(static function (Container $c3): void {
@@ -173,7 +173,7 @@ 
                     }, name: 'root');
                 });
             });
-        } catch (NamedScopeDuplicationException $e) {
+        }catch (NamedScopeDuplicationException $e){
             self::assertSame('root', $e->getScope());
             throw $e;
         }
@@ -188,14 +188,14 @@ 
     {
         self::expectException(BadScopeException::class);
 
-        try {
+        try{
             $root = self::makeContainer();
             $root->runScoped(static function (Container $c1): void {
                 $c1->runScoped(static function (Container $c2): void {
                     $c2->get(AttrScopeFoo::class);
                 });
             }, name: 'bar');
-        } catch (BadScopeException $e) {
+        }catch (BadScopeException $e){
             self::assertSame('foo', $e->getScope());
             throw $e;
         }

Braces +10 added lines, -4 removed lines

@@ -165,7 +165,8 @@ 
 
         $root = self::makeContainer();
 
-        try {
+        try
+        {
             $root->runScoped(static function (Container $c1): void {
                 $c1->runScoped(static function (Container $c2): void {
                     $c2->runScoped(static function (Container $c3): void {
@@ -173,7 +174,9 @@ 
                     }, name: 'root');
                 });
             });
-        } catch (NamedScopeDuplicationException $e) {
+        }
+        catch (NamedScopeDuplicationException $e)
+        {
             self::assertSame('root', $e->getScope());
             throw $e;
         }
@@ -188,14 +191,17 @@ 
     {
         self::expectException(BadScopeException::class);
 
-        try {
+        try
+        {
             $root = self::makeContainer();
             $root->runScoped(static function (Container $c1): void {
                 $c1->runScoped(static function (Container $c2): void {
                     $c2->get(AttrScopeFoo::class);
                 });
             }, name: 'bar');
-        } catch (BadScopeException $e) {
+        }
+        catch (BadScopeException $e)
+        {
             self::assertSame('foo', $e->getScope());
             throw $e;
         }

src/Core/tests/Scope/ExceptionsTest.php

Spacing +6 added lines, -6 removed lines

@@ -21,10 +21,10 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try{
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -39,10 +39,10 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try{
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -57,10 +57,10 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try{
                 $c1->get(DatetimeCarrier::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 self::assertInstanceOf(NotFoundException::class, $e);
                 self::assertInstanceOf(NotFoundException::class, $e->getPrevious());
                 self::assertStringContainsString(

Braces +15 added lines, -6 removed lines

@@ -21,10 +21,13 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try
+            {
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -39,10 +42,13 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try
+            {
                 $c1->get(ExceptionConstructor::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 self::assertInstanceOf(\Exception::class, $e);
                 throw $e;
             }
@@ -57,10 +63,13 @@ 
         $container = new Container();
 
         $container->runScoped(static function (Container $c1): void {
-            try {
+            try
+            {
                 $c1->get(DatetimeCarrier::class);
                 self::fail('Exception should be thrown');
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 self::assertInstanceOf(NotFoundException::class, $e);
                 self::assertInstanceOf(NotFoundException::class, $e->getPrevious());
                 self::assertStringContainsString(

src/Http/tests/PipelineTest.php

Braces +4 added lines, -2 removed lines

@@ -60,7 +60,8 @@ 
 
     public function testMiddlewareProcessingEventShouldBeDispatched(): void
     {
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
             {
                 return (new ResponseFactory(new HttpConfig(['headers' => []])))->createResponse(200);
@@ -89,7 +90,8 @@ 
         $this->container->getBinder('http')
             ->bind(ServerRequestInterface::class, static fn(CurrentRequest $cr): ?ServerRequestInterface => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
 
         self::assertSame(200, $response->getStatusCode());
         self::assertSame('OK', $response->getReasonPhrase());
-        self::assertSame('response', (string) $response->getBody());
+        self::assertSame('response', (string)$response->getBody());
     }
 
     public function testHandle(): void
@@ -46,7 +46,7 @@ 
 
         self::assertSame(200, $response->getStatusCode());
         self::assertSame('OK', $response->getReasonPhrase());
-        self::assertSame('response', (string) $response->getBody());
+        self::assertSame('response', (string)$response->getBody());
     }
 
     public function testHandleException(): void
@@ -59,7 +59,7 @@ 
 
     public function testMiddlewareProcessingEventShouldBeDispatched(): void
     {
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface{
             public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
             {
                 return (new ResponseFactory(new HttpConfig(['headers' => []])))->createResponse(200);
@@ -88,7 +88,7 @@ 
         $this->container->getBinder('http')
             ->bind(ServerRequestInterface::class, static fn(CurrentRequest $cr): ?ServerRequestInterface => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface{
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,

src/Distribution/src/Bootloader/DistributionBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
             static function (DistributionConfig $config): DistributionInterface {
                 $manager = new Manager($config->getDefaultDriver());
 
-                foreach ($config->getResolvers() as $name => $resolver) {
+                foreach ($config->getResolvers() as $name => $resolver){
                     $manager->add($name, $resolver);
                 }
 

Braces +2 added lines, -1 removed lines

@@ -48,7 +48,8 @@
             static function (DistributionConfig $config): DistributionInterface {
                 $manager = new Manager($config->getDefaultDriver());
 
-                foreach ($config->getResolvers() as $name => $resolver) {
+                foreach ($config->getResolvers() as $name => $resolver)
+                {
                     $manager->add($name, $resolver);
                 }
 

src/Router/src/Router.php

Braces +49 added lines, -22 removed lines

@@ -68,13 +68,17 @@ 
         return $this->tracer->trace(
             name: 'Routing',
             callback: function (SpanInterface $span) use ($request): ResponseInterface {
-                try {
+                try
+                {
                     $route = $this->matchRoute($request, $routeName);
-                } catch (RouteException $e) {
+                }
+                catch (RouteException $e)
+                {
                     throw new RouterException('Invalid route definition', $e->getCode(), $e);
                 }
 
-                if ($route === null) {
+                if ($route === null)
+                {
                     $this->eventDispatcher?->dispatch(new RouteNotFound($request));
                     throw new RouteNotFoundException($request->getUri());
                 }
@@ -109,7 +113,8 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name]))
+        {
             return $this->routes[$name];
         }
 
@@ -118,7 +123,8 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default))
+        {
             return $this->routes + [null => $this->default];
         }
 
@@ -127,9 +133,12 @@ 
 
     public function uri(string $route, iterable $parameters = []): UriInterface
     {
-        try {
+        try
+        {
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException) {
+        }
+        catch (UndefinedRouteException)
+        {
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -141,9 +150,11 @@ 
         /** @var GroupRegistry $groups */
         $groups = $this->container->get(GroupRegistry::class);
 
-        foreach ($routes->getCollection() as $name => $configurator) {
+        foreach ($routes->getCollection() as $name => $configurator)
+        {
             $target = $configurator->target;
-            if ($configurator->core !== null && $target instanceof AbstractTarget) {
+            if ($configurator->core !== null && $target instanceof AbstractTarget)
+            {
                 $target = $target->withCore($configurator->core);
             }
 
@@ -152,21 +163,25 @@ 
                 : \ltrim($configurator->pattern, '/');
             $route = new Route($pattern, $target, $configurator->defaults);
 
-            if ($configurator->middleware !== null) {
+            if ($configurator->middleware !== null)
+            {
                 $route = $route->withMiddleware(...$configurator->middleware);
             }
 
-            if ($configurator->methods !== null) {
+            if ($configurator->methods !== null)
+            {
                 $route = $route->withVerbs(...$configurator->methods);
             }
 
-            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME)
+            {
                 $group = $groups->getGroup($configurator->group ?? $groups->getDefaultGroup());
                 $group->setPrefix($configurator->prefix);
                 $group->addRoute($name, $route);
             }
 
-            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME)
+            {
                 $this->setDefault($route);
             }
         }
@@ -177,17 +192,20 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, ?string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route)
+        {
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null)
+            {
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null)
+        {
             return $this->default->match($request);
         }
 
@@ -200,14 +218,18 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer())
+        {
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
 
-        try {
+        try
+        {
             $uriHandler = $route->getUriHandler();
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
             $uriHandler = $this->uriHandler;
         }
 
@@ -239,11 +261,16 @@ 
         /**
          * @var Matches $matches
          */
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name']))
+        {
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        }
+        elseif ($this->default !== null)
+        {
             $routeObject = $this->default;
-        } else {
+        }
+        else
+        {
             throw new UndefinedRouteException(\sprintf('Unable to locate route candidate for `%s`', $route));
         }
 

Spacing +26 added lines, -26 removed lines

@@ -52,9 +52,9 @@ 
         private readonly ContainerInterface $container,
         private readonly ?EventDispatcherInterface $eventDispatcher = null,
         ?TracerInterface $tracer = null,
-    ) {
+    ){
         $this->tracer = $tracer ?? new NullTracer();
-        $this->basePath = '/' . \ltrim($basePath, '/');
+        $this->basePath = '/'.\ltrim($basePath, '/');
     }
 
     /**
@@ -68,19 +68,19 @@ 
         return $this->tracer->trace(
             name: 'Routing',
             callback: function (SpanInterface $span) use ($request): ResponseInterface {
-                try {
+                try{
                     $route = $this->matchRoute($request, $routeName);
-                } catch (RouteException $e) {
+                }catch (RouteException $e){
                     throw new RouterException('Invalid route definition', $e->getCode(), $e);
                 }
 
-                if ($route === null) {
+                if ($route === null){
                     $this->eventDispatcher?->dispatch(new RouteNotFound($request));
                     throw new RouteNotFoundException($request->getUri());
                 }
 
                 $span
-                    ->setAttribute('request.uri', (string) $request->getUri())
+                    ->setAttribute('request.uri', (string)$request->getUri())
                     ->setAttribute('route.name', $routeName)
                     ->setAttribute('route.matches', $route->getMatches() ?? []);
 
@@ -109,7 +109,7 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name])){
             return $this->routes[$name];
         }
 
@@ -118,7 +118,7 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default)){
             return $this->routes + [null => $this->default];
         }
 
@@ -127,9 +127,9 @@ 
 
     public function uri(string $route, iterable $parameters = []): UriInterface
     {
-        try {
+        try{
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException) {
+        }catch (UndefinedRouteException){
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -141,9 +141,9 @@ 
         /** @var GroupRegistry $groups */
         $groups = $this->container->get(GroupRegistry::class);
 
-        foreach ($routes->getCollection() as $name => $configurator) {
+        foreach ($routes->getCollection() as $name => $configurator){
             $target = $configurator->target;
-            if ($configurator->core !== null && $target instanceof AbstractTarget) {
+            if ($configurator->core !== null && $target instanceof AbstractTarget){
                 $target = $target->withCore($configurator->core);
             }
 
@@ -152,21 +152,21 @@ 
                 : \ltrim($configurator->pattern, '/');
             $route = new Route($pattern, $target, $configurator->defaults);
 
-            if ($configurator->middleware !== null) {
+            if ($configurator->middleware !== null){
                 $route = $route->withMiddleware(...$configurator->middleware);
             }
 
-            if ($configurator->methods !== null) {
+            if ($configurator->methods !== null){
                 $route = $route->withVerbs(...$configurator->methods);
             }
 
-            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME){
                 $group = $groups->getGroup($configurator->group ?? $groups->getDefaultGroup());
                 $group->setPrefix($configurator->prefix);
                 $group->addRoute($name, $route);
             }
 
-            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME){
                 $this->setDefault($route);
             }
         }
@@ -177,17 +177,17 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, ?string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route){
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null){
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null){
             return $this->default->match($request);
         }
 
@@ -200,14 +200,14 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer()){
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
 
-        try {
+        try{
             $uriHandler = $route->getUriHandler();
-        } catch (\Throwable) {
+        }catch (\Throwable){
             $uriHandler = $this->uriHandler;
         }
 
@@ -230,7 +230,7 @@ 
                 $route,
                 $matches,
             )
-        ) {
+        ){
             throw new UndefinedRouteException(
                 "Unable to locate route or use default route with 'name/controller:action' pattern",
             );
@@ -239,11 +239,11 @@ 
         /**
          * @var Matches $matches
          */
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name'])){
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        } elseif ($this->default !== null){
             $routeObject = $this->default;
-        } else {
+        }else{
             throw new UndefinedRouteException(\sprintf('Unable to locate route candidate for `%s`', $route));
         }
 

src/Core/tests/ExceptionsTest.php

Spacing +4 added lines, -4 removed lines

@@ -135,9 +135,9 @@ 
     {
         $container = new Container();
 
-        try {
+        try{
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             self::assertSame(<<<MARKDOWN
                 Can't resolve `invalid`: undefined class or binding `invalid`.
                 Container trace list:
@@ -166,9 +166,9 @@ 
     {
         $this->expectException(ContainerException::class);
 
-        try {
+        try{
             $container->get(ClassWithUndefinedDependency::class);
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             self::assertSame($message, $e->getMessage());
 
             throw $e;

Braces +10 added lines, -4 removed lines

@@ -135,9 +135,12 @@ 
     {
         $container = new Container();
 
-        try {
+        try
+        {
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame(<<<MARKDOWN
                 Can't resolve `invalid`: undefined class or binding `invalid`.
                 Container trace list:
@@ -166,9 +169,12 @@ 
     {
         $this->expectException(ContainerException::class);
 
-        try {
+        try
+        {
             $container->get(ClassWithUndefinedDependency::class);
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame($message, $e->getMessage());
 
             throw $e;