findPolyfill()

Complexity

Conditions	5
Paths	5

Size

Total Lines	32
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

#!/usr/bin/env php
function findPolyfill(string $directory): int
{
    // See issue #4215 - code which should have erred in unit test
    // succeeded because a dev package required polyfills.
    $retCode = 0;
    $polyfill81 = 'MYSQLI_REFRESH_REPLICA\\b'
        . '|fdiv[(]'
        . '|array_is_list[(]'
        . '|enum_exists[(]';
    $polyfill = '/\\b(?:'
        . $polyfill81
        . '})/';

    $it = new RecursiveIteratorIterator(
        new RecursiveDirectoryIterator($directory, FilesystemIterator::UNIX_PATHS)
    );

    foreach ($it as $file) {
        if ($file->getExtension() === 'php') {
            $fullname = $it->getPath() . '/' . $it->getBaseName();

The method getPath() does not exist on RecursiveIteratorIterator.

The method getBaseName() does not exist on RecursiveIteratorIterator.

            $contents = file_get_contents($fullname);
            if ($contents === false) {
                echo "failed to read $fullname\n";
                ++$retCode;
            } elseif (preg_match_all($polyfill, $contents, $matches)) {
                var_dump($fullname, $matches);

var_dump($fullname, $matches) looks like debug code. Are you sure you do not want to remove it?

                ++$retCode;
            }
        }
    }

    return $retCode;
}