Passed
Pull Request — master (#961)
by Maxim
10:51 queued 02:34
created
src/Filters/tests/Model/Schema/InputMapperTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -20,9 +20,9 @@
 block discarded – undo
20 20
 
21 21
 final class InputMapperTest extends BaseTestCase
22 22
 {
23
-    private m\LegacyMockInterface|m\MockInterface|FilterProviderInterface $provider;
23
+    private m\LegacyMockInterface | m\MockInterface | FilterProviderInterface $provider;
24 24
     private InputMapper $mapper;
25
-    private m\LegacyMockInterface|m\MockInterface|SchemaProviderInterface $schemaProvider;
25
+    private m\LegacyMockInterface | m\MockInterface | SchemaProviderInterface $schemaProvider;
26 26
 
27 27
     public function setUp(): void
28 28
     {
Please login to merge, or discard this patch.
src/Filters/tests/Model/FilterProviderTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 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
             return $factory->make(SchemaProvider::class, [
42 42
                 'readers' => [
43 43
                     $factory->make(AttributeReader::class),
Please login to merge, or discard this patch.
src/Filters/src/Model/Interceptor/Validation/ValidateFilterInterceptor.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 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
             throw new ValidationException(
23 23
                 $errorMapper->mapErrors($errors),
24 24
                 $parameters['context'] ?? null
Please login to merge, or discard this patch.
src/Filters/src/Model/Interceptor/Validation/Core.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 {
16 16
     public function __construct(
17 17
         private readonly ContainerInterface $container
18
-    ) {
18
+    ){
19 19
     }
20 20
 
21 21
     public function callAction(string $controller, string $action, array $parameters = []): array
@@ -29,9 +29,9 @@  discard block
 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
             $definition = $filter->filterDefinition();
34
-            if ($definition instanceof ShouldBeValidated) {
34
+            if ($definition instanceof ShouldBeValidated){
35 35
                 /** @var ValidationProviderInterface $validationProvider */
36 36
                 $validationProvider = $this->container->get(ValidationProviderInterface::class);
37 37
 
Please login to merge, or discard this patch.
src/Filters/src/Model/Schema/InputMapper.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
         private readonly FilterProviderInterface $provider,
20 20
         private readonly SchemaProviderInterface $schemaProvider,
21 21
         private readonly FilterFactory $filterFactory
22
-    ) {
22
+    ){
23 23
     }
24 24
 
25 25
     public function map(array $mappingSchema, InputInterface $input, array $setters = []): array
@@ -27,17 +27,17 @@  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
+            if (empty($map[Builder::SCHEMA_FILTER])){
32 32
                 continue;
33 33
             }
34 34
 
35 35
             $nested = $map[Builder::SCHEMA_FILTER];
36
-            if (empty($map[Builder::SCHEMA_ARRAY])) {
36
+            if (empty($map[Builder::SCHEMA_ARRAY])){
37 37
                 // slicing down
38
-                try {
38
+                try{
39 39
                     $result[$field] = $this->provider->createFilter($nested, $input->withPrefix($map[Builder::SCHEMA_ORIGIN]));
40
-                } catch (ValidationException $e) {
40
+                }catch (ValidationException $e){
41 41
                     $errors[$field] = $e->errors;
42 42
                     unset($result[$field]);
43 43
                 }
@@ -47,10 +47,10 @@  discard block
 block discarded – undo
47 47
             $values = [];
48 48
 
49 49
             // List of "key" => "location in request"
50
-            foreach ($this->iterate($map, $input) as $index => $origin) {
51
-                try {
50
+            foreach ($this->iterate($map, $input) as $index => $origin){
51
+                try{
52 52
                     $values[$index] = $this->provider->createFilter($nested, $input->withPrefix($origin));
53
-                } catch (ValidationException $e) {
53
+                }catch (ValidationException $e){
54 54
                     $errors[$field][$index] = $e->errors;
55 55
                 }
56 56
             }
@@ -64,13 +64,13 @@  discard block
 block discarded – undo
64 64
     public function mapData(array $mappingSchema, InputInterface $input, array $setters = []): array
65 65
     {
66 66
         $result = [];
67
-        foreach ($mappingSchema as $field => $map) {
68
-            if (empty($map[Builder::SCHEMA_FILTER])) {
67
+        foreach ($mappingSchema as $field => $map){
68
+            if (empty($map[Builder::SCHEMA_FILTER])){
69 69
                 $value = $input->getValue($map[Builder::SCHEMA_SOURCE], $map[Builder::SCHEMA_ORIGIN]);
70 70
 
71
-                if ($value !== null) {
71
+                if ($value !== null){
72 72
                     /** @var Setter $setter */
73
-                    foreach ($setters[$field] ?? [] as $setter) {
73
+                    foreach ($setters[$field] ?? [] as $setter){
74 74
                         $value = $setter->updateValue($value);
75 75
                     }
76 76
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
                 continue;
80 80
             }
81 81
 
82
-            if (empty($map[Builder::SCHEMA_ARRAY])) {
82
+            if (empty($map[Builder::SCHEMA_ARRAY])){
83 83
                 $filter = $this->filterFactory->createFilterInstance($map[Builder::SCHEMA_FILTER]);
84 84
                 $result[$field] = $this->mapData(
85 85
                     $this->schemaProvider->getSchema($filter),
@@ -102,12 +102,12 @@  discard block
 block discarded – undo
102 102
             $schema[Builder::SCHEMA_ITERATE_ORIGIN]
103 103
         );
104 104
 
105
-        if (empty($values) || !\is_array($values)) {
105
+        if (empty($values) || !\is_array($values)){
106 106
             return [];
107 107
         }
108 108
 
109
-        foreach (\array_keys($values) as $key) {
110
-            yield $key => $schema[Builder::SCHEMA_ORIGIN] . '.' . $key;
109
+        foreach (\array_keys($values) as $key){
110
+            yield $key => $schema[Builder::SCHEMA_ORIGIN].'.'.$key;
111 111
         }
112 112
     }
113 113
 }
Please login to merge, or discard this patch.
src/Filters/src/Model/Schema/SchemaProvider.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,12 +28,12 @@  discard block
 block discarded – undo
28 28
     public function __construct(
29 29
         private readonly array $readers,
30 30
         private readonly Builder $schemaBuilder
31
-    ) {
31
+    ){
32 32
     }
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
             $this->read($filter);
38 38
         }
39 39
 
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
 
43 43
     public function getSetters(FilterInterface $filter): array
44 44
     {
45
-        if (!isset($this->setters[$filter::class])) {
45
+        if (!isset($this->setters[$filter::class])){
46 46
             $this->read($filter);
47 47
         }
48 48
 
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
     {
54 54
         $this->schema[$filter::class] = [];
55 55
         $this->setters[$filter::class] = [];
56
-        foreach ($this->readers as $reader) {
56
+        foreach ($this->readers as $reader){
57 57
             [$readSchema, $readSetters] = $reader->read($filter);
58 58
             $this->schema[$filter::class] = \array_merge($this->schema[$filter::class], $readSchema);
59 59
             $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
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
 {
20 20
     public function __construct(
21 21
         private readonly AttributesReader $reader
22
-    ) {
22
+    ){
23 23
     }
24 24
 
25 25
     /**
@@ -33,15 +33,15 @@  discard block
 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
             /** @var object $attribute */
38
-            foreach ($this->reader->getPropertyMetadata($property) as $attribute) {
39
-                if ($attribute instanceof AbstractInput || $attribute instanceof NestedFilter) {
38
+            foreach ($this->reader->getPropertyMetadata($property) as $attribute){
39
+                if ($attribute instanceof AbstractInput || $attribute instanceof NestedFilter){
40 40
                     $schema[$property->getName()] = $attribute->getSchema($property);
41
-                } elseif ($attribute instanceof NestedArray) {
41
+                } elseif ($attribute instanceof NestedArray){
42 42
                     $prefix = $attribute->input->key ?? $attribute->prefix ?? $property->getName();
43
-                    $schema[$property->getName()] = [$attribute->class, $prefix . '.*'];
44
-                } elseif ($attribute instanceof Setter) {
43
+                    $schema[$property->getName()] = [$attribute->class, $prefix.'.*'];
44
+                } elseif ($attribute instanceof Setter){
45 45
                     $setters[$property->getName()][] = $attribute;
46 46
                 }
47 47
             }
Please login to merge, or discard this patch.
src/Filters/src/Model/Mapper/SetterRegistry.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@
 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
             $this->register($setter);
16 16
         }
17 17
     }
Please login to merge, or discard this patch.
src/Filters/src/Model/Mapper/Mapper.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
     public function __construct(
17 17
         private readonly SchemaProviderInterface $schemaProvider,
18 18
         private readonly SetterRegistryInterface $registry
19
-    ) {
19
+    ){
20 20
     }
21 21
 
22 22
     /**
@@ -26,30 +26,30 @@  discard block
 block discarded – undo
26 26
     {
27 27
         $class = new \ReflectionClass($filter);
28 28
 
29
-        foreach ($this->schemaProvider->getSchema($filter) as $field => $map) {
30
-            if (!empty($map[Builder::SCHEMA_FILTER]) && $data[$field] === []) {
29
+        foreach ($this->schemaProvider->getSchema($filter) as $field => $map){
30
+            if (!empty($map[Builder::SCHEMA_FILTER]) && $data[$field] === []){
31 31
                 continue;
32 32
             }
33 33
 
34 34
             $property = $class->getProperty($field);
35
-            foreach ($this->registry->getSetters() as $setter) {
36
-                if (!empty($map[Builder::SCHEMA_FILTER])) {
35
+            foreach ($this->registry->getSetters() as $setter){
36
+                if (!empty($map[Builder::SCHEMA_FILTER])){
37 37
                     $this->registry->getDefault()->setValue($filter, $property, $data[$field]);
38 38
                     break;
39 39
                 }
40 40
 
41 41
                 $type = $property->getType();
42
-                if (!isset($data[$field]) && $type->allowsNull()) {
42
+                if (!isset($data[$field]) && $type->allowsNull()){
43 43
                     $this->registry->getDefault()->setValue($filter, $property, null);
44 44
                     break;
45 45
                 }
46 46
 
47
-                if (!$type instanceof \ReflectionNamedType || $type->isBuiltin()) {
47
+                if (!$type instanceof \ReflectionNamedType || $type->isBuiltin()){
48 48
                     $this->registry->getDefault()->setValue($filter, $property, $data[$field]);
49 49
                     break;
50 50
                 }
51 51
 
52
-                if ($setter->supports($type)) {
52
+                if ($setter->supports($type)){
53 53
                     $setter->setValue($filter, $property, $data[$field]);
54 54
                 }
55 55
             }
Please login to merge, or discard this patch.