Bavix\Helpers\Dir

Complexity

Total Complexity

8

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	make()	0	3	2
A	isDir()	0	3	1
A	remove()	0	33	5

<?php

namespace Bavix\Helpers;

class Dir
{

    /**
     * @param string $path
     *
     * @return bool
     */
    public static function make($path)
    {
        return is_dir($path) || @mkdir($path, 0777, true);
    }

    /**
     * @param string $path
     *
     * @return bool
     */
    public static function isDir($path)
    {
        return is_dir($path);
    }

    /**
     * @param string $path
     * @param bool $recursive
     *
     * @return bool
     */
    public static function remove($path, $recursive = false)
    {
        $path = \rtrim($path, '/') . '/';

        if ($recursive)
        {

            $storage = \array_merge(
                \glob($path . '*'),

It seems like glob($path . '*') can also be of type false; however, parameter $array1 of array_merge() does only seem to accept array, maybe add an additional type check?

                \glob($path . '.*')

It seems like glob($path . '.*') can also be of type false; however, parameter $array2 of array_merge() does only seem to accept array|null, maybe add an additional type check?

            );

            foreach ($storage as $item)
            {

                if (in_array(basename($item), ['.', '..'], true))
                {
                    continue;
                }

                if (static::isDir($item))
                {
                    static::remove($item, $recursive);
                    continue;
                }

                File::remove($item);

            }

        }

        return rmdir($path);
    }

}