spiral / framework

src/Interceptors/src/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/Interceptors/src/Context/CallContext.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         private TargetInterface $target,
         private array $arguments = [],
         array $attributes = [],
-    ) {
+    ){
         $this->attributes = $attributes;
     }
 

src/Interceptors/src/Context/TargetInterface.php

Spacing +1 added lines, -1 removed lines

@@ -58,5 +58,5 @@
      *
      * @return callable|array{class-string, non-empty-string}|null
      */
-    public function getCallable(): callable|array|null;
+    public function getCallable(): callable | array | null;
 }

src/Hmvc/src/CompatiblePipelineBuilder.php

Braces +2 added lines, -1 removed lines

@@ -31,7 +31,8 @@
     public function withInterceptors(CoreInterceptorInterface|InterceptorInterface ...$interceptors): static
     {
         $clone = clone $this;
-        foreach ($interceptors as $interceptor) {
+        foreach ($interceptors as $interceptor)
+        {
             $clone->pipeline->addInterceptor($interceptor);
         }
 

Spacing +3 added lines, -3 removed lines

@@ -23,17 +23,17 @@
         $this->pipeline = new InterceptorPipeline($dispatcher);
     }
 
-    public function withInterceptors(CoreInterceptorInterface|InterceptorInterface ...$interceptors): static
+    public function withInterceptors(CoreInterceptorInterface | InterceptorInterface ...$interceptors): static
     {
         $clone = clone $this;
-        foreach ($interceptors as $interceptor) {
+        foreach ($interceptors as $interceptor){
             $clone->pipeline->addInterceptor($interceptor);
         }
 
         return $clone;
     }
 
-    public function build(HandlerInterface|CoreInterface $handler): InterceptorPipeline
+    public function build(HandlerInterface | CoreInterface $handler): InterceptorPipeline
     {
         /** @psalm-suppress InvalidArgument */
         return $handler instanceof HandlerInterface

src/Hmvc/src/Exception/ControllerException.php

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 // Load the original class to make an alias
 use Spiral\Interceptors\Exception\TargetCallException;
 
-if (!\class_exists(TargetCallException::class)) {
+if (!\class_exists(TargetCallException::class)){
     /**
      * Unable to perform user action or find controller.
      *

Braces +2 added lines, -1 removed lines

@@ -7,7 +7,8 @@
 // Load the original class to make an alias
 use Spiral\Interceptors\Exception\TargetCallException;
 
-if (!\class_exists(TargetCallException::class)) {
+if (!\class_exists(TargetCallException::class))
+{
     /**
      * Unable to perform user action or find controller.
      *

src/Hmvc/src/Exception/InterceptorException.php

Braces +2 added lines, -1 removed lines

@@ -5,7 +5,8 @@
 namespace Spiral\Core\Exception;
 
 // Load the original class to make an alias
-if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)) {
+if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class))
+{
     /**
      * @deprecated will be removed in Spiral v4.0
      * Use {@see \Spiral\Interceptors\Exception\InterceptorException} instead.

Spacing +2 added lines, -2 removed lines

@@ -5,10 +5,10 @@
 namespace Spiral\Core\Exception;
 
 // Load the original class to make an alias
-if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)) {
+if (!\class_exists(\Spiral\Interceptors\Exception\InterceptorException::class)){
     /**
      * @deprecated will be removed in Spiral v4.0
      * Use {@see \Spiral\Interceptors\Exception\InterceptorException} instead.
      */
-    class InterceptorException extends \RuntimeException {}
+    class InterceptorException extends \RuntimeException{}
 }

src/Hmvc/src/AbstractCore.php

Braces +10 added lines, -4 removed lines

@@ -65,7 +65,8 @@ 
 
     protected function resolveArguments(\ReflectionMethod $method, array $parameters): array
     {
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter)
+        {
             $name = $parameter->getName();
             if (
                 \array_key_exists($name, $parameters) &&
@@ -86,9 +87,12 @@ 
      */
     private function invoke(?object $object, string $class, \ReflectionMethod $method, array $arguments): mixed
     {
-        try {
+        try
+        {
             $args = $this->resolveArguments($method, $arguments);
-        } catch (ArgumentResolvingException | InvalidArgumentException $e) {
+        }
+        catch (ArgumentResolvingException | InvalidArgumentException $e)
+        {
             throw new ControllerException(
                 \sprintf(
                     'Missing/invalid parameter %s of `%s`->`%s`',
@@ -99,7 +103,9 @@ 
                 ControllerException::BAD_ARGUMENT,
                 $e,
             );
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

Spacing +7 added lines, -7 removed lines

@@ -29,7 +29,7 @@ 
     public function __construct(
         /** @internal */
         #[Proxy] protected ContainerInterface $container,
-    ) {
+    ){
         // TODO: can we simplify this?
         // resolver is usually the container itself
         /** @psalm-suppress MixedAssignment */
@@ -37,7 +37,7 @@ 
             ? $container
             : $container
                 ->get(InvokerInterface::class)
-                ->invoke(static fn(#[Proxy] ResolverInterface $resolver): ResolverInterface => $resolver);
+                ->invoke(static fn(#[Proxy] ResolverInterface $resolver) : ResolverInterface => $resolver);
     }
 
     /**
@@ -65,13 +65,13 @@ 
 
     protected function resolveArguments(\ReflectionMethod $method, array $parameters): array
     {
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter){
             $name = $parameter->getName();
             if (
                 \array_key_exists($name, $parameters) &&
                 $parameters[$name] === null &&
                 $parameter->isDefaultValueAvailable()
-            ) {
+            ){
                 /** @psalm-suppress MixedAssignment */
                 $parameters[$name] = $parameter->getDefaultValue();
             }
@@ -86,9 +86,9 @@ 
      */
     private function invoke(?object $object, string $class, \ReflectionMethod $method, array $arguments): mixed
     {
-        try {
+        try{
             $args = $this->resolveArguments($method, $arguments);
-        } catch (ArgumentResolvingException | InvalidArgumentException $e) {
+        }catch (ArgumentResolvingException | InvalidArgumentException $e){
             throw new ControllerException(
                 \sprintf(
                     'Missing/invalid parameter %s of `%s`->`%s`',
@@ -99,7 +99,7 @@ 
                 ControllerException::BAD_ARGUMENT,
                 $e,
             );
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new ControllerException(
                 $e->getMessage(),
                 ControllerException::ERROR,

src/Interceptors/tests/Unit/Handler/AutowireHandlerTest.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
     public function createHandler(array $definitions = []): AutowireHandler
     {
         $container = new Container();
-        foreach ($definitions as $id => $definition) {
+        foreach ($definitions as $id => $definition){
             $container->bind($id, $definition);
         }
 

Braces +2 added lines, -1 removed lines

@@ -108,7 +108,8 @@
     public function createHandler(array $definitions = []): AutowireHandler
     {
         $container = new Container();
-        foreach ($definitions as $id => $definition) {
+        foreach ($definitions as $id => $definition)
+        {
             $container->bind($id, $definition);
         }
 

src/Views/src/ViewManager.php

Braces +16 added lines, -8 removed lines

@@ -28,15 +28,18 @@ 
             'namespaces' => $config->getNamespaces(),
         ]);
 
-        foreach ($this->config->getDependencies() as $dependency) {
+        foreach ($this->config->getDependencies() as $dependency)
+        {
             $this->addDependency($dependency->resolve($factory));
         }
 
-        foreach ($this->config->getEngines() as $engine) {
+        foreach ($this->config->getEngines() as $engine)
+        {
             $this->addEngine($engine->resolve($factory));
         }
 
-        if ($this->config->isCacheEnabled()) {
+        if ($this->config->isCacheEnabled())
+        {
             $this->cache = new ViewCache();
         }
     }
@@ -93,7 +96,8 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context)
+        {
             $engine->reset($path, $context);
             $engine->compile($path, $context);
         }
@@ -110,7 +114,8 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context)
+        {
             $engine->reset($path, $context);
         }
     }
@@ -122,7 +127,8 @@ 
      */
     public function get(string $path): ViewInterface
     {
-        if ($this->cache?->has($this->context, $path)) {
+        if ($this->cache?->has($this->context, $path))
+        {
             return $this->cache->get($this->context, $path);
         }
 
@@ -150,8 +156,10 @@ 
      */
     private function findEngine(string $path): EngineInterface
     {
-        foreach ($this->engines as $engine) {
-            if ($engine->getLoader()->exists($path)) {
+        foreach ($this->engines as $engine)
+        {
+            if ($engine->getLoader()->exists($path))
+            {
                 return $engine;
             }
         }

Spacing +9 added lines, -9 removed lines

@@ -22,21 +22,21 @@ 
         private readonly GlobalVariablesInterface $globalVariables,
         FactoryInterface $factory,
         ?ContextInterface $context = null,
-    ) {
+    ){
         $this->context = $context ?? new ViewContext();
         $this->loader = $factory->make(LoaderInterface::class, [
             'namespaces' => $config->getNamespaces(),
         ]);
 
-        foreach ($this->config->getDependencies() as $dependency) {
+        foreach ($this->config->getDependencies() as $dependency){
             $this->addDependency($dependency->resolve($factory));
         }
 
-        foreach ($this->config->getEngines() as $engine) {
+        foreach ($this->config->getEngines() as $engine){
             $this->addEngine($engine->resolve($factory));
         }
 
-        if ($this->config->isCacheEnabled()) {
+        if ($this->config->isCacheEnabled()){
             $this->cache = new ViewCache();
         }
     }
@@ -92,7 +92,7 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context){
             $engine->reset($path, $context);
             $engine->compile($path, $context);
         }
@@ -109,7 +109,7 @@ 
 
         // Rotate all possible context variants and warm up cache
         $generator = new ContextGenerator($this->context);
-        foreach ($generator->generate() as $context) {
+        foreach ($generator->generate() as $context){
             $engine->reset($path, $context);
         }
     }
@@ -121,7 +121,7 @@ 
      */
     public function get(string $path): ViewInterface
     {
-        if ($this->cache?->has($this->context, $path)) {
+        if ($this->cache?->has($this->context, $path)){
             return $this->cache->get($this->context, $path);
         }
 
@@ -148,8 +148,8 @@ 
      */
     private function findEngine(string $path): EngineInterface
     {
-        foreach ($this->engines as $engine) {
-            if ($engine->getLoader()->exists($path)) {
+        foreach ($this->engines as $engine){
+            if ($engine->getLoader()->exists($path)){
                 return $engine;
             }
         }