Wszetko\Sitemap\Helpers\Directory

Complexity

Total Complexity

10

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Test Coverage

Coverage

95.65%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	checkDirectory()	0	19	3
B	removeDir()	0	23	7

<?php

declare(strict_types=1);

/**
 * This file is part of Wszetko Sitemap.
 *
 * (c) Paweł Kłopotek-Główczewski <pawelkg@pawelkg.com>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Wszetko\Sitemap\Helpers;

use Exception;

/**
 * Class Directory.
 *
 * @package Wszetko\Sitemap\Helpers
 */
class Directory
{
    /**
     * @param string $directory
     *
     * @return string
     *
     * @throws \Exception
     */
    public static function checkDirectory(string $directory): string
    {
        $dir = realpath($directory);

        if (false === $dir) {
            @mkdir(

It seems like you do not handle an error condition for mkdir(). This can introduce security issues, and is generally not recommended.

                $directory,
                0777,
                true
            );

            $dir = realpath($directory);
        }

        if (false === $dir) {
            throw new Exception("Can't get directory $directory.");
        }

        return $dir;
    }

    /**
     * @param string $dir
     */
    public static function removeDir(string $dir): void
    {
        if (!is_dir($dir)) {
            return;
        }

        $objects = scandir($dir);

        if (false !== $objects) {
            foreach ($objects as $object) {
                if ('.' === $object || '..' === $object) {
                    continue;
                }

                if (is_dir($dir . '/' . $object)) {
                    self::removeDir($dir . '/' . $object);
                } else {
                    unlink($dir . '/' . $object);
                }
            }
        }

        rmdir($dir);
    }
}