Soluble\MediaTools\Video\DetectionService

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getInterlaceDetect()	0	2	1
A	detectInterlacement()	0	5	1
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace Soluble\MediaTools\Video;

use Soluble\MediaTools\Config\FFMpegConfig;
use Soluble\MediaTools\Config\FFProbeConfig;
use Soluble\MediaTools\Exception\FileNotFoundException;
use Soluble\MediaTools\Video\Detection\InterlaceDetect;
use Soluble\MediaTools\Video\Detection\InterlaceGuess;
use Symfony\Component\Process\Exception\RuntimeException as SPRuntimeException;

class DetectionService implements DetectionServiceInterface
{
    /** @var FFProbeConfig */
    protected $ffprobeConfig;

    /** @var FFMpegConfig */
    protected $ffmpegConfig;

    /** @var array */
    protected $interlaceDetectCache;

    public function __construct(FFProbeConfig $ffProbeConfig, FFMpegConfig $ffmpegConfig)
    {
        $this->ffprobeConfig = $ffProbeConfig;
        $this->ffmpegConfig  = $ffmpegConfig;
    }

    public function getInterlaceDetect(): InterlaceDetect
    {
    }

In this branch, the function will implicitly return null which is incompatible with the type-hinted return Soluble\MediaTools\Video\Detection\InterlaceDetect. Consider adding a return statement or allowing null as return value.

    /**
     * @param int $maxFramesToAnalyze interlacement detection can be heavy, limit the number of frames to analyze
     *
     * @throws SPRuntimeException
     * @throws FileNotFoundException
     */
    public function detectInterlacement(string $file, int $maxFramesToAnalyze = 1000): InterlaceGuess
    {
        $interlaceDetect = new InterlaceDetect($this->ffmpegConfig);

        return $interlaceDetect->guessInterlacing($file, $maxFramesToAnalyze);
    }
}