spiral / framework

src/Core/src/Exception/Container/RecursiveProxyException.php

Spacing +3 added lines, -3 removed lines

@@ -13,11 +13,11 @@
     public function __construct(
         public readonly string $alias,
         public readonly ?string $bindingScope = null,
-        public readonly ?array $callingScope = null,
-    ) {
+        public readonly ? array $callingScope = null,
+    ){
         $message = "Recursive proxy detected for `$alias`.";
         $bindingScope === null or $message .= "\nBinding scope: `$bindingScope`.";
-        $callingScope === null or $message .= "\nCalling scope: `" . \implode('.', $callingScope) . '`.';
+        $callingScope === null or $message .= "\nCalling scope: `".\implode('.', $callingScope).'`.';
         parent::__construct($message);
     }
 }

src/Core/src/Config/Proxy.php

Spacing +2 added lines, -2 removed lines

@@ -17,8 +17,8 @@
     public function __construct(
         protected readonly string $interface,
         public readonly bool $singleton = false,
-        public readonly ?\Closure $fallbackFactory = null,
-    ) {
+        public readonly ? \Closure $fallbackFactory = null,
+    ){
         \interface_exists($interface) or throw new \InvalidArgumentException(
             "Interface `{$interface}` does not exist.",
         );

src/Core/src/Internal/Proxy/Resolver.php

Braces +16 added lines, -7 removed lines

@@ -24,12 +24,15 @@ 
     ): object {
         $c ??= ContainerScope::getContainer() ?? throw new ContainerException('Proxy is out of scope.');
 
-        try {
+        try
+        {
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, $context) ?? throw new ContainerException(
                 'Resolved `null` from the container.',
             );
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $scope = self::getScope($c);
             throw new ContainerException(
                 $scope === null
@@ -39,7 +42,8 @@ 
             );
         }
 
-        if (!Proxy::isProxy($result)) {
+        if (!Proxy::isProxy($result))
+        {
             return $result;
         }
 
@@ -48,10 +52,13 @@ 
          * to try to get the instance from the Proxy Fallback Factory.
          * If there is no the Proxy Fallback Factory, {@see RecursiveProxyException} will be thrown.
          */
-        try {
+        try
+        {
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, new RetryContext($context));
-        } catch (RecursiveProxyException $e) {
+        }
+        catch (RecursiveProxyException $e)
+        {
             throw new RecursiveProxyException($e->alias, $e->bindingScope, self::getScope($c));
         }
 
@@ -66,8 +73,10 @@ 
      */
     private static function getScope(ContainerInterface $c): ?array
     {
-        if (!$c instanceof Container) {
-            if (!Proxy::isProxy($c)) {
+        if (!$c instanceof Container)
+        {
+            if (!Proxy::isProxy($c))
+            {
                 return null;
             }
 

Spacing +10 added lines, -10 removed lines

@@ -19,27 +19,27 @@ 
 {
     public static function resolve(
         string $alias,
-        \Stringable|string|null $context = null,
+        \Stringable | string | null $context = null,
         ?ContainerInterface $c = null,
     ): object {
         $c ??= ContainerScope::getContainer() ?? throw new ContainerException('Proxy is out of scope.');
 
-        try {
+        try{
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, $context) ?? throw new ContainerException(
                 'Resolved `null` from the container.',
             );
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $scope = self::getScope($c);
             throw new ContainerException(
                 $scope === null
                     ? "Unable to resolve `{$alias}` in a Proxy."
                     : \sprintf('Unable to resolve `%s` in a Proxy in `%s` scope.', $alias, \implode('.', $scope)),
-                previous: $e,
+                previous : $e,
             );
         }
 
-        if (!Proxy::isProxy($result)) {
+        if (!Proxy::isProxy($result)){
             return $result;
         }
 
@@ -48,10 +48,10 @@ 
          * to try to get the instance from the Proxy Fallback Factory.
          * If there is no the Proxy Fallback Factory, {@see RecursiveProxyException} will be thrown.
          */
-        try {
+        try{
             /** @psalm-suppress TooManyArguments */
             $result = $c->get($alias, new RetryContext($context));
-        } catch (RecursiveProxyException $e) {
+        }catch (RecursiveProxyException $e){
             throw new RecursiveProxyException($e->alias, $e->bindingScope, self::getScope($c));
         }
 
@@ -66,8 +66,8 @@ 
      */
     private static function getScope(ContainerInterface $c): ?array
     {
-        if (!$c instanceof Container) {
-            if (!Proxy::isProxy($c)) {
+        if (!$c instanceof Container){
+            if (!Proxy::isProxy($c)){
                 return null;
             }
 
@@ -75,7 +75,7 @@ 
         }
 
         return \array_reverse(\array_map(
-            static fn(?string $name): string => $name ?? 'null',
+            static fn(?string $name) : string => $name ?? 'null',
             Introspector::scopeNames($c),
         ));
     }

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

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
 {
     public function init(HttpBootloader $http): void
     {
-        foreach ($this->globalMiddleware() as $middleware) {
+        foreach ($this->globalMiddleware() as $middleware){
             $http->addMiddleware($middleware);
         }
     }
@@ -55,11 +55,11 @@ 
 
     private function registerMiddlewareGroups(BinderInterface $binder, array $groups): void
     {
-        foreach ($groups as $group => $middleware) {
+        foreach ($groups as $group => $middleware){
             $binder
                 ->getBinder('http')
                 ->bind(
-                    'middleware:' . $group,
+                    'middleware:'.$group,
                     static function (FactoryInterface $factory) use ($middleware): LazyPipeline {
                         return $factory->make(LazyPipeline::class)->withAddedMiddleware(...$middleware);
                     },
@@ -69,8 +69,8 @@ 
 
     private function registerMiddlewareForRouteGroups(GroupRegistry $registry, array $groups): void
     {
-        foreach ($groups as $group) {
-            $registry->getGroup($group)->addMiddleware('middleware:' . $group);
+        foreach ($groups as $group){
+            $registry->getGroup($group)->addMiddleware('middleware:'.$group);
         }
     }
 }

src/Router/src/Traits/PipelineTrait.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
 {
     use ContainerTrait;
 
-    protected Pipeline|LazyPipeline|null $pipeline = null;
+    protected Pipeline | LazyPipeline | null $pipeline = null;
 
     /** @psalm-var array<array-key, MiddlewareType> */
     protected array $middleware = [];
@@ -44,16 +44,16 @@ 
         $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 +80,11 @@ 
     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 +97,11 @@ 
     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);
         }
     }

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/tests/Fixtures/UserContext.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 
 final class UserContext
 {
-    private function __construct() {}
+    private function __construct(){}
 
     public static function create(): self
     {

Braces +3 added lines, -1 removed lines

@@ -6,7 +6,9 @@
 
 final class UserContext
 {
-    private function __construct() {}
+    private function __construct()
+    {
+}
 
     public static function create(): self
     {

src/Router/tests/Fixtures/UserContextController.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 {
     public function __construct(
         private readonly UserContext $scope,
-    ) {}
+    ){}
 
     public function scope(): string
     {

Braces +2 added lines, -1 removed lines

@@ -8,7 +8,8 @@
 {
     public function __construct(
         private readonly UserContext $scope,
-    ) {}
+    ) {
+}
 
     public function scope(): string
     {

src/Cache/src/Event/KeyDeleting.php

Spacing +1 added lines, -1 removed lines

@@ -7,4 +7,4 @@
 /**
  * Triggered before cache item is deleted.
  */
-final class KeyDeleting extends CacheEvent {}
+final class KeyDeleting extends CacheEvent{}

Braces +3 added lines, -1 removed lines

@@ -7,4 +7,6 @@
 /**
  * Triggered before cache item is deleted.
  */
-final class KeyDeleting extends CacheEvent {}
+final class KeyDeleting extends CacheEvent
+{
+}

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;
             }
         }