Streaming\Traits\Representations

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getRepresentations()	0	3	1
A	addRepresentation()	0	8	2
A	autoGenerateRepresentations()	0	10	2

<?php

/**
 * This file is part of the PHP-FFmpeg-video-streaming package.
 *
 * (c) Amin Yazdanpanah <contact@aminyazdanpanah.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Streaming\Traits;

use Streaming\AutoRepresentations;
use Streaming\Exception\Exception;
use Streaming\Representation;

trait Representations
{
    /** @var array */
    protected $representations = [];

    /**
     * @param Representation $rep
     * @return $this
     * @throws Exception
     */
    public function addRepresentation(Representation $rep)
    {
        if (!$this->format) {
            throw new Exception('Format has not been set');
        }

        $this->representations[] = $rep;
        return $this;
    }

    /**
     * @return array
     */
    public function getRepresentations(): array
    {
        return $this->representations;
    }

    /**
     * @param array $side_values
     * @param array|null $k_bitrate_values
     * @return $this
     * @throws Exception
     */
    public function autoGenerateRepresentations(array $side_values = null, array $k_bitrate_values = null)
    {
        if (!$this->format) {
            throw new Exception('Format has not been set');
        }

        $this->representations = (new AutoRepresentations($this->getMedia()->probe(), $side_values, $k_bitrate_values))

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

            ->get();

        return $this;
    }
}