ProtoneMedia\LaravelFFMpeg\Filesystem\TemporaryDirectories::manager()

Complexity

Conditions	2
Paths	2

Size

Total Lines	7
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	TemporaryDirectories::create()	0	5	1

<?php

namespace ProtoneMedia\LaravelFFMpeg\Filesystem;

use Illuminate\Filesystem\Filesystem;
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.

            (new Filesystem)->deleteDirectory(
                storage_path("ffmpeg_temp/{$directory}")
            );
        }

        static::$directories = [];
    }
}