Streaming\Metadata

Complexity

Total Complexity

18

Size/Duplication

Total Lines	171
Duplicated Lines	0 %

Importance

Changes	2
Bugs	2	Features	0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	getVideoStreams()	0	3	1
A	getResolutions()	0	7	2
A	streamToArray()	0	3	1
A	getVideoMetadata()	0	5	1
A	getStreamsMetadata()	0	34	4
A	__construct()	0	5	1
A	get()	0	5	1
A	repToArray()	0	6	1
A	getFormat()	0	3	1
A	getJson()	0	3	1
A	export()	0	3	1
A	saveAsJson()	0	13	3

<?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;


use FFMpeg\FFProbe\DataMapping\Format;
use FFMpeg\FFProbe\DataMapping\Stream as VideoStream;
use FFMpeg\FFProbe\DataMapping\StreamCollection;
use Streaming\Exception\InvalidArgumentException;

class Metadata
{
    /** @var Stream */
    private $stream;

    /** @var \FFMpeg\FFProbe\DataMapping\Format */
    private $format;

    /** @var \FFMpeg\FFProbe\DataMapping\StreamCollection */
    private $video_streams;

    /**
     * Metadata constructor.
     * @param Stream $stream
     */
    public function __construct(Stream $stream)
    {
        $this->stream = $stream;
        $this->format = $stream->getMedia()->getFormat();

It seems like $stream->getMedia()->getFormat() of type FFMpeg\Media\Video or Streaming\Media is incompatible with the declared type FFMpeg\FFProbe\DataMapping\Format of property $format.

The method getFormat() does not exist on Streaming\Media. Since you implemented __call, consider adding a @method annotation.

        $this->video_streams = $stream->getMedia()->getStreams();

It seems like $stream->getMedia()->getStreams() of type FFMpeg\Media\Video or Streaming\Media is incompatible with the declared type FFMpeg\FFProbe\DataMapping\StreamCollection of property $video_streams.

The method getStreams() does not exist on Streaming\Media. Since you implemented __call, consider adding a @method annotation.

    }

    /**
     * @return \FFMpeg\FFProbe\DataMapping\Format
     */
    public function getFormat(): Format
    {
        return $this->format;
    }

    /**
     * @return \FFMpeg\FFProbe\DataMapping\StreamCollection
     */
    public function getVideoStreams(): StreamCollection
    {
        return $this->video_streams;
    }

    /**
     * @param VideoStream $stream
     * @return array
     */
    private function streamToArray(VideoStream $stream): array
    {
        return $stream->all();
    }

    /**
     * @return mixed
     */
    private function getVideoMetadata(): array
    {
        return [
            'format' => $this->getFormat()->all(),
            'streams' => array_map([$this, 'streamToArray'], $this->getVideoStreams()->all())
        ];
    }

    /**
     * @param Representation $rep
     * @return array
     */
    private function repToArray(Representation $rep): array
    {
        return [
            "dimension" => strtoupper($rep->size2string()),
            "video_kilo_bitrate" => $rep->getKiloBitrate(),
            "audio_kilo_bitrate" => $rep->getAudioKiloBitrate() ?? "Not specified"
        ];
    }

    /**
     * @return array
     */
    private function getResolutions(): array
    {
        if (!method_exists($this->stream, 'getRepresentations')) {
            return [];
        }

        return array_map([$this, 'repToArray'], $this->stream->getRepresentations()->all());
    }


    /**
     * @return array
     */
    public function getStreamsMetadata(): array
    {
        $dirname = $this->stream->pathInfo(PATHINFO_DIRNAME);
        $basename = $this->stream->pathInfo(PATHINFO_BASENAME);
        $filename = $dirname . DIRECTORY_SEPARATOR . $basename;

        $technique = explode("\\", get_class($this->stream));
        $format = explode("\\", get_class($this->stream->getFormat()));

        $metadata = [
            "filename" => $filename,
            "size_of_stream_dir" => File::directorySize($dirname),
            "created_at" => file_exists($filename) ? date("Y-m-d H:i:s", filemtime($filename)) : 'The file has been deleted',
            "resolutions" => $this->getResolutions(),
            "format" => end($format),
            "streaming_technique" => end($technique)
        ];

        if ($this->stream instanceof DASH) {
            $metadata = array_merge($metadata, ["seg_duration" => $this->stream->getSegDuration()]);
        } elseif ($this->stream instanceof HLS) {
            $metadata = array_merge(
                $metadata,
                [
                    "hls_time" => (int)$this->stream->getHlsTime(),
                    "hls_cache" => (bool)$this->stream->isHlsAllowCache(),
                    "encrypted_hls" => (bool)$this->stream->getHlsKeyInfoFile(),
                    "ts_sub_directory" => $this->stream->getSegSubDirectory(),
                    "base_url" => $this->stream->getHlsBaseUrl()
                ]
            );
        }

        return $metadata;
    }

    /**
     * @return array
     */
    public function get(): array
    {
        return [
            "video" => $this->getVideoMetadata(),
            "stream" => $this->getStreamsMetadata()
        ];
    }

    /**
     * @param null $opts

Are you sure the doc-type for parameter $opts is correct as it would always require null to be passed?

     * @return string
     */
    public function getJson($opts = null): string
    {
        return json_encode($this->get(), $opts ?? JSON_PRETTY_PRINT);
    }

    /**
     * @param string $filename
     * @param int $opts
     * @return string
     */
    public function saveAsJson(string $filename = null, int $opts = null): string
    {
        if (is_null($filename)) {
            if ($this->stream->isTmpDir()) {
                throw new InvalidArgumentException("It is a temp directory! It is not possible to save it");
            }

            $name = uniqid(($this->stream->pathInfo(PATHINFO_FILENAME) ?? "meta") . "-") . ".json";
            $filename = $this->stream->pathInfo(PATHINFO_DIRNAME) . DIRECTORY_SEPARATOR . $name;
        }
        File::put($filename, $this->getJson($opts));

It seems like $opts can also be of type integer; however, parameter $opts of Streaming\Metadata::getJson() does only seem to accept null, maybe add an additional type check?

        return $filename;
    }

    /**
     * @param string|null $save_to
     * @param int|null $opts
     * @return array
     */
    public function export(string $save_to = null, int $opts = null): array
    {
        return array_merge($this->get(), ['filename' => $this->saveAsJson($save_to, $opts)]);
    }
}