RicardoFiorani\Adapter\Dailymotion\DailymotionServiceAdapter

Complexity

Total Complexity

13

Size/Duplication

Total Lines	95
Duplicated Lines	8.42 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getServiceName()	0	4	1
A	hasThumbnail()	0	4	1
A	getThumbNailSizes()	0	6	1
A	getThumbnail()	8	8	2
A	getSmallThumbnail()	0	5	1
A	getMediumThumbnail()	0	5	1
A	getLargeThumbnail()	0	5	1
A	getLargestThumbnail()	0	5	1
A	getEmbedUrl()	0	4	2
A	isEmbeddable()	0	4	1

<?php declare(strict_types=1);

namespace RicardoFiorani\Adapter\Dailymotion;

use RicardoFiorani\Adapter\AbstractServiceAdapter;
use RicardoFiorani\Adapter\Exception\InvalidThumbnailSizeException;
use RicardoFiorani\Renderer\EmbedRendererInterface;

class DailymotionServiceAdapter extends AbstractServiceAdapter
{
    const THUMBNAIL_DEFAULT = 'thumbnail';

    public function __construct(string $url, string $pattern, EmbedRendererInterface $renderer)
    {
        $videoId = strtok(basename($url), '_');
        $this->setVideoId($videoId);

        return parent::__construct($url, $pattern, $renderer);

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }

    public function getServiceName(): string
    {
        return 'Dailymotion';
    }

    public function hasThumbnail(): bool
    {
        return true;
    }

    public function getThumbNailSizes(): array
    {
        return array(
            self::THUMBNAIL_DEFAULT,
        );
    }

    /**
     * @throws InvalidThumbnailSizeException
     * @throws \RicardoFiorani\Adapter\Exception\InvalidUrlException
     */
    public function getThumbnail(string $size, bool $forceSecure = false): string

This method seems to be duplicated in your project.

    {
        if (false == in_array($size, $this->getThumbNailSizes())) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            throw new InvalidThumbnailSizeException();
        }

        return $this->getScheme($forceSecure) . '://www.dailymotion.com/' . $size . '/video/' . $this->videoId;
    }

    /**
     * @throws InvalidThumbnailSizeException
     * @throws \RicardoFiorani\Adapter\Exception\InvalidUrlException
     */
    public function getSmallThumbnail(bool $forceSecure = false): string
    {
        //Since this service does not provide other thumbnails sizes we just return the default size
        return $this->getThumbnail(self::THUMBNAIL_DEFAULT, $forceSecure);
    }

    /**
     * @throws InvalidThumbnailSizeException
     * @throws \RicardoFiorani\Adapter\Exception\InvalidUrlException
     */
    public function getMediumThumbnail(bool $forceSecure = false): string
    {
        //Since this service does not provide other thumbnails sizes we just return the default size
        return $this->getThumbnail(self::THUMBNAIL_DEFAULT, $forceSecure);
    }

    /**
     * @throws InvalidThumbnailSizeException
     * @throws \RicardoFiorani\Adapter\Exception\InvalidUrlException
     */
    public function getLargeThumbnail(bool $forceSecure = false): string
    {
        //Since this service does not provide other thumbnails sizes we just return the default size
        return $this->getThumbnail(self::THUMBNAIL_DEFAULT, $forceSecure);
    }

    /**
     * @throws InvalidThumbnailSizeException
     * @throws \RicardoFiorani\Adapter\Exception\InvalidUrlException
     */
    public function getLargestThumbnail(bool $forceSecure = false): string
    {
        //Since this service does not provide other thumbnails sizes we just return the default size
        return $this->getThumbnail(self::THUMBNAIL_DEFAULT, $forceSecure);
    }

    /**
     * @throws \RicardoFiorani\Adapter\Exception\InvalidUrlException
     */
    public function getEmbedUrl(bool $forceAutoplay = false, bool $forceSecure = false): string
    {
        return $this->getScheme($forceSecure) . '://www.dailymotion.com/embed/video/' . $this->videoId . ($forceAutoplay ? '?amp&autoplay=1' : '');

This line exceeds maximum limit of 120 characters; contains 147 characters

    }

    public function isEmbeddable(): bool
    {
        return true;
    }
}