spiral / framework

src/Interceptors/tests/Unit/Context/TargetTest.php

Braces +2 added lines, -1 removed lines

@@ -124,7 +124,8 @@
         $reflection = $target->getReflection();
         self::assertSame($hasReflection, $reflection !== null);
         self::assertNull($target->getObject());
-        if ($hasReflection) {
+        if ($hasReflection)
+        {
             self::assertInstanceOf(\ReflectionMethod::class, $reflection);
             self::assertSame($action, $reflection->getName());
         }

Spacing +11 added lines, -11 removed lines

@@ -36,7 +36,7 @@ 
         $target = Target::fromReflectionFunction($reflection, ['print_r-path']);
 
         self::assertSame($reflection, $target->getReflection());
-        self::assertSame('print_r-path', (string) $target);
+        self::assertSame('print_r-path', (string)$target);
         self::assertNull($target->getObject());
     }
 
@@ -45,7 +45,7 @@ 
         $target = Target::fromClosure(\print_r(...), ['print_r-path']);
 
         self::assertNotNull($target->getReflection());
-        self::assertSame('print_r-path', (string) $target);
+        self::assertSame('print_r-path', (string)$target);
         self::assertNull($target->getObject());
     }
 
@@ -54,7 +54,7 @@ 
         $target = Target::fromClosure($this->{__FUNCTION__}(...), ['print_r-path']);
 
         self::assertNotNull($target->getReflection());
-        self::assertSame('print_r-path', (string) $target);
+        self::assertSame('print_r-path', (string)$target);
         self::assertNull($target->getObject());
     }
 
@@ -65,7 +65,7 @@ 
         $target = Target::fromReflectionMethod($reflection, self::class);
 
         self::assertSame($reflection, $target->getReflection());
-        self::assertSame(self::class . '->' . __FUNCTION__, (string) $target);
+        self::assertSame(self::class.'->'.__FUNCTION__, (string)$target);
         self::assertNull($target->getObject());
     }
 
@@ -76,7 +76,7 @@ 
         $target = Target::fromReflectionMethod($reflection, $this);
 
         self::assertSame($reflection, $target->getReflection());
-        self::assertSame(self::class . '->' . __FUNCTION__, (string) $target);
+        self::assertSame(self::class.'->'.__FUNCTION__, (string)$target);
         self::assertNotNull($target->getObject());
     }
 
@@ -89,10 +89,10 @@ 
         // Immutability
         self::assertNotSame($target, $target2);
         self::assertSame(['bar', 'baz'], $target2->getPath());
-        self::assertSame('bar.baz', (string) $target2);
+        self::assertSame('bar.baz', (string)$target2);
         // First target is not changed
         self::assertSame(['foo', 'bar', 'baz'], $target->getPath());
-        self::assertSame($str, (string) $target);
+        self::assertSame($str, (string)$target);
     }
 
     #[DataProvider('providePathChunks')]
@@ -102,7 +102,7 @@ 
         $target = Target::fromPathString($str, $separator);
 
         self::assertSame($chunks, $target->getPath());
-        self::assertSame($str, (string) $target);
+        self::assertSame($str, (string)$target);
     }
 
     #[DataProvider('providePathChunks')]
@@ -112,7 +112,7 @@ 
         $target = Target::fromPathArray($chunks, $separator);
 
         self::assertSame($chunks, $target->getPath());
-        self::assertSame($str, (string) $target);
+        self::assertSame($str, (string)$target);
     }
 
     #[DataProvider('providePairs')]
@@ -124,7 +124,7 @@ 
         $reflection = $target->getReflection();
         self::assertSame($hasReflection, $reflection !== null);
         self::assertNull($target->getObject());
-        if ($hasReflection) {
+        if ($hasReflection){
             self::assertInstanceOf(\ReflectionMethod::class, $reflection);
             self::assertSame($action, $reflection->getName());
         }
@@ -148,7 +148,7 @@ 
         $target = Target::fromPathString($str);
 
         self::assertSame(['foo', 'bar', 'baz'], $target->getPath());
-        self::assertSame($str, (string) $target);
+        self::assertSame($str, (string)$target);
     }
 
     public function testPrivateConstructor(): void

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

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
     public function testInterceptorCallingEventShouldBeDispatched(): void
     {
         $context = $this->createPathContext(['test', 'test2']);
-        $interceptor = new class implements InterceptorInterface {
+        $interceptor = new class implements InterceptorInterface{
             public function intercept(CallContextInterface $context, HandlerInterface $handler): mixed
             {
                 return null;
@@ -39,7 +39,7 @@ 
         $pipeline = $this->createPipeline(interceptors: [$interceptor], dispatcher: $dispatcher);
 
         $pipeline->withHandler(
-            new class implements HandlerInterface {
+            new class implements HandlerInterface{
                 public function handle(CallContextInterface $context): mixed
                 {
                     return null;
@@ -125,8 +125,8 @@ 
      */
     private function createPipeline(
         array $interceptors = [],
-        HandlerInterface|null $lastHandler = null,
-        EventDispatcherInterface|null $dispatcher = null,
+        HandlerInterface | null $lastHandler = null,
+        EventDispatcherInterface | null $dispatcher = null,
     ): InterceptorPipeline {
         $pipeline = new InterceptorPipeline($dispatcher);
 

Braces +4 added lines, -2 removed lines

@@ -20,7 +20,8 @@ 
     public function testInterceptorCallingEventShouldBeDispatched(): void
     {
         $context = $this->createPathContext(['test', 'test2']);
-        $interceptor = new class implements InterceptorInterface {
+        $interceptor = new class implements InterceptorInterface
+        {
             public function intercept(CallContextInterface $context, HandlerInterface $handler): mixed
             {
                 return null;
@@ -39,7 +40,8 @@ 
         $pipeline = $this->createPipeline(interceptors: [$interceptor], dispatcher: $dispatcher);
 
         $pipeline->withHandler(
-            new class implements HandlerInterface {
+            new class implements HandlerInterface
+            {
                 public function handle(CallContextInterface $context): mixed
                 {
                     return null;

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,