Bavix\Helpers\Dir::remove()

Complexity

Conditions	5
Paths	2

Size

Total Lines	33
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	5

Importance

Changes

0

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