Vaimo\ComposerPatches\Console\OutputAnalyser::scanOutputForFailures()

Complexity

Conditions	8
Paths	10

Size

Total Lines	38
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    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;
    }