Vaimo\ComposerPatches\Console\OutputAnalyser

Complexity

Total Complexity

8

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
B	scanOutputForFailures()	0	38	8

<?php
/**
 * Copyright © Vaimo Group. All rights reserved.
 * See LICENSE_VAIMO.txt for license details.
 */
namespace Vaimo\ComposerPatches\Console;

class OutputAnalyser
{
    public function scanOutputForFailures($output, $separatorMatcher, array $failureMessages)
    {
        $patternsWithResults = array_filter(
            $failureMessages,
            function ($pattern) use ($output) {
                return $pattern && preg_match($pattern, $output);
            }
        );

        if (!$patternsWithResults) {

The expression $patternsWithResults of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            return array();
        }

        $lines = explode(PHP_EOL, $output);

        $matches = array();

        foreach ($patternsWithResults as $patternCode => $pattern) {
            if (!isset($matches[$pattern])) {
                $matches[$patternCode] = array();
            }
            
            foreach ($lines as $line) {
                if (preg_match($separatorMatcher, $line)) {
                    $matches[$patternCode][] = $line;

                    continue;
                }

                if (!preg_match($pattern, $line)) {
                    continue;
                }

                $matches[$patternCode][] = $line;
            }
        }
        
        return $matches;
    }
}