Pbmedia\LaravelFFMpeg\Exporters\HandlesConcatenation

Complexity

Total Complexity

4

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

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

<?php

namespace Pbmedia\LaravelFFMpeg\Exporters;

use Pbmedia\LaravelFFMpeg\Filesystem\Media;

trait HandlesConcatenation
{
    protected bool $concatWithTranscoding = false;
    protected bool $concatWithVideo       = false;
    protected bool $concatWithAudio       = false;

    public function concatWithTranscoding($hasVideo = true, $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]']);
    }
}