ComposerRequireChecker\FileLocator\LocateAllFilesByExtension

Complexity

Total Complexity

10

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Test Coverage

Coverage

96.3%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	prepareBlacklistPatterns()	0	13	3
A	__invoke()	0	10	2
A	filterFilesByExtension()	0	17	5

<?php

namespace ComposerRequireChecker\FileLocator;

use Traversable;

final class LocateAllFilesByExtension
{
    public function __invoke(Traversable $directories, string $fileExtension, ?array $blacklist): Traversable
    {
        foreach ($directories as $directory) {
            yield from $this->filterFilesByExtension(
                new \RecursiveIteratorIterator(
                    new \RecursiveDirectoryIterator($directory),
                    \RecursiveIteratorIterator::LEAVES_ONLY
                ),
                $fileExtension,
                $blacklist
            );
        }
    }

    private function filterFilesByExtension(Traversable $files, string $fileExtension, ?array $blacklist): Traversable
    {
        $extensionMatcher = '/.*' . preg_quote($fileExtension) . '$/';

        $blacklist = $this->prepareBlacklistPatterns($blacklist);

        /* @var $file \SplFileInfo */
        foreach ($files as $file) {
            if ($blacklist && preg_match('{('.implode('|', $blacklist).')}', $file->getPathname())) {

The expression $blacklist 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.

                continue;
            }

            if (!preg_match($extensionMatcher, $file->getBasename())) {
                continue;
            }

            yield $file->getPathname();
        }
    }

    private function prepareBlacklistPatterns(?array $blacklistPaths)
    {
        if ($blacklistPaths === null) {

The condition $blacklistPaths === null is always false.

            return $blacklistPaths;
        }

        foreach ($blacklistPaths as &$path) {
            $path = preg_replace('{/+}', '/', preg_quote(trim(strtr($path, '\\', '/'), '/')));
            $path = str_replace('\\*\\*', '.+?', $path);
            $path = str_replace('\\*', '[^/]+?', $path);
        }

        return $blacklistPaths;
    }
}