ProtoneMedia\LaravelFFMpeg\FFMpeg\RebuildsCommands

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	buildCommand()	0	8	1
A	rebuildCommandWithHeaders()	0	13	2

<?php

namespace ProtoneMedia\LaravelFFMpeg\FFMpeg;

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

trait RebuildsCommands
{
    use InteractsWithBeforeSavingCallbacks;
    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);

        $command = $this->rebuildCommandWithHeaders($command);
        $command = $this->rebuildCommandWithCallbacks($command);

        return $command;
    }

    private function rebuildCommandWithHeaders($command)
    {
        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();
    }
}