Sauls\Component\Helper\Operation\FilesystemOperation\RemoveDirectoryRecursively

Complexity

Total Complexity

8

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	createDirectoryPath()	0	3	1
A	removeSubDirectoriesAndFiles()	0	10	4
A	execute()	0	14	3

<?php
/**
 * This file is part of the sauls/helpers package.
 *
 * @author    Saulius Vaičeliūnas <vaiceliunas@inbox.lt>
 * @link      http://saulius.vaiceliunas.lt
 * @copyright 2018
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sauls\Component\Helper\Operation\FilesystemOperation;


class RemoveDirectoryRecursively implements RemoveDirectoryRecursivelyInterface
{
    public function execute(string $directory): bool
    {
        if (\is_dir($directory)) {
            $objects = \scandir($directory, SCANDIR_SORT_NONE);
            foreach ($objects as $object) {
                $this->removeSubDirectoriesAndFiles($object, $directory);
            }

            \reset($objects);

It seems like $objects can also be of type false; however, parameter $array of reset() does only seem to accept array, maybe add an additional type check?

            return \rmdir($directory);
        }

        return false;
    }

    private function removeSubDirectoriesAndFiles($object, string $directory): void
    {
        if ($object !== '.' && $object !== '..') {

            $currentDirectory = $this->createDirectoryPath([$directory, $object]);

            if (filetype($currentDirectory) === 'dir') {
                $this->execute($currentDirectory);
            } else {
                \unlink($currentDirectory);
            }
        }
    }

    private function createDirectoryPath(array $directories): string
    {
        return implode(DIRECTORY_SEPARATOR, $directories);
    }
}