spiral / framework

src/Framework/Domain/FilterInterceptor.php

Spacing +14 added lines, -14 removed lines

@@ -50,20 +50,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);
             }
 
@@ -81,11 +81,11 @@ 
      */
     protected function renderInvalid(FilterInterface $filter)
     {
-        if ($filter instanceof ShouldRenderErrors) {
+        if ($filter instanceof ShouldRenderErrors){
             return $filter->render();
         }
 
-        switch ($this->strategy) {
+        switch ($this->strategy){
             case self::STRATEGY_JSON_RESPONSE:
                 return [
                     'status' => 400,
@@ -104,25 +104,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->cache[$key][$parameter->getName()] = $class->getName();
             }
         }
@@ -140,11 +140,11 @@ 
     {
         $type = $parameter->getType();
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType){
             return null;
         }
 
-        if ($type->isBuiltin()) {
+        if ($type->isBuiltin()){
             return null;
         }
 

Braces +29 added lines, -14 removed lines

@@ -50,20 +50,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);
             }
 
@@ -81,11 +85,13 @@ 
      */
     protected function renderInvalid(FilterInterface $filter)
     {
-        if ($filter instanceof ShouldRenderErrors) {
+        if ($filter instanceof ShouldRenderErrors)
+        {
             return $filter->render();
         }
 
-        switch ($this->strategy) {
+        switch ($this->strategy)
+        {
             case self::STRATEGY_JSON_RESPONSE:
                 return [
                     'status' => 400,
@@ -104,25 +110,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->cache[$key][$parameter->getName()] = $class->getName();
             }
         }
@@ -140,11 +153,13 @@ 
     {
         $type = $parameter->getType();
 
-        if (!$type instanceof \ReflectionNamedType) {
+        if (!$type instanceof \ReflectionNamedType)
+        {
             return null;
         }
 
-        if ($type->isBuiltin()) {
+        if ($type->isBuiltin())
+        {
             return null;
         }