TheCodingMachine\Funky\Utils\FileSystem

Complexity

Total Complexity

11

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	rmdir()	0	14	6
B	mkdir()	0	15	5

<?php
namespace TheCodingMachine\Funky\Utils;

use TheCodingMachine\Funky\IoException;

class FileSystem
{
    public static function mkdir(string $dir, int $mode = 0777): void
    {
        if (is_dir($dir)) {
            return;
        }

        if (true !== @mkdir($dir, $mode, true)) {
            $error = error_get_last();
            if (!is_dir($dir)) {
                // The directory was not created by a concurrent process.
                // Let's throw an exception with a developer friendly error message if we have one
                if ($error) {

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

                    throw IoException::cannotCreateDirectory($dir, $error['message']);
                }
                throw IoException::cannotCreateDirectory($dir, 'unknown error');
            }
        }
    }

    public static function rmdir(string $dir): void
    {
        if (is_dir($dir)) {
            $objects = scandir($dir);
            foreach ($objects as $object) {
                if ($object !== "." && $object !== "..") {
                    if (is_dir($dir. '/' .$object)) {
                        self::rmdir($dir. '/' .$object);
                    } else {
                        unlink($dir. '/' .$object);
                    }
                }
            }
            rmdir($dir);
        }
    }
}