Passed
Push — master ( d04b57...fdae84 )
by butschster
14:43 queued 08:29
created
src/Dumper/src/Dumper.php 1 patch
Braces   +84 added lines, -38 removed lines patch added patch discarded remove patch
@@ -57,7 +57,8 @@  discard block
 block discarded – undo
57 57
 
58 58
     public function __construct(LoggerInterface $logger = null)
59 59
     {
60
-        if ($logger !== null) {
60
+        if ($logger !== null)
61
+        {
61 62
             $this->setLogger($logger);
62 63
         }
63 64
     }
@@ -83,7 +84,8 @@  discard block
 block discarded – undo
83 84
         $r = $this->getRenderer($target);
84 85
         $dump = $r->wrapContent($this->renderValue($r, $value));
85 86
 
86
-        switch ($target) {
87
+        switch ($target)
88
+        {
87 89
             case self::OUTPUT:
88 90
                 echo $dump;
89 91
                 break;
@@ -93,7 +95,8 @@  discard block
 block discarded – undo
93 95
                 return $dump;
94 96
 
95 97
             case self::LOGGER:
96
-                if ($this->logger === null) {
98
+                if ($this->logger === null)
99
+                {
97 100
                     throw new DumperException('Unable to dump value to log, no associated LoggerInterface.');
98 101
                 }
99 102
                 $this->logger->debug((string) $dump);
@@ -114,7 +117,8 @@  discard block
 block discarded – undo
114 117
      */
115 118
     public function setRenderer(int $target, RendererInterface $renderer): Dumper
116 119
     {
117
-        if (!isset($this->targets[$target])) {
120
+        if (!isset($this->targets[$target]))
121
+        {
118 122
             throw new DumperException(sprintf('Undefined dump target %d', $target));
119 123
         }
120 124
 
@@ -131,19 +135,25 @@  discard block
 block discarded – undo
131 135
      */
132 136
     private function getRenderer(int $target): RendererInterface
133 137
     {
134
-        if ($target == self::OUTPUT && System::isCLI()) {
135
-            if (System::isColorsSupported(STDOUT)) {
138
+        if ($target == self::OUTPUT && System::isCLI())
139
+        {
140
+            if (System::isColorsSupported(STDOUT))
141
+            {
136 142
                 $target = self::OUTPUT_CLI_COLORS;
137
-            } else {
143
+            }
144
+            else
145
+            {
138 146
                 $target = self::OUTPUT_CLI;
139 147
             }
140 148
         }
141 149
 
142
-        if (!isset($this->targets[$target])) {
150
+        if (!isset($this->targets[$target]))
151
+        {
143 152
             throw new DumperException(sprintf('Undefined dump target %d', $target));
144 153
         }
145 154
 
146
-        if (is_string($this->targets[$target])) {
155
+        if (is_string($this->targets[$target]))
156
+        {
147 157
             $this->targets[$target] = new $this->targets[$target]();
148 158
         }
149 159
 
@@ -166,28 +176,35 @@  discard block
 block discarded – undo
166 176
         int $level = 0,
167 177
         bool $hideHeader = false
168 178
     ): string {
169
-        if (!$hideHeader && !empty($name)) {
179
+        if (!$hideHeader && !empty($name))
180
+        {
170 181
             $header = $r->indent($level) . $r->apply($name, 'name') . $r->apply(' = ', 'syntax', '=');
171
-        } else {
182
+        }
183
+        else
184
+        {
172 185
             $header = $r->indent($level);
173 186
         }
174 187
 
175
-        if ($level > $this->maxLevel) {
188
+        if ($level > $this->maxLevel)
189
+        {
176 190
             //Renderer is not reference based, we can't dump too deep values
177 191
             return $r->indent($level) . $r->apply('-too deep-', 'maxLevel') . "\n";
178 192
         }
179 193
 
180 194
         $type = strtolower(gettype($value));
181 195
 
182
-        if ($type === 'array') {
196
+        if ($type === 'array')
197
+        {
183 198
             return $header . $this->renderArray($r, $value, $level, $hideHeader);
184 199
         }
185 200
 
186
-        if ($type === 'object') {
201
+        if ($type === 'object')
202
+        {
187 203
             return $header . $this->renderObject($r, $value, $level, $hideHeader);
188 204
         }
189 205
 
190
-        if ($type === 'resource') {
206
+        if ($type === 'resource')
207
+        {
191 208
             //No need to dump resource value
192 209
             $element = get_resource_type($value) . ' resource ';
193 210
 
@@ -201,7 +218,8 @@  discard block
 block discarded – undo
201 218
         $header .= $r->apply("{$type}({$length})", 'type', $type);
202 219
 
203 220
         $element = null;
204
-        switch ($type) {
221
+        switch ($type)
222
+        {
205 223
             case 'string':
206 224
                 $element = $r->escapeStrings() ? htmlspecialchars($value) : $value;
207 225
                 break;
@@ -211,7 +229,8 @@  discard block
 block discarded – undo
211 229
                 break;
212 230
 
213 231
             default:
214
-                if ($value !== null) {
232
+                if ($value !== null)
233
+                {
215 234
                     //Not showing null value, type is enough
216 235
                     $element = var_export($value, true);
217 236
                 }
@@ -223,19 +242,25 @@  discard block
 block discarded – undo
223 242
 
224 243
     private function renderArray(RendererInterface $r, array $array, int $level, bool $hideHeader = false): string
225 244
     {
226
-        if (!$hideHeader) {
245
+        if (!$hideHeader)
246
+        {
227 247
             $count = count($array);
228 248
 
229 249
             //Array size and scope
230 250
             $output = $r->apply("array({$count})", 'type', 'array') . "\n";
231 251
             $output .= $r->indent($level) . $r->apply('[', 'syntax', '[') . "\n";
232
-        } else {
252
+        }
253
+        else
254
+        {
233 255
             $output = '';
234 256
         }
235 257
 
236
-        foreach ($array as $key => $value) {
237
-            if (!is_numeric($key)) {
238
-                if (is_string($key) && $r->escapeStrings()) {
258
+        foreach ($array as $key => $value)
259
+        {
260
+            if (!is_numeric($key))
261
+            {
262
+                if (is_string($key) && $r->escapeStrings())
263
+                {
239 264
                     $key = htmlspecialchars($key);
240 265
                 }
241 266
 
@@ -245,7 +270,8 @@  discard block
 block discarded – undo
245 270
             $output .= $this->renderValue($r, $value, "[{$key}]", $level + 1);
246 271
         }
247 272
 
248
-        if (!$hideHeader) {
273
+        if (!$hideHeader)
274
+        {
249 275
             //Closing array scope
250 276
             $output .= $r->indent($level) . $r->apply(']', 'syntax', ']') . "\n";
251 277
         }
@@ -264,29 +290,38 @@  discard block
 block discarded – undo
264 290
         bool $hideHeader = false,
265 291
         string $class = ''
266 292
     ): string {
267
-        if (!$hideHeader) {
293
+        if (!$hideHeader)
294
+        {
268 295
             $type = ($class ?: get_class($value)) . ' object ';
269 296
 
270 297
             $header = $r->apply($type, 'type', 'object') . "\n";
271 298
             $header .= $r->indent($level) . $r->apply('(', 'syntax', '(') . "\n";
272
-        } else {
299
+        }
300
+        else
301
+        {
273 302
             $header = '';
274 303
         }
275 304
 
276 305
         //Let's use method specifically created for dumping
277
-        if (method_exists($value, '__debugInfo') || $value instanceof \Closure) {
278
-            if ($value instanceof \Closure) {
306
+        if (method_exists($value, '__debugInfo') || $value instanceof \Closure)
307
+        {
308
+            if ($value instanceof \Closure)
309
+            {
279 310
                 $debugInfo = $this->describeClosure($value);
280
-            } else {
311
+            }
312
+            else
313
+            {
281 314
                 $debugInfo = $value->__debugInfo();
282 315
             }
283 316
 
284
-            if (is_array($debugInfo)) {
317
+            if (is_array($debugInfo))
318
+            {
285 319
                 //Pretty view
286 320
                 $debugInfo = (object)$debugInfo;
287 321
             }
288 322
 
289
-            if (is_object($debugInfo)) {
323
+            if (is_object($debugInfo))
324
+            {
290 325
                 //We are not including syntax elements here
291 326
                 return $this->renderObject($r, $debugInfo, $level, false, get_class($value));
292 327
             }
@@ -299,7 +334,8 @@  discard block
 block discarded – undo
299 334
         $refection = new \ReflectionObject($value);
300 335
 
301 336
         $output = '';
302
-        foreach ($refection->getProperties() as $property) {
337
+        foreach ($refection->getProperties() as $property)
338
+        {
303 339
             $output .= $this->renderProperty($r, $value, $property, $level);
304 340
         }
305 341
 
@@ -313,7 +349,8 @@  discard block
 block discarded – undo
313 349
      */
314 350
     private function renderProperty(RendererInterface $r, $value, \ReflectionProperty $p, int $level): string
315 351
     {
316
-        if ($p->isStatic()) {
352
+        if ($p->isStatic())
353
+        {
317 354
             return '';
318 355
         }
319 356
 
@@ -333,9 +370,12 @@  discard block
 block discarded – undo
333 370
         //To read private and protected properties
334 371
         $p->setAccessible(true);
335 372
 
336
-        if ($value instanceof \stdClass) {
373
+        if ($value instanceof \stdClass)
374
+        {
337 375
             $name = $r->apply($p->getName(), 'dynamic');
338
-        } else {
376
+        }
377
+        else
378
+        {
339 379
             //Property name includes access level
340 380
             $name = $p->getName() . $r->apply(':' . $access, 'access', $access);
341 381
         }
@@ -348,9 +388,12 @@  discard block
 block discarded – undo
348 388
      */
349 389
     private function describeClosure(\Closure $closure): array
350 390
     {
351
-        try {
391
+        try
392
+        {
352 393
             $r = new \ReflectionFunction($closure);
353
-        } catch (\ReflectionException $e) {
394
+        }
395
+        catch (\ReflectionException $e)
396
+        {
354 397
             return ['closure' => 'unable to resolve'];
355 398
         }
356 399
 
@@ -368,9 +411,12 @@  discard block
 block discarded – undo
368 411
      */
369 412
     private function getAccess(\ReflectionProperty $p): string
370 413
     {
371
-        if ($p->isPrivate()) {
414
+        if ($p->isPrivate())
415
+        {
372 416
             return 'private';
373
-        } elseif ($p->isProtected()) {
417
+        }
418
+        elseif ($p->isProtected())
419
+        {
374 420
             return 'protected';
375 421
         }
376 422
 
Please login to merge, or discard this patch.