ProtoneMedia\LaravelFFMpeg\FFMpeg\BuildsCommandsWithHttpHeaders

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	buildCommand()	0	15	2

<?php

namespace ProtoneMedia\LaravelFFMpeg\FFMpeg;

use FFMpeg\Format\FormatInterface;
use Illuminate\Support\Collection;

trait BuildsCommandsWithHttpHeaders
{
    use InteractsWithHttpHeaders;

    /**
     * Builds the command using the underlying library and then
     * prepends the input with the headers.
     *
     * @param \FFMpeg\Format\FormatInterface $format
     * @param string $outputPathfile
     * @return array
     */
    protected function buildCommand(FormatInterface $format, $outputPathfile)
    {
        $command = parent::buildCommand($format, $outputPathfile);

        if (empty($this->headers)) {
            return $command;
        }

        return Collection::make($command)->map(function ($command) {
            return static::mergeBeforePathInput(
                $command,
                $this->getPathfile(),

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

                static::compileHeaders($this->headers)
            );
        })->all();
    }
}