NovaChunkedVideo\HasChunkFolder

Complexity

Total Complexity

4

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	storeFile()	0	3	1
A	store()	0	5	1
A	chunksFolder()	0	5	1
A	getChunksFolder()	0	3	1

<?php


namespace NovaChunkedVideo;

use Laravel\Nova\Http\Requests\NovaRequest;

trait HasChunkFolder
{

    /**
     * Temporary chunk folder
     *
     * @var string
     */
    protected string $chunksFolder;

    /**
     * The callback used to store file.
     *
     * @var callable
     */
    public ?\Closure $storeFileCallback = null;

    /**
     * Set chunk folder
     *
     * @param string $chunksFolder
     *
     * @return $this
     */
    public function chunksFolder(string $chunksFolder)
    {
        $this->chunksFolder = $chunksFolder;

        return $this;
    }

    /**
     * Get chunk folder
     *
     * @return string
     */
    public function getChunksFolder(): string
    {
        return $this->chunksFolder;
    }

    /**
     * Specify the callback that should be used to save file.
     *
     * @param callable $storeFileCallback
     *
     * @return $this
     */
    public function store(callable $storeFileCallback)
    {
        $this->storeFileCallback = $storeFileCallback;

        return $this;
    }

    /**
     * Store file
     *
     * @param NovaRequest $request
     * @param $filePath
     *
     * @return string|null
     */
    public function storeFile(NovaRequest $request, $filePath): ?string
    {
        return call_user_func($this->storeFileCallback, $filePath, $this->getStorageDisk(), $this->resource, $this->attribute, $request);

It seems like getStorageDisk() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like $this->storeFileCallback can also be of type null; however, parameter $callback of call_user_func() does only seem to accept callable, maybe add an additional type check?

    }
}