BitWasp\Bitcoind\Utils\File::recursiveDelete()

Complexity

Conditions	6
Paths	5

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	21.1875

Importance

Changes

0

<?php
    public static function recursiveDelete(string $src)
    {
        if (!is_dir($src)) {
            throw new BitcoindException("Parameter 1 for recursiveDelete should be a directory");
        }

        $dir = opendir($src);
        while (false !== ( $file = readdir($dir))) {

It seems like $dir can also be of type false; however, parameter $dir_handle of readdir() does only seem to accept resource, maybe add an additional type check?

            if (( $file != '.' ) && ( $file != '..' )) {
                $full = $src . '/' . $file;
                if (is_dir($full)) {
                    self::recursiveDelete($full);
                } else {
                    unlink($full);
                }
            }
        }
        closedir($dir);

It seems like $dir can also be of type false; however, parameter $dir_handle of closedir() does only seem to accept resource, maybe add an additional type check?

        rmdir($src);
    }