bavix / helpers

src/Helpers/Dir.php

<?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);
    }

}