@@ -320,12 +320,14 @@ |
||
320 | 320 | return array(); |
321 | 321 | // @codeCoverageIgnoreEnd |
322 | 322 | } |
323 | - if ($result === array()) $this->err(sprintf( |
|
323 | + if ($result === array()) { |
|
324 | + $this->err(sprintf( |
|
324 | 325 | "ineffective pattern: %s", |
325 | 326 | $pattern === $glob |
326 | 327 | ? $pattern |
327 | 328 | : sprintf("'%s' -> '%s'", $pattern, $glob) |
328 | 329 | )); |
330 | + } |
|
329 | 331 | if (!is_array($result)) { |
330 | 332 | // @codeCoverageIgnoreStart |
331 | 333 | throw new \UnexpectedValueException( |
@@ -272,7 +272,9 @@ |
||
272 | 272 | } |
273 | 273 | |
274 | 274 | foreach ($array as $section => $refs) { |
275 | - if (!in_array($section, $sections, true)) continue; |
|
275 | + if (!in_array($section, $sections, true)) { |
|
276 | + continue; |
|
277 | + } |
|
276 | 278 | if (!is_array($refs)) { |
277 | 279 | ParseException::__("'$section' requires a list"); |
278 | 280 | } |
@@ -119,12 +119,12 @@ |
||
119 | 119 | } |
120 | 120 | // match |
121 | 121 | foreach (array_unique( |
122 | - /** @scrutinizer ignore-type */ |
|
123 | - preg_split( |
|
124 | - '~\\\\.(*SKIP)(*FAIL)|,~', |
|
125 | - $matches[2] |
|
126 | - ) |
|
127 | - ) as $segment) { |
|
122 | + /** @scrutinizer ignore-type */ |
|
123 | + preg_split( |
|
124 | + '~\\\\.(*SKIP)(*FAIL)|,~', |
|
125 | + $matches[2] |
|
126 | + ) |
|
127 | + ) as $segment) { |
|
128 | 128 | $permutation = $matches[1] . $segment . $matches[3]; |
129 | 129 | in_array($permutation, $stack, true) || $stack[] = $permutation; |
130 | 130 | } |
@@ -64,9 +64,15 @@ |
||
64 | 64 | $show = $args->hasOption('show'); |
65 | 65 | $list = $args->hasOption('list'); |
66 | 66 | |
67 | - if ($images) return $this->shower()->showImages(); |
|
68 | - if ($show) return $this->shower()->showPipelines(); |
|
69 | - if ($list) return $this->shower()->showPipelineIds(); |
|
67 | + if ($images) { |
|
68 | + return $this->shower()->showImages(); |
|
69 | + } |
|
70 | + if ($show) { |
|
71 | + return $this->shower()->showPipelines(); |
|
72 | + } |
|
73 | + if ($list) { |
|
74 | + return $this->shower()->showPipelineIds(); |
|
75 | + } |
|
70 | 76 | |
71 | 77 | return null; |
72 | 78 | } |