Jobtech\LaravelChunky\Strategies\Concerns\HandlesFFMpeg

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	visibility()	0	5	1
A	mergeWithFFMpeg()	0	15	2

<?php

namespace Jobtech\LaravelChunky\Strategies\Concerns;

use Illuminate\Support\Arr;
use ProtoneMedia\LaravelFFMpeg\Support\FFMpeg;

trait HandlesFFMpeg
{
    public function mergeWithFFMpeg(): void
    {
        $exporter = FFMpeg::fromDisk($this->manager->chunksFilesystem())
            ->open($this->mapChunksToArray())

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

            ->export()
            ->inFormat($this->guessFormat())
            ->concatWithoutTranscoding();

        if (! empty($visibility = $this->visibility())) {
            $exporter->withVisibility($visibility);
        }

        $exporter->toDisk(
            $this->manager->getMergeDisk()
        )->save($this->destination);
    }

    /**
     * Retrieve visibility option.
     *
     * @return string|null
     */
    public function visibility(): ?string
    {
        return Arr::get(
            $this->manager->getMergeOptions(),
            'visibility'
        );
    }

    /**
     * Guess format from destination file extension.
     *
     * @return mixed
     */
    abstract public function guessFormat();
}