Helldar\Support\Helpers\Filesystem\Directory

Complexity

Total Complexity

16

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
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.

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->doesntExist($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);
    }
}