ProtoneMedia\LaravelFFMpeg\Exporters\HandlesConcatenation

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	addConcatFilterAndMapping()	0	14	3
A	concatWithTranscoding()	0	7	1

<?php

namespace ProtoneMedia\LaravelFFMpeg\Exporters;

use ProtoneMedia\LaravelFFMpeg\Filesystem\Media;

trait HandlesConcatenation
{
    /**
     * @var boolean
     */
    protected $concatWithTranscoding = false;

    /**
     * @var boolean
     */
    protected $concatWithVideo = false;

    /**
     * @var boolean
     */
    protected $concatWithAudio = false;

    public function concatWithTranscoding(bool $hasVideo = true, bool $hasAudio = true): self
    {
        $this->concatWithTranscoding = true;
        $this->concatWithVideo       = $hasVideo;
        $this->concatWithAudio       = $hasAudio;

        return $this;
    }

    private function addConcatFilterAndMapping(Media $outputMedia)
    {
        $sources = $this->driver->getMediaCollection()->map(function ($media, $key) {
            return "[{$key}]";
        });

        $concatWithVideo = $this->concatWithVideo ? 1 : 0;
        $concatWithAudio = $this->concatWithAudio ? 1 : 0;

        $this->addFilter(

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

            $sources->implode(''),
            "concat=n={$sources->count()}:v={$concatWithVideo}:a={$concatWithAudio}",
            '[concat]'
        )->addFormatOutputMapping($this->format, $outputMedia, ['[concat]']);
    }
}