Passed
Push — master ( 1e96ad...00da65 )
by Smoren
01:46
created
src/Components/Schemator.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -48,9 +48,9 @@  discard block
 block discarded – undo
48 48
     {
49 49
         $toAccessor = $this->nestedAccessorFactory->create($result, $this->pathDelimiter);
50 50
 
51
-        foreach($schema as $keyTo => $keyFrom) {
51
+        foreach ($schema as $keyTo => $keyFrom) {
52 52
             $value = $this->getValue($source, $keyFrom, $strict);
53
-            if($keyTo === '') {
53
+            if ($keyTo === '') {
54 54
                 return $value;
55 55
             }
56 56
             $toAccessor->set($keyTo, $value, $strict);
@@ -73,19 +73,19 @@  discard block
 block discarded – undo
73 73
      */
74 74
     public function getValue($source, $key, bool $strict = false)
75 75
     {
76
-        if($key === '' || $key === null) {
76
+        if ($key === '' || $key === null) {
77 77
             return $source;
78 78
         }
79 79
 
80
-        if($source === null || (!is_array($source) && !is_object($source))) {
80
+        if ($source === null || (!is_array($source) && !is_object($source))) {
81 81
             return $this->getValueFromUnsupportedSource($source, $key, $strict);
82 82
         }
83 83
 
84
-        if(is_string($key)) {
84
+        if (is_string($key)) {
85 85
             return $this->getValueByKey($source, $key, $strict);
86 86
         }
87 87
 
88
-        if(is_array($key)) {
88
+        if (is_array($key)) {
89 89
             return $this->getValueByFilters($source, $key, $strict);
90 90
         }
91 91
 
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
         try {
115 115
             $fromAccessor = $this->nestedAccessorFactory->create($source, $this->pathDelimiter);
116 116
             return $fromAccessor->get($key, $strict);
117
-        } catch(NestedAccessorException $e) {
117
+        } catch (NestedAccessorException $e) {
118 118
             throw SchematorException::createAsCannotGetValue($source, $key, $e);
119 119
         }
120 120
     }
@@ -130,13 +130,13 @@  discard block
 block discarded – undo
130 130
     protected function getValueByFilters($source, array $filters, bool $strict)
131 131
     {
132 132
         $result = $source;
133
-        foreach($filters as $filterConfig) {
134
-            if(is_string($filterConfig)) {
133
+        foreach ($filters as $filterConfig) {
134
+            if (is_string($filterConfig)) {
135 135
                 $result = $this->getValue($result, $filterConfig, $strict);
136
-            } elseif(is_array($filterConfig)) {
136
+            } elseif (is_array($filterConfig)) {
137 137
                 $result = $this->runFilter($filterConfig, $result, $source, $strict);
138 138
             } else {
139
-                if($strict) {
139
+                if ($strict) {
140 140
                     throw SchematorException::createAsUnsupportedFilterConfigType($filterConfig);
141 141
                 }
142 142
                 $result = null;
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
      */
157 157
     protected function getValueFromUnsupportedSource($source, $key, bool $strict)
158 158
     {
159
-        if(!$strict) {
159
+        if (!$strict) {
160 160
             return null;
161 161
         }
162 162
         throw SchematorException::createAsUnsupportedSourceType($source, $key);
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
      */
173 173
     protected function getValueByUnsupportedKey($source, $key, bool $strict)
174 174
     {
175
-        if(!$strict) {
175
+        if (!$strict) {
176 176
             return null;
177 177
         }
178 178
         throw SchematorException::createAsUnsupportedKeyType($source, $key);
@@ -198,8 +198,8 @@  discard block
 block discarded – undo
198 198
                 new FilterContext($this, $source, $rootSource),
199 199
                 ...$filterConfig
200 200
             );
201
-        } catch(Throwable $e) {
202
-            if($strict) {
201
+        } catch (Throwable $e) {
202
+            if ($strict) {
203 203
                 throw SchematorException::createAsFilterError($filterName, $filterConfig, $source, $e);
204 204
             }
205 205
             return null;
Please login to merge, or discard this patch.
src/Components/NestedAccessor.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -29,11 +29,11 @@  discard block
 block discarded – undo
29 29
      */
30 30
     public function __construct(&$source, string $pathDelimiter = '.')
31 31
     {
32
-        if($source === null) {
32
+        if ($source === null) {
33 33
             $source = [];
34 34
         }
35 35
 
36
-        if(is_scalar($source)) {
36
+        if (is_scalar($source)) {
37 37
             throw NestedAccessorException::createAsSourceIsScalar($source);
38 38
         }
39 39
 
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
     public function get(?string $path = null, bool $strict = true)
53 53
     {
54 54
         // when path is not specified
55
-        if($path === null || $path === '') {
55
+        if ($path === null || $path === '') {
56 56
             // let's return the full source
57 57
             return $this->source;
58 58
         }
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
         );
71 71
 
72 72
         // when strict mode is on and we got errors
73
-        if($strict && $errorsCount) {
73
+        if ($strict && $errorsCount) {
74 74
             throw NestedAccessorException::createAsCannotGetValue($path, $errorsCount);
75 75
         }
76 76
 
@@ -101,9 +101,9 @@  discard block
 block discarded – undo
101 101
     protected function _get($source, array $path, &$result, int &$errorsCount): void
102 102
     {
103 103
         // if path stack is empty — we reached target value of given path in source argument
104
-        if(!count($path)) {
104
+        if (!count($path)) {
105 105
             // so if result is multiple
106
-            if(is_array($result)) {
106
+            if (is_array($result)) {
107 107
                 // we append source to result
108 108
                 $result[] = $source;
109 109
             } else {
@@ -115,11 +115,11 @@  discard block
 block discarded – undo
115 115
         }
116 116
 
117 117
         // let's iterate every path part from stack
118
-        while(count($path)) {
118
+        while (count($path)) {
119 119
             $key = array_pop($path);
120 120
 
121
-            if(is_array($source)) {
122
-                if(!array_key_exists($key, $source)) {
121
+            if (is_array($source)) {
122
+                if (!array_key_exists($key, $source)) {
123 123
                     // path part key is missing in source array
124 124
                     $errorsCount++;
125 125
                     // we cannot go deeper
@@ -127,8 +127,8 @@  discard block
 block discarded – undo
127 127
                 }
128 128
                 // go to the next nested level
129 129
                 $source = $source[$key];
130
-            } elseif(is_object($source)) {
131
-                if(!property_exists($source, $key)) {
130
+            } elseif (is_object($source)) {
131
+                if (!property_exists($source, $key)) {
132 132
                     // path part key is missing in source object
133 133
                     $errorsCount++;
134 134
                     // we cannot go deeper
@@ -145,13 +145,13 @@  discard block
 block discarded – undo
145 145
 
146 146
             // when it's not the last iteration of the stack
147 147
             // and the source is non-associative array (list)
148
-            if(count($path) && is_array($source) && !ArrHelper::isAssoc($source)) {
148
+            if (count($path) && is_array($source) && !ArrHelper::isAssoc($source)) {
149 149
                 // the result will be multiple
150
-                if(!is_array($result)) {
150
+                if (!is_array($result)) {
151 151
                     $result = [];
152 152
                 }
153 153
                 // and we need to use recursive call for each item of this array
154
-                foreach($source as $item) {
154
+                foreach ($source as $item) {
155 155
                     $this->_get($item, $path, $result, $errorsCount);
156 156
                 }
157 157
                 // we don't need to do something in this recursive branch
@@ -177,15 +177,15 @@  discard block
 block discarded – undo
177 177
     {
178 178
         $temp = &$source;
179 179
         // let's iterate every path part to go deeper into nesting
180
-        foreach($path as $key) {
181
-            if(isset($temp) && is_scalar($temp)) {
180
+        foreach ($path as $key) {
181
+            if (isset($temp) && is_scalar($temp)) {
182 182
                 // value in the middle of the path must me an array
183 183
                 $temp = [];
184 184
             }
185 185
 
186 186
             // go to the next nested level
187
-            if(is_object($temp)) {
188
-                if($strict && !property_exists($temp, $key)) {
187
+            if (is_object($temp)) {
188
+                if ($strict && !property_exists($temp, $key)) {
189 189
                     throw NestedAccessorException::createAsCannotSetValue($key);
190 190
                 }
191 191
                 $temp = &$temp->{$key};
Please login to merge, or discard this patch.