ProtoneMedia\LaravelFFMpeg\Filesystem\TemporaryDirectories

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	deleteAll()	0	7	2
A	create()	0	5	1

<?php

namespace ProtoneMedia\LaravelFFMpeg\Filesystem;

use Illuminate\Support\Facades\File;
use Illuminate\Support\Str;

class TemporaryDirectories
{
    private static $directories = [];

    public static function create(): string
    {
        $directory = static::$directories[] = Str::random();

Since $directories is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $directories to at least protected.

        return storage_path("ffmpeg_temp/{$directory}");
    }

    /**
     * Loop through all directories and delete them.
     */
    public static function deleteAll(): void
    {
        foreach (static::$directories as $directory) {

Since $directories is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $directories to at least protected.

            File::deleteDirectory(storage_path("ffmpeg_temp/{$directory}"));
        }

        static::$directories = [];
    }
}