ProtoneMedia\LaravelFFMpeg\Drivers\InteractsWithMediaStreams

Complexity

Total Complexity

8

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getVideoStream()	0	4	1
A	getAudioStream()	0	4	1
A	getStreams()	0	9	2
A	getDurationInMiliseconds()	0	12	3
A	getDurationInSeconds()	0	3	1

<?php

namespace ProtoneMedia\LaravelFFMpeg\Drivers;

use FFMpeg\FFProbe\DataMapping\Stream;
use Illuminate\Support\Arr;
use ProtoneMedia\LaravelFFMpeg\Filesystem\Media;
use ProtoneMedia\LaravelFFMpeg\Filesystem\MediaCollection;

trait InteractsWithMediaStreams
{
    /**
     * Returns an array with all streams.
     *
     * @return array
     */
    public function getStreams(): array
    {
        if (!$this->isAdvancedMedia()) {

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

            return iterator_to_array($this->media->getStreams());
        }

        return $this->mediaCollection->map(function (Media $media) {
            return $this->fresh()->open(MediaCollection::make([$media]))->getStreams();

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

        })->collapse()->all();
    }

    /**
     * Gets the duration of the media from the first stream or from the format.
     */
    public function getDurationInMiliseconds(): int
    {
        $stream = Arr::first($this->getStreams());

        if ($stream->has('duration')) {
            return $stream->get('duration') * 1000;
        }

        $format = $this->getFormat();

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

        if ($format->has('duration')) {
            return $format->get('duration') * 1000;
        }
    }

    public function getDurationInSeconds(): int
    {
        return round($this->getDurationInMiliseconds() / 1000);
    }

    /**
     * Gets the first audio streams of the media.
     */
    public function getAudioStream(): ?Stream
    {
        return Arr::first($this->getStreams(), function (Stream $stream) {
            return $stream->isAudio();
        });
    }

    /**
     * Gets the first video streams of the media.
     */
    public function getVideoStream(): ?Stream
    {
        return Arr::first($this->getStreams(), function (Stream $stream) {
            return $stream->isVideo();
        });
    }
}