spiral / framework

src/Router/tests/Diactoros/UploadedFileFactory.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         ?string $clientFilename = null,
         ?string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null){
             $size = $stream->getSize();
         }
 

Braces +2 added lines, -1 removed lines

@@ -21,7 +21,8 @@
         ?string $clientFilename = null,
         ?string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null)
+        {
             $size = $stream->getSize();
         }
 

src/Router/tests/TestCore.php

Spacing +1 added lines, -1 removed lines

@@ -14,6 +14,6 @@
 
     public function callAction(string $controller, ?string $action = null, array $parameters = []): string
     {
-        return '@wrapped.' . $this->core->callAction($controller, $action, $parameters);
+        return '@wrapped.'.$this->core->callAction($controller, $action, $parameters);
     }
 }

src/Router/src/Exception/RouteNotFoundException.php

Spacing +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
         private readonly UriInterface $uri,
         int $code = 0,
         ?\Throwable $previous = null
-    ) {
-        parent::__construct(\sprintf('Unable to route `%s`.', (string) $uri), $code, $previous);
+    ){
+        parent::__construct(\sprintf('Unable to route `%s`.', (string)$uri), $code, $previous);
     }
 
     public function getUri(): UriInterface

src/Router/src/Router.php

Spacing +25 added lines, -25 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,13 +68,13 @@ 
         return $this->tracer->trace(
             name: 'Routing',
             callback: function (SpanInterface $span) use ($request) {
-                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 +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));
         }
 

Braces +51 added lines, -23 removed lines

@@ -67,14 +67,19 @@ 
 
         return $this->tracer->trace(
             name: 'Routing',
-            callback: function (SpanInterface $span) use ($request) {
-                try {
+            callback: function (SpanInterface $span) use ($request)
+            {
+                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 +114,8 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name]))
+        {
             return $this->routes[$name];
         }
 
@@ -118,7 +124,8 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default))
+        {
             return $this->routes + [null => $this->default];
         }
 
@@ -127,9 +134,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 +151,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 +164,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 +193,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 +219,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 +262,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));
         }
 

src/Router/src/Loader/LoaderRegistry.php

Spacing +4 added lines, -4 removed lines

@@ -16,15 +16,15 @@
      */
     public function __construct(array $loaders = [])
     {
-        foreach ($loaders as $loader) {
+        foreach ($loaders as $loader){
             $this->addLoader($loader);
         }
     }
 
-    public function resolve(mixed $resource, ?string $type = null): LoaderInterface|false
+    public function resolve(mixed $resource, ?string $type = null): LoaderInterface | false
     {
-        foreach ($this->loaders as $loader) {
-            if ($loader->supports($resource, $type)) {
+        foreach ($this->loaders as $loader){
+            if ($loader->supports($resource, $type)){
                 return $loader;
             }
         }

Braces +6 added lines, -3 removed lines

@@ -16,15 +16,18 @@
      */
     public function __construct(array $loaders = [])
     {
-        foreach ($loaders as $loader) {
+        foreach ($loaders as $loader)
+        {
             $this->addLoader($loader);
         }
     }
 
     public function resolve(mixed $resource, ?string $type = null): LoaderInterface|false
     {
-        foreach ($this->loaders as $loader) {
-            if ($loader->supports($resource, $type)) {
+        foreach ($this->loaders as $loader)
+        {
+            if ($loader->supports($resource, $type))
+            {
                 return $loader;
             }
         }

src/Router/src/Loader/DelegatingLoader.php

Spacing +2 added lines, -2 removed lines

@@ -10,12 +10,12 @@
 {
     public function __construct(
         private readonly LoaderRegistryInterface $registry
-    ) {
+    ){
     }
 
     public function load(mixed $resource, ?string $type = null): mixed
     {
-        if (false === $loader = $this->registry->resolve($resource, $type)) {
+        if (false === $loader = $this->registry->resolve($resource, $type)){
             throw new LoaderLoadException(\sprintf('Loader for type [%s] not found.', $type ?? ''));
         }
 

Braces +2 added lines, -1 removed lines

@@ -15,7 +15,8 @@
 
     public function load(mixed $resource, ?string $type = null): mixed
     {
-        if (false === $loader = $this->registry->resolve($resource, $type)) {
+        if (false === $loader = $this->registry->resolve($resource, $type))
+        {
             throw new LoaderLoadException(\sprintf('Loader for type [%s] not found.', $type ?? ''));
         }
 

src/Router/src/Loader/PhpFileLoader.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     public function __construct(
         private readonly FactoryInterface $factory,
         private readonly ResolverInterface $resolver,
-    ) {
+    ){
     }
 
     /**
@@ -24,7 +24,7 @@ 
      */
     public function load(mixed $resource, ?string $type = null): RouteCollection
     {
-        if (!\file_exists($resource)) {
+        if (!\file_exists($resource)){
             throw new LoaderLoadException(\sprintf('File [%s] does not exist.', $resource));
         }
 

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
      */
     public function load(mixed $resource, ?string $type = null): RouteCollection
     {
-        if (!\file_exists($resource)) {
+        if (!\file_exists($resource))
+        {
             throw new LoaderLoadException(\sprintf('File [%s] does not exist.', $resource));
         }
 

src/Router/src/Loader/Configurator/RoutingConfigurator.php

Spacing +4 added lines, -4 removed lines

@@ -14,19 +14,19 @@
     public function __construct(
         private readonly RouteCollection $collection,
         private readonly LoaderInterface $loader
-    ) {
+    ){
     }
 
-    public function import(string|array $resource, ?string $type = null): ImportConfigurator
+    public function import(string | array $resource, ?string $type = null): ImportConfigurator
     {
         $imported = $this->loader->load($resource, $type) ?: [];
 
-        if (!\is_array($imported)) {
+        if (!\is_array($imported)){
             return new ImportConfigurator($this->collection, $imported);
         }
 
         $mergedCollection = new RouteCollection();
-        foreach ($imported as $subCollection) {
+        foreach ($imported as $subCollection){
             $mergedCollection->addCollection($subCollection);
         }
 

src/Router/src/Loader/LoaderRegistryInterface.php

Spacing +1 added lines, -1 removed lines

@@ -11,5 +11,5 @@
      *
      * @param string|null $type The resource type or null if unknown
      */
-    public function resolve(mixed $resource, ?string $type = null): LoaderInterface|false;
+    public function resolve(mixed $resource, ?string $type = null): LoaderInterface | false;
 }