mostafaznv / pdf-optimizer

src/Laravel/Concerns/Disk.php

<?php

namespace Mostafaznv\PdfOptimizer\Laravel\Concerns;

use Illuminate\Filesystem\Filesystem;
use Illuminate\Filesystem\FilesystemAdapter;
use Illuminate\Support\Str;
//use League\Flysystem\Local\LocalFilesystemAdapter;


class Disk
{
    private ?string $disk;
    private string  $temporaryDirectory;


    public function __construct(?string $disk = null)
    {
        $this->disk = $disk;
        $this->temporaryDirectory = pdf_temp_dir() . '/pdf-optimizer/' . Str::uuid()->toString();
    }

    public function __destruct()
    {
        $this->cleanupTemporaryDirectory();
    }

    public static function make(?string $disk = null): self
    {
        return new static($disk);
    }


    public function getAdapter(): ?FilesystemAdapter
    {
        return $this->disk ? app('filesystem')->disk($this->disk) : null;

The expression return $this->disk ? app('filesystem')->disk($this->disk) : null could return the type Illuminate\Contracts\Filesystem\Filesystem which includes types incompatible with the type-hinted return Illuminate\Filesystem\FilesystemAdapter|null. Consider adding an additional type-check to rule them out.

    }

    public function getTemporaryDisk(): FilesystemAdapter
    {
        if (!is_dir($this->temporaryDirectory)) {
            mkdir($this->temporaryDirectory, 0777, true);
        }

        return app('filesystem')->createLocalDriver([
            'root' => $this->temporaryDirectory,
        ]);
    }

    public function cleanupTemporaryDirectory(): self
    {
        $filesystem = new Filesystem();
        $filesystem->deleteDirectory($this->temporaryDirectory);

        return $this;
    }

    public function isLocalDisk(): bool
    {
        if ($this->getAdapter()) {
            // $this->getAdapter() instanceof LocalFilesystemAdapter
            return config("filesystems.disks.$this->disk.driver") == 'local';
        }

        return false;
    }

}