Helldar\Support\Helpers\Filesystem\Directory

Complexity

Total Complexity

18

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	isDirectory()	0	7	2
A	all()	0	7	2
A	exists()	0	3	2
A	delete()	0	19	5
A	names()	0	13	4
A	doesntExist()	0	3	1
A	make()	0	7	2

<?php

namespace Helldar\Support\Helpers\Filesystem;

use DirectoryIterator;
use FilesystemIterator;
use Helldar\Support\Exceptions\DirectoryNotFoundException;
use Helldar\Support\Facades\Helpers\Filesystem\File;

This use statement conflicts with another class in this namespace, Helldar\Support\Helpers\Filesystem\File. Consider defining an alias.

use Helldar\Support\Facades\Helpers\Instance;
use SplFileInfo;

final class Directory
{
    /**
     * @param  string  $path
     *
     * @throws \Helldar\Support\Exceptions\DirectoryNotFoundException
     *
     * @return DirectoryIterator
     */
    public function all(string $path): DirectoryIterator
    {
        if ($this->doesntExist($path)) {
            throw new DirectoryNotFoundException($path);
        }

        return new DirectoryIterator($path);
    }

    public function names(string $path): array
    {
        $items = [];

        foreach ($this->all($path) as $directory) {
            if ($directory->isDir() && ! $directory->isDot()) {
                $items[] = $directory->getFilename();
            }
        }

        sort($items);

        return array_values($items);
    }

    public function make(string $path, int $mode = 755): bool
    {
        if ($this->doesntExist($path)) {
            return mkdir($path, $mode, true);
        }

        return true;
    }

    public function delete(string $path): bool
    {
        if (! $this->isDirectory($path)) {
            throw new DirectoryNotFoundException($path);
        }

        $items = new FilesystemIterator($path);

        $success = true;

        foreach ($items as $item) {
            $item->isDir() && ! $item->isLink()
                ? $this->delete($item->getPathname())
                : File::delete($item->getPathname());
        }

        @rmdir($path);

It seems like you do not handle an error condition for rmdir(). This can introduce security issues, and is generally not recommended.

        return $success;
    }

    public function exists(string $path): bool
    {
        return file_exists($path) && is_dir($path);
    }

    public function doesntExist(string $path): bool
    {
        return ! $this->exists($path);
    }

    /**
     * @param  DirectoryIterator|\SplFileInfo|string  $value
     *
     * @return bool
     */
    public function isDirectory($value): bool
    {
        if (Instance::of($value, [SplFileInfo::class, DirectoryIterator::class])) {
            return $value->isDir();
        }

        return is_dir($value);
    }
}