Skip to content

Commit c341d74

Browse files
authored
Merge pull request #1033 from PHPCSStandards/phpcs-4.0/modernize/messagecollector-use-foreach-list
Modernize: use list in foreach
2 parents c41e13f + 1c185ad commit c341d74

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/Util/MessageCollector.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -182,8 +182,8 @@ public function display($order=self::ORDERBY_SEVERITY)
182182
*/
183183
private function prefixAll(array $messages)
184184
{
185-
foreach ($messages as $i => $details) {
186-
$messages[$i]['message'] = $this->prefix($details['message'], $details['type']);
185+
foreach ($messages as $i => list('message' => $message, 'type' => $type)) {
186+
$messages[$i]['message'] = $this->prefix($message, $type);
187187
}
188188

189189
return $messages;
@@ -242,22 +242,22 @@ private function sortBySeverity(array $messages)
242242
$notices = [];
243243
$deprecations = [];
244244

245-
foreach ($messages as $details) {
246-
switch ($details['type']) {
245+
foreach ($messages as list('message' => $message, 'type' => $type)) {
246+
switch ($type) {
247247
case self::ERROR:
248-
$errors[] = $details['message'];
248+
$errors[] = $message;
249249
break;
250250

251251
case self::WARNING:
252-
$warnings[] = $details['message'];
252+
$warnings[] = $message;
253253
break;
254254

255255
case self::DEPRECATED:
256-
$deprecations[] = $details['message'];
256+
$deprecations[] = $message;
257257
break;
258258

259259
default:
260-
$notices[] = $details['message'];
260+
$notices[] = $message;
261261
break;
262262
}
263263
}

0 commit comments

Comments
 (0)