Spatie\DirectoryCleanup\DirectoryCleaner

Complexity

Total Complexity

4

Size/Duplication

Total Lines	65
Duplicated Lines	38.46 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	setDirectory()	0	6	1
A	deleteDirectoriesOlderThanMinutes()	13	13	1
A	deleteFilesOlderThanMinutes()	12	12	1

<?php

namespace Spatie\DirectoryCleanup;

use Carbon\Carbon;
use Illuminate\Support\Collection;
use Illuminate\Filesystem\Filesystem;

class DirectoryCleaner
{
    /** @var \Illuminate\Filesystem\Filesystem */
    protected $filesystem;

    /** @var string */
    protected $directory;

    /**
     * @param \Illuminate\Filesystem\Filesystem $filesystem
     */
    public function __construct(Filesystem $filesystem)
    {
        $this->filesystem = $filesystem;
    }

    /**
     * @param string $directory
     *
     * @return $this
     */
    public function setDirectory(string $directory)
    {
        $this->directory = $directory;

        return $this;
    }

    /**
     * @param int $minutes
     *
     * @return \Illuminate\Support\Collection
     */
    public function deleteDirectoriesOlderThanMinutes(int $minutes) : Collection

This method seems to be duplicated in your project.

    {
        $timeInPast = Carbon::now()->subMinutes($minutes);

        return collect($this->filesystem->directories($this->directory))
            ->filter(function ($file) use ($timeInPast) {
                return Carbon::createFromTimestamp(filemtime($file))
                             ->lt($timeInPast);
            })
            ->each(function ($file) {
                $this->filesystem->deleteDirectory($file);
            });
    }

    /**
     * @param int $minutes
     *
     * @return \Illuminate\Support\Collection
     */
    public function deleteFilesOlderThanMinutes(int $minutes) : Collection {

This method seems to be duplicated in your project.

        $timeInPast = Carbon::now()->subMinutes($minutes);

        return collect($this->filesystem->files($this->directory))
            ->filter(function ($file) use ($timeInPast) {
                return Carbon::createFromTimestamp(filemtime($file))
                    ->lt($timeInPast);
            })
            ->each(function ($file) {
                $this->filesystem->delete($file);
            });
    }
}