BitWasp\Bitcoind\Utils\File

Complexity

Total Complexity

6

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Test Coverage

Coverage

25%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	recursiveDelete()	0	19	6

<?php

namespace BitWasp\Bitcoind\Utils;

use BitWasp\Bitcoind\Exception\BitcoindException;

class File
{
    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);
    }
}