spiral / framework

src/Attributes/tests/TestCase.php

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
      */
     protected function iterableToArray(iterable $iterable): array
     {
-        if ($iterable instanceof \Traversable) {
+        if ($iterable instanceof \Traversable){
             return \iterator_to_array($iterable, false);
         }
 
@@ -49,7 +49,7 @@ 
     {
         $instance = new $class();
 
-        foreach ($fields as $field => $value) {
+        foreach ($fields as $field => $value){
             $instance->$field = $value;
         }
 
@@ -65,12 +65,12 @@ 
      * @param string $message
      *   Expected message.
      */
-    public function expectExceptionMessageEquals(string $message) {
+    public function expectExceptionMessageEquals(string $message){
         // The "contains" check produces a more readable message.
         $this->expectExceptionMessage($message);
         // The regex check is the only way to check exact equality.
         $this->expectExceptionMessageMatches(
-            '@^' . preg_quote($message, '@') . '$@'
+            '@^'.preg_quote($message, '@').'$@'
         );
     }
 
@@ -87,7 +87,7 @@ 
      * @throws \ReflectionException
      *   The class does not exist.
      */
-    public static function assertExceptionSource(\Throwable $e, string $class, string $search) {
+    public static function assertExceptionSource(\Throwable $e, string $class, string $search){
         $rc = new \ReflectionClass($class);
         $file = $rc->getFileName();
         self::assertSame($file, $e->getFile(), 'Exception file');

Braces +8 added lines, -4 removed lines

@@ -24,7 +24,8 @@ 
      */
     protected function iterableToArray(iterable $iterable): array
     {
-        if ($iterable instanceof \Traversable) {
+        if ($iterable instanceof \Traversable)
+        {
             return \iterator_to_array($iterable, false);
         }
 
@@ -49,7 +50,8 @@ 
     {
         $instance = new $class();
 
-        foreach ($fields as $field => $value) {
+        foreach ($fields as $field => $value)
+        {
             $instance->$field = $value;
         }
 
@@ -65,7 +67,8 @@ 
      * @param string $message
      *   Expected message.
      */
-    public function expectExceptionMessageEquals(string $message) {
+    public function expectExceptionMessageEquals(string $message)
+    {
         // The "contains" check produces a more readable message.
         $this->expectExceptionMessage($message);
         // The regex check is the only way to check exact equality.
@@ -87,7 +90,8 @@ 
      * @throws \ReflectionException
      *   The class does not exist.
      */
-    public static function assertExceptionSource(\Throwable $e, string $class, string $search) {
+    public static function assertExceptionSource(\Throwable $e, string $class, string $search)
+    {
         $rc = new \ReflectionClass($class);
         $file = $rc->getFileName();
         self::assertSame($file, $e->getFile(), 'Exception file');

src/Attributes/tests/Instantiator/NamedArgumentsInstantiatorTestCase.php

Spacing +24 added lines, -24 removed lines

@@ -73,23 +73,23 @@ 
 
     public function testUnknownSequentialAfterNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }else{
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_POSITIONAL_AFTER_NAMED */
         $this->expectExceptionMessageEquals('Cannot use positional argument after named argument');
 
-        try {
+        try{
             $this->new($class = NamedArgumentConstructorFixture::class, [
                 'a' => 'A',
                 5 => 'five',
             ]);
-        } catch (\Throwable $e) {
-            if (PHP_VERSION_ID < 80000) {
+        }catch (\Throwable $e){
+            if (PHP_VERSION_ID < 80000){
                 self::assertExceptionSource($e, $class, 'function __construct');
-            } else {
+            }else{
                 self::assertExceptionSource($e, $class, 'class ');
             }
             throw $e;
@@ -98,9 +98,9 @@ 
 
     public function testKnownSequentialAfterNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }else{
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_POSITIONAL_AFTER_NAMED */
@@ -123,15 +123,15 @@ 
             )
         );
 
-        try {
+        try{
             $this->new($class = NamedRequiredArgumentConstructorFixture::class, [
                 'a',
                 'c' => 'C',
             ]);
-        } catch (\Throwable $e) {
-            if (PHP_VERSION_ID < 80000) {
+        }catch (\Throwable $e){
+            if (PHP_VERSION_ID < 80000){
                 self::assertExceptionSource($e, $class, 'function __construct');
-            } else {
+            }else{
                 self::assertExceptionSource($e, $class, 'class ');
             }
             throw $e;
@@ -140,22 +140,22 @@ 
 
     public function testUnknownArg()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }else{
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_UNKNOWN_ARGUMENT */
         $this->expectExceptionMessageEquals('Unknown named parameter $d');
 
-        try {
+        try{
             $this->new($class = NamedArgumentConstructorFixture::class, [
                 'd' => 'D',
             ]);
-        } catch (\Throwable $e) {
-            if (PHP_VERSION_ID < 80000) {
+        }catch (\Throwable $e){
+            if (PHP_VERSION_ID < 80000){
                 self::assertExceptionSource($e, $class, 'function __construct');
-            } else {
+            }else{
                 self::assertExceptionSource($e, $class, 'class ');
             }
             throw $e;
@@ -164,9 +164,9 @@ 
 
     public function testOverwriteArg()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }else{
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_OVERWRITE_ARGUMENT */
@@ -195,9 +195,9 @@ 
 
     public function testVariadicPositionalAfterNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }else{
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_POSITIONAL_AFTER_NAMED */
@@ -213,7 +213,7 @@ 
 
     public function testVariadicMixed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
             /* @see NamedArgumentsInstantiator::ERROR_NAMED_ARG_TO_VARIADIC */
             $this->expectExceptionMessageEquals('Cannot pass named argument $x to variadic parameter ...$args in PHP < 8');
@@ -234,7 +234,7 @@ 
 
     public function testVariadicNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000){
             $this->expectException(\BadMethodCallException::class);
             /* @see NamedArgumentsInstantiator::ERROR_NAMED_ARG_TO_VARIADIC */
             $this->expectExceptionMessageEquals('Cannot pass named argument $x to variadic parameter ...$args in PHP < 8');

Braces +59 added lines, -24 removed lines

@@ -73,23 +73,32 @@ 
 
     public function testUnknownSequentialAfterNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }
+        else
+        {
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_POSITIONAL_AFTER_NAMED */
         $this->expectExceptionMessageEquals('Cannot use positional argument after named argument');
 
-        try {
+        try
+        {
             $this->new($class = NamedArgumentConstructorFixture::class, [
                 'a' => 'A',
                 5 => 'five',
             ]);
-        } catch (\Throwable $e) {
-            if (PHP_VERSION_ID < 80000) {
+        }
+        catch (\Throwable $e)
+        {
+            if (PHP_VERSION_ID < 80000)
+            {
                 self::assertExceptionSource($e, $class, 'function __construct');
-            } else {
+            }
+            else
+            {
                 self::assertExceptionSource($e, $class, 'class ');
             }
             throw $e;
@@ -98,9 +107,12 @@ 
 
     public function testKnownSequentialAfterNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }
+        else
+        {
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_POSITIONAL_AFTER_NAMED */
@@ -123,15 +135,21 @@ 
             )
         );
 
-        try {
+        try
+        {
             $this->new($class = NamedRequiredArgumentConstructorFixture::class, [
                 'a',
                 'c' => 'C',
             ]);
-        } catch (\Throwable $e) {
-            if (PHP_VERSION_ID < 80000) {
+        }
+        catch (\Throwable $e)
+        {
+            if (PHP_VERSION_ID < 80000)
+            {
                 self::assertExceptionSource($e, $class, 'function __construct');
-            } else {
+            }
+            else
+            {
                 self::assertExceptionSource($e, $class, 'class ');
             }
             throw $e;
@@ -140,22 +158,31 @@ 
 
     public function testUnknownArg()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }
+        else
+        {
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_UNKNOWN_ARGUMENT */
         $this->expectExceptionMessageEquals('Unknown named parameter $d');
 
-        try {
+        try
+        {
             $this->new($class = NamedArgumentConstructorFixture::class, [
                 'd' => 'D',
             ]);
-        } catch (\Throwable $e) {
-            if (PHP_VERSION_ID < 80000) {
+        }
+        catch (\Throwable $e)
+        {
+            if (PHP_VERSION_ID < 80000)
+            {
                 self::assertExceptionSource($e, $class, 'function __construct');
-            } else {
+            }
+            else
+            {
                 self::assertExceptionSource($e, $class, 'class ');
             }
             throw $e;
@@ -164,9 +191,12 @@ 
 
     public function testOverwriteArg()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }
+        else
+        {
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_OVERWRITE_ARGUMENT */
@@ -195,9 +225,12 @@ 
 
     public function testVariadicPositionalAfterNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
-        } else {
+        }
+        else
+        {
             $this->expectException(\Error::class);
         }
         /* @see NamedArgumentsInstantiator::ERROR_POSITIONAL_AFTER_NAMED */
@@ -213,7 +246,8 @@ 
 
     public function testVariadicMixed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
             /* @see NamedArgumentsInstantiator::ERROR_NAMED_ARG_TO_VARIADIC */
             $this->expectExceptionMessageEquals('Cannot pass named argument $x to variadic parameter ...$args in PHP < 8');
@@ -234,7 +268,8 @@ 
 
     public function testVariadicNamed()
     {
-        if (PHP_VERSION_ID < 80000) {
+        if (PHP_VERSION_ID < 80000)
+        {
             $this->expectException(\BadMethodCallException::class);
             /* @see NamedArgumentsInstantiator::ERROR_NAMED_ARG_TO_VARIADIC */
             $this->expectExceptionMessageEquals('Cannot pass named argument $x to variadic parameter ...$args in PHP < 8');

src/Attributes/src/Internal/Instantiator/NamedArgumentsInstantiator.php

Spacing +23 added lines, -23 removed lines

@@ -49,17 +49,17 @@ 
      */
     public function instantiate(\ReflectionClass $attr, array $arguments, \Reflector $context = null): object
     {
-        if ($this->isNamedArgumentsSupported()) {
-            try {
+        if ($this->isNamedArgumentsSupported()){
+            try{
                 return $attr->newInstanceArgs($arguments);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 throw Exception::withLocation($e, $attr->getFileName(), $attr->getStartLine());
             }
         }
 
         $constructor = $this->getConstructor($attr);
 
-        if ($constructor === null) {
+        if ($constructor === null){
             return $attr->newInstanceWithoutConstructor();
         }
 
@@ -78,9 +78,9 @@ 
      */
     private function resolveParameters(\ReflectionClass $ctx, \ReflectionMethod $constructor, array $arguments): array
     {
-        try {
+        try{
             return $this->doResolveParameters($ctx, $constructor, $arguments);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw Exception::withLocation($e, $constructor->getFileName(), $constructor->getStartLine());
         }
     }
@@ -93,16 +93,16 @@ 
     {
         $namedArgsBegin = $this->analyzeKeys($arguments);
 
-        if ($namedArgsBegin === null) {
+        if ($namedArgsBegin === null){
             // Only numeric / positional keys exist.
             return $arguments;
         }
 
-        if ($namedArgsBegin === 0) {
+        if ($namedArgsBegin === 0){
             // Only named keys exist.
             $passed = [];
             $named = $arguments;
-        } else {
+        }else{
             // Numeric/positional keys followed by named keys.
             // No need to preserve numeric keys.
             $passed = array_slice($arguments, 0, $namedArgsBegin);
@@ -138,11 +138,11 @@ 
         $arguments = array_merge($arguments);
 
         $i = 0;
-        foreach ($arguments as $k => $_) {
-            if ($k !== $i) {
+        foreach ($arguments as $k => $_){
+            if ($k !== $i){
                 // This must be a string key.
                 // Any further numeric keys are illegal.
-                if (\array_key_exists($i, $arguments)) {
+                if (\array_key_exists($i, $arguments)){
                     throw new \BadMethodCallException(self::ERROR_POSITIONAL_AFTER_NAMED);
                 }
                 return $i;
@@ -179,25 +179,25 @@ 
     {
         // Analyze parameters.
         $n = count($parameters);
-        if ($n > 0 && end($parameters)->isVariadic()) {
+        if ($n > 0 && end($parameters)->isVariadic()){
             $variadicParameter = end($parameters);
             // Don't include the variadic parameter in the mapping process.
             --$n;
-        } else {
+        }else{
             $variadicParameter = null;
         }
 
         // Process parameters that are not already filled with positional args.
         // This loop will do nothing if $namedArgsBegin >= $n. That's ok.
-        for ($i = $namedArgsBegin; $i < $n; ++$i) {
+        for ($i = $namedArgsBegin; $i < $n; ++$i){
             $parameter = $parameters[$i];
             $k = $parameter->getName();
-            if (array_key_exists($k, $named)) {
+            if (array_key_exists($k, $named)){
                 $passed[] = $named[$k];
                 unset($named[$k]);
-            } elseif ($parameter->isDefaultValueAvailable()) {
+            } elseif ($parameter->isDefaultValueAvailable()){
                 $passed[] = $parameter->getDefaultValue();
-            } else {
+            }else{
                 $message = \vsprintf(self::ERROR_ARGUMENT_NOT_PASSED, [
                     $ctx->getName(),
                     $parameter->getPosition() + 1,
@@ -208,7 +208,7 @@ 
             }
         }
 
-        if ($named === []) {
+        if ($named === []){
             // No unknown argument names exist.
             return $passed;
         }
@@ -218,11 +218,11 @@ 
         $badArgName = key($named);
 
         // Check collision with positional arguments.
-        foreach ($parameters as $i => $parameter) {
-            if ($i >= $namedArgsBegin) {
+        foreach ($parameters as $i => $parameter){
+            if ($i >= $namedArgsBegin){
                 break;
             }
-            if ($parameter->getName() === $badArgName) {
+            if ($parameter->getName() === $badArgName){
                 // The named argument overwrites a positional argument.
                 $message = \sprintf(self::ERROR_OVERWRITE_ARGUMENT, $badArgName);
                 throw new \BadMethodCallException($message);
@@ -230,7 +230,7 @@ 
         }
 
         // Special handling if a variadic parameter is present.
-        if ($variadicParameter !== null) {
+        if ($variadicParameter !== null){
             // The last parameter is variadic.
             // Since PHP 8+, variadic parameters can consume named arguments.
             // However, this code only runs if PHP < 8.

Braces +52 added lines, -23 removed lines

@@ -49,17 +49,22 @@ 
      */
     public function instantiate(\ReflectionClass $attr, array $arguments, \Reflector $context = null): object
     {
-        if ($this->isNamedArgumentsSupported()) {
-            try {
+        if ($this->isNamedArgumentsSupported())
+        {
+            try
+            {
                 return $attr->newInstanceArgs($arguments);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 throw Exception::withLocation($e, $attr->getFileName(), $attr->getStartLine());
             }
         }
 
         $constructor = $this->getConstructor($attr);
 
-        if ($constructor === null) {
+        if ($constructor === null)
+        {
             return $attr->newInstanceWithoutConstructor();
         }
 
@@ -78,9 +83,12 @@ 
      */
     private function resolveParameters(\ReflectionClass $ctx, \ReflectionMethod $constructor, array $arguments): array
     {
-        try {
+        try
+        {
             return $this->doResolveParameters($ctx, $constructor, $arguments);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw Exception::withLocation($e, $constructor->getFileName(), $constructor->getStartLine());
         }
     }
@@ -93,16 +101,20 @@ 
     {
         $namedArgsBegin = $this->analyzeKeys($arguments);
 
-        if ($namedArgsBegin === null) {
+        if ($namedArgsBegin === null)
+        {
             // Only numeric / positional keys exist.
             return $arguments;
         }
 
-        if ($namedArgsBegin === 0) {
+        if ($namedArgsBegin === 0)
+        {
             // Only named keys exist.
             $passed = [];
             $named = $arguments;
-        } else {
+        }
+        else
+        {
             // Numeric/positional keys followed by named keys.
             // No need to preserve numeric keys.
             $passed = array_slice($arguments, 0, $namedArgsBegin);
@@ -138,11 +150,14 @@ 
         $arguments = array_merge($arguments);
 
         $i = 0;
-        foreach ($arguments as $k => $_) {
-            if ($k !== $i) {
+        foreach ($arguments as $k => $_)
+        {
+            if ($k !== $i)
+            {
                 // This must be a string key.
                 // Any further numeric keys are illegal.
-                if (\array_key_exists($i, $arguments)) {
+                if (\array_key_exists($i, $arguments))
+                {
                     throw new \BadMethodCallException(self::ERROR_POSITIONAL_AFTER_NAMED);
                 }
                 return $i;
@@ -179,25 +194,34 @@ 
     {
         // Analyze parameters.
         $n = count($parameters);
-        if ($n > 0 && end($parameters)->isVariadic()) {
+        if ($n > 0 && end($parameters)->isVariadic())
+        {
             $variadicParameter = end($parameters);
             // Don't include the variadic parameter in the mapping process.
             --$n;
-        } else {
+        }
+        else
+        {
             $variadicParameter = null;
         }
 
         // Process parameters that are not already filled with positional args.
         // This loop will do nothing if $namedArgsBegin >= $n. That's ok.
-        for ($i = $namedArgsBegin; $i < $n; ++$i) {
+        for ($i = $namedArgsBegin; $i < $n; ++$i)
+        {
             $parameter = $parameters[$i];
             $k = $parameter->getName();
-            if (array_key_exists($k, $named)) {
+            if (array_key_exists($k, $named))
+            {
                 $passed[] = $named[$k];
                 unset($named[$k]);
-            } elseif ($parameter->isDefaultValueAvailable()) {
+            }
+            elseif ($parameter->isDefaultValueAvailable())
+            {
                 $passed[] = $parameter->getDefaultValue();
-            } else {
+            }
+            else
+            {
                 $message = \vsprintf(self::ERROR_ARGUMENT_NOT_PASSED, [
                     $ctx->getName(),
                     $parameter->getPosition() + 1,
@@ -208,7 +232,8 @@ 
             }
         }
 
-        if ($named === []) {
+        if ($named === [])
+        {
             // No unknown argument names exist.
             return $passed;
         }
@@ -218,11 +243,14 @@ 
         $badArgName = key($named);
 
         // Check collision with positional arguments.
-        foreach ($parameters as $i => $parameter) {
-            if ($i >= $namedArgsBegin) {
+        foreach ($parameters as $i => $parameter)
+        {
+            if ($i >= $namedArgsBegin)
+            {
                 break;
             }
-            if ($parameter->getName() === $badArgName) {
+            if ($parameter->getName() === $badArgName)
+            {
                 // The named argument overwrites a positional argument.
                 $message = \sprintf(self::ERROR_OVERWRITE_ARGUMENT, $badArgName);
                 throw new \BadMethodCallException($message);
@@ -230,7 +258,8 @@ 
         }
 
         // Special handling if a variadic parameter is present.
-        if ($variadicParameter !== null) {
+        if ($variadicParameter !== null)
+        {
             // The last parameter is variadic.
             // Since PHP 8+, variadic parameters can consume named arguments.
             // However, this code only runs if PHP < 8.

src/Framework/Domain/FilterWithRendererInterceptor.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
         ReaderInterface $reader,
         ?RenderErrors $renderErrors = null,
         int $strategy = self::STRATEGY_JSON_RESPONSE
-    ) {
+    ){
         parent::__construct($container, $strategy);
 
         $this->reader = $reader;
@@ -49,7 +49,7 @@ 
     {
         parent::buildCache($parameter, $class, $key);
 
-        if (null !== ($renderWith = $this->reader->firstClassMetadata($class, RenderWith::class))) {
+        if (null !== ($renderWith = $this->reader->firstClassMetadata($class, RenderWith::class))){
             $this->renderersCache[$class->getName()] = $this->container->get($renderWith->getRenderer());
         }
     }

Braces +2 added lines, -1 removed lines

@@ -49,7 +49,8 @@
     {
         parent::buildCache($parameter, $class, $key);
 
-        if (null !== ($renderWith = $this->reader->firstClassMetadata($class, RenderWith::class))) {
+        if (null !== ($renderWith = $this->reader->firstClassMetadata($class, RenderWith::class)))
+        {
             $this->renderersCache[$class->getName()] = $this->container->get($renderWith->getRenderer());
         }
     }

src/Framework/Domain/DefaultFilterErrorsRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      */
     public function render(FilterInterface $filter)
     {
-        switch ($this->strategy) {
+        switch ($this->strategy){
             case FilterInterceptor::STRATEGY_JSON_RESPONSE:
                 return [
                     'status' => 400,

Braces +2 added lines, -1 removed lines

@@ -25,7 +25,8 @@
      */
     public function render(FilterInterface $filter)
     {
-        switch ($this->strategy) {
+        switch ($this->strategy)
+        {
             case FilterInterceptor::STRATEGY_JSON_RESPONSE:
                 return [
                     'status' => 400,

src/Framework/Domain/FilterInterceptor.php

Spacing +13 added lines, -13 removed lines

@@ -37,20 +37,20 @@ 
      */
     public function process(string $controller, string $action, array $parameters, CoreInterface $core)
     {
-        foreach ($this->getDeclaredFilters($controller, $action) as $parameter => $filterClass) {
-            if (isset($parameters[$parameter])) {
+        foreach ($this->getDeclaredFilters($controller, $action) as $parameter => $filterClass){
+            if (isset($parameters[$parameter])){
                 continue;
             }
 
             /** @var FilterInterface $filter */
             $filter = $this->container->get($filterClass);
 
-            if (isset($parameters['@context'])) {
+            if (isset($parameters['@context'])){
                 // other interceptors can define the validation contex
                 $filter->setContext($parameters['@context']);
             }
 
-            if (!$filter->isValid()) {
+            if (!$filter->isValid()){
                 return $this->renderInvalid($filter);
             }
 
@@ -68,7 +68,7 @@ 
      */
     protected function renderInvalid(FilterInterface $filter)
     {
-        switch ($this->strategy) {
+        switch ($this->strategy){
             case self::STRATEGY_JSON_RESPONSE:
                 return [
                     'status' => 400,
@@ -87,25 +87,25 @@ 
     private function getDeclaredFilters(string $controller, string $action): array
     {
         $key = sprintf('%s:%s', $controller, $action);
-        if (array_key_exists($key, $this->cache)) {
+        if (array_key_exists($key, $this->cache)){
             return $this->cache[$key];
         }
 
         $this->cache[$key] = [];
-        try {
+        try{
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             return [];
         }
 
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter){
             $class = $this->getParameterClass($parameter);
 
-            if ($class === null) {
+            if ($class === null){
                 continue;
             }
 
-            if ($class->implementsInterface(FilterInterface::class)) {
+            if ($class->implementsInterface(FilterInterface::class)){
                 $this->buildCache($parameter, $class, $key);
             }
         }
@@ -123,11 +123,11 @@ 
     {
         $type = $parameter->getType();
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType){
             return null;
         }
 
-        if ($type->isBuiltin()) {
+        if ($type->isBuiltin()){
             return null;
         }
 

Braces +27 added lines, -13 removed lines

@@ -37,20 +37,24 @@ 
      */
     public function process(string $controller, string $action, array $parameters, CoreInterface $core)
     {
-        foreach ($this->getDeclaredFilters($controller, $action) as $parameter => $filterClass) {
-            if (isset($parameters[$parameter])) {
+        foreach ($this->getDeclaredFilters($controller, $action) as $parameter => $filterClass)
+        {
+            if (isset($parameters[$parameter]))
+            {
                 continue;
             }
 
             /** @var FilterInterface $filter */
             $filter = $this->container->get($filterClass);
 
-            if (isset($parameters['@context'])) {
+            if (isset($parameters['@context']))
+            {
                 // other interceptors can define the validation contex
                 $filter->setContext($parameters['@context']);
             }
 
-            if (!$filter->isValid()) {
+            if (!$filter->isValid())
+            {
                 return $this->renderInvalid($filter);
             }
 
@@ -68,7 +72,8 @@ 
      */
     protected function renderInvalid(FilterInterface $filter)
     {
-        switch ($this->strategy) {
+        switch ($this->strategy)
+        {
             case self::STRATEGY_JSON_RESPONSE:
                 return [
                     'status' => 400,
@@ -87,25 +92,32 @@ 
     private function getDeclaredFilters(string $controller, string $action): array
     {
         $key = sprintf('%s:%s', $controller, $action);
-        if (array_key_exists($key, $this->cache)) {
+        if (array_key_exists($key, $this->cache))
+        {
             return $this->cache[$key];
         }
 
         $this->cache[$key] = [];
-        try {
+        try
+        {
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             return [];
         }
 
-        foreach ($method->getParameters() as $parameter) {
+        foreach ($method->getParameters() as $parameter)
+        {
             $class = $this->getParameterClass($parameter);
 
-            if ($class === null) {
+            if ($class === null)
+            {
                 continue;
             }
 
-            if ($class->implementsInterface(FilterInterface::class)) {
+            if ($class->implementsInterface(FilterInterface::class))
+            {
                 $this->buildCache($parameter, $class, $key);
             }
         }
@@ -123,11 +135,13 @@ 
     {
         $type = $parameter->getType();
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType)
+        {
             return null;
         }
 
-        if ($type->isBuiltin()) {
+        if ($type->isBuiltin())
+        {
             return null;
         }