Passed
Pull Request — master (#961)
by Maxim
09:08
created
src/Filters/tests/Model/FilterProviderTest.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,8 @@
 block discarded – undo
37 37
         parent::setUp();
38 38
 
39 39
         $this->container->bindSingleton(ReaderInterface::class, (new Factory())->create());
40
-        $this->container->bindSingleton(SchemaProviderInterface::class, static function (FactoryInterface $factory) {
40
+        $this->container->bindSingleton(SchemaProviderInterface::class, static function (FactoryInterface $factory)
41
+        {
41 42
             return $factory->make(SchemaProvider::class, [
42 43
                 'readers' => [
43 44
                     $factory->make(AttributeReader::class),
Please login to merge, or discard this patch.
src/Filters/src/Model/Interceptor/Validation/ValidateFilterInterceptor.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,8 @@
 block discarded – undo
18 18
         $errors = array_merge($core->callAction($controller, $action, $parameters), $parameters['errors'] ?? []);
19 19
         $errorMapper = new ErrorMapper($parameters['schema']);
20 20
 
21
-        if ($errors !== []) {
21
+        if ($errors !== [])
22
+        {
22 23
             throw new ValidationException(
23 24
                 $errorMapper->mapErrors($errors),
24 25
                 $parameters['context'] ?? null
Please login to merge, or discard this patch.
src/Filters/src/Model/Interceptor/Validation/Core.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,9 +29,11 @@
 block discarded – undo
29 29
 
30 30
     private function validateFilter(FilterInterface $filter, array $data, mixed $context = null): array
31 31
     {
32
-        if ($filter instanceof HasFilterDefinition) {
32
+        if ($filter instanceof HasFilterDefinition)
33
+        {
33 34
             $definition = $filter->filterDefinition();
34
-            if ($definition instanceof ShouldBeValidated) {
35
+            if ($definition instanceof ShouldBeValidated)
36
+            {
35 37
                 /** @var ValidationProviderInterface $validationProvider */
36 38
                 $validationProvider = $this->container->get(ValidationProviderInterface::class);
37 39
 
Please login to merge, or discard this patch.
src/Filters/src/Model/Schema/InputMapper.php 1 patch
Braces   +32 added lines, -15 removed lines patch added patch discarded remove patch
@@ -27,17 +27,23 @@  discard block
 block discarded – undo
27 27
         $errors = [];
28 28
         $result = $this->mapData($mappingSchema, $input, $setters);
29 29
 
30
-        foreach ($mappingSchema as $field => $map) {
31
-            if (empty($map[Builder::SCHEMA_FILTER])) {
30
+        foreach ($mappingSchema as $field => $map)
31
+        {
32
+            if (empty($map[Builder::SCHEMA_FILTER]))
33
+            {
32 34
                 continue;
33 35
             }
34 36
 
35 37
             $nested = $map[Builder::SCHEMA_FILTER];
36
-            if (empty($map[Builder::SCHEMA_ARRAY])) {
38
+            if (empty($map[Builder::SCHEMA_ARRAY]))
39
+            {
37 40
                 // slicing down
38
-                try {
41
+                try
42
+                {
39 43
                     $result[$field] = $this->provider->createFilter($nested, $input->withPrefix($map[Builder::SCHEMA_ORIGIN]));
40
-                } catch (ValidationException $e) {
44
+                }
45
+                catch (ValidationException $e)
46
+                {
41 47
                     $errors[$field] = $e->errors;
42 48
                     unset($result[$field]);
43 49
                 }
@@ -47,10 +53,14 @@  discard block
 block discarded – undo
47 53
             $values = [];
48 54
 
49 55
             // List of "key" => "location in request"
50
-            foreach ($this->iterate($map, $input) as $index => $origin) {
51
-                try {
56
+            foreach ($this->iterate($map, $input) as $index => $origin)
57
+            {
58
+                try
59
+                {
52 60
                     $values[$index] = $this->provider->createFilter($nested, $input->withPrefix($origin));
53
-                } catch (ValidationException $e) {
61
+                }
62
+                catch (ValidationException $e)
63
+                {
54 64
                     $errors[$field][$index] = $e->errors;
55 65
                 }
56 66
             }
@@ -64,13 +74,17 @@  discard block
 block discarded – undo
64 74
     public function mapData(array $mappingSchema, InputInterface $input, array $setters = []): array
65 75
     {
66 76
         $result = [];
67
-        foreach ($mappingSchema as $field => $map) {
68
-            if (empty($map[Builder::SCHEMA_FILTER])) {
77
+        foreach ($mappingSchema as $field => $map)
78
+        {
79
+            if (empty($map[Builder::SCHEMA_FILTER]))
80
+            {
69 81
                 $value = $input->getValue($map[Builder::SCHEMA_SOURCE], $map[Builder::SCHEMA_ORIGIN]);
70 82
 
71
-                if ($value !== null) {
83
+                if ($value !== null)
84
+                {
72 85
                     /** @var Setter $setter */
73
-                    foreach ($setters[$field] ?? [] as $setter) {
86
+                    foreach ($setters[$field] ?? [] as $setter)
87
+                    {
74 88
                         $value = $setter->updateValue($value);
75 89
                     }
76 90
 
@@ -79,7 +93,8 @@  discard block
 block discarded – undo
79 93
                 continue;
80 94
             }
81 95
 
82
-            if (empty($map[Builder::SCHEMA_ARRAY])) {
96
+            if (empty($map[Builder::SCHEMA_ARRAY]))
97
+            {
83 98
                 $filter = $this->filterFactory->createFilterInstance($map[Builder::SCHEMA_FILTER]);
84 99
                 $result[$field] = $this->mapData(
85 100
                     $this->schemaProvider->getSchema($filter),
@@ -102,11 +117,13 @@  discard block
 block discarded – undo
102 117
             $schema[Builder::SCHEMA_ITERATE_ORIGIN]
103 118
         );
104 119
 
105
-        if (empty($values) || !\is_array($values)) {
120
+        if (empty($values) || !\is_array($values))
121
+        {
106 122
             return [];
107 123
         }
108 124
 
109
-        foreach (\array_keys($values) as $key) {
125
+        foreach (\array_keys($values) as $key)
126
+        {
110 127
             yield $key => $schema[Builder::SCHEMA_ORIGIN] . '.' . $key;
111 128
         }
112 129
     }
Please login to merge, or discard this patch.
src/Filters/src/Model/Schema/SchemaProvider.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -33,7 +33,8 @@  discard block
 block discarded – undo
33 33
 
34 34
     public function getSchema(FilterInterface $filter): array
35 35
     {
36
-        if (!isset($this->schema[$filter::class])) {
36
+        if (!isset($this->schema[$filter::class]))
37
+        {
37 38
             $this->read($filter);
38 39
         }
39 40
 
@@ -42,7 +43,8 @@  discard block
 block discarded – undo
42 43
 
43 44
     public function getSetters(FilterInterface $filter): array
44 45
     {
45
-        if (!isset($this->setters[$filter::class])) {
46
+        if (!isset($this->setters[$filter::class]))
47
+        {
46 48
             $this->read($filter);
47 49
         }
48 50
 
@@ -53,7 +55,8 @@  discard block
 block discarded – undo
53 55
     {
54 56
         $this->schema[$filter::class] = [];
55 57
         $this->setters[$filter::class] = [];
56
-        foreach ($this->readers as $reader) {
58
+        foreach ($this->readers as $reader)
59
+        {
57 60
             [$readSchema, $readSetters] = $reader->read($filter);
58 61
             $this->schema[$filter::class] = \array_merge($this->schema[$filter::class], $readSchema);
59 62
             $this->setters[$filter::class] = \array_merge($this->setters[$filter::class], $readSetters);
Please login to merge, or discard this patch.
src/Filters/src/Model/Schema/AttributeReader.php 1 patch
Braces   +12 added lines, -5 removed lines patch added patch discarded remove patch
@@ -33,15 +33,22 @@
 block discarded – undo
33 33
         $setters = [];
34 34
         $class = new \ReflectionClass($filter);
35 35
 
36
-        foreach ($class->getProperties() as $property) {
36
+        foreach ($class->getProperties() as $property)
37
+        {
37 38
             /** @var object $attribute */
38
-            foreach ($this->reader->getPropertyMetadata($property) as $attribute) {
39
-                if ($attribute instanceof AbstractInput || $attribute instanceof NestedFilter) {
39
+            foreach ($this->reader->getPropertyMetadata($property) as $attribute)
40
+            {
41
+                if ($attribute instanceof AbstractInput || $attribute instanceof NestedFilter)
42
+                {
40 43
                     $schema[$property->getName()] = $attribute->getSchema($property);
41
-                } elseif ($attribute instanceof NestedArray) {
44
+                }
45
+                elseif ($attribute instanceof NestedArray)
46
+                {
42 47
                     $prefix = $attribute->input->key ?? $attribute->prefix ?? $property->getName();
43 48
                     $schema[$property->getName()] = [$attribute->class, $prefix . '.*'];
44
-                } elseif ($attribute instanceof Setter) {
49
+                }
50
+                elseif ($attribute instanceof Setter)
51
+                {
45 52
                     $setters[$property->getName()][] = $attribute;
46 53
                 }
47 54
             }
Please login to merge, or discard this patch.
src/Filters/src/Model/Mapper/SetterRegistry.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,8 @@
 block discarded – undo
11 11
 
12 12
     public function __construct(array $setters = [])
13 13
     {
14
-        foreach ($setters as $setter) {
14
+        foreach ($setters as $setter)
15
+        {
15 16
             $this->register($setter);
16 17
         }
17 18
     }
Please login to merge, or discard this patch.
src/Filters/src/Model/Mapper/Uuid.php 1 patch
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -13,7 +13,8 @@  discard block
 block discarded – undo
13 13
 
14 14
     public function supports(\ReflectionNamedType $type): bool
15 15
     {
16
-        if ($this->interfaceExists === null) {
16
+        if ($this->interfaceExists === null)
17
+        {
17 18
             $this->interfaceExists = \interface_exists(UuidInterface::class);
18 19
         }
19 20
 
@@ -27,16 +28,20 @@  discard block
 block discarded – undo
27 28
 
28 29
     private function implements(string $haystack, string $interface): bool
29 30
     {
30
-        if ($haystack === $interface) {
31
+        if ($haystack === $interface)
32
+        {
31 33
             return true;
32 34
         }
33 35
 
34
-        foreach ((array)\class_implements($haystack) as $implements) {
35
-            if ($implements === $interface) {
36
+        foreach ((array)\class_implements($haystack) as $implements)
37
+        {
38
+            if ($implements === $interface)
39
+            {
36 40
                 return true;
37 41
             }
38 42
 
39
-            if (self::implements($implements, $interface)) {
43
+            if (self::implements($implements, $interface))
44
+            {
40 45
                 return true;
41 46
             }
42 47
         }
Please login to merge, or discard this patch.
src/Filters/src/Model/Factory/FilterFactory.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,8 @@
 block discarded – undo
22 22
         $class = new \ReflectionClass($name);
23 23
 
24 24
         $args = [];
25
-        if ($constructor = $class->getConstructor()) {
25
+        if ($constructor = $class->getConstructor())
26
+        {
26 27
             $args = $this->resolver->resolveArguments($constructor);
27 28
         }
28 29
 
Please login to merge, or discard this patch.