RicardoFiorani\Adapter\AbstractServiceAdapter

Complexity

Total Complexity

13

Size/Duplication

Total Lines	139
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

97.22%

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getRawUrl()	0	4	1
A	setRawUrl()	0	4	1
A	getVideoId()	0	4	1
A	setVideoId()	0	4	1
A	getPattern()	0	4	1
A	setPattern()	0	4	1
A	getRenderer()	0	4	1
A	setRenderer()	0	4	1
A	getEmbedCode()	0	12	2
A	getScheme()	0	8	2

<?php
/**
 * Created by PhpStorm.
 * User: Ricardo Fiorani
 * Date: 29/08/2015
 * Time: 19:37.
 */
namespace RicardoFiorani\Adapter;

use RicardoFiorani\Exception\NotEmbeddableException;
use RicardoFiorani\Renderer\EmbedRendererInterface;

abstract class AbstractServiceAdapter implements VideoAdapterInterface
{
    /**
     * @var string
     */
    public $rawUrl;

    /**
     * @var string
     */
    public $videoId;

    /**
     * @var string
     */
    public $pattern;

    /**
     * @var EmbedRendererInterface
     */
    public $renderer;

    /**
     * AbstractVideoAdapter constructor.
     *
     * @param string $url
     * @param string $pattern
     * @param EmbedRendererInterface $renderer
     */
    public function __construct($url, $pattern, EmbedRendererInterface $renderer)
    {
        $this->rawUrl = $url;

Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space

        $this->pattern = $pattern;

Equals sign not aligned with surrounding assignments; expected 2 spaces but found 1 space

        $this->renderer = $renderer;
    }

    /**
     * Returns the input URL.
     *
     * @return string
     */
    public function getRawUrl()
    {
        return $this->rawUrl;
    }

    /**
     * @param string $rawUrl
     */
    public function setRawUrl($rawUrl)
    {
        $this->rawUrl = $rawUrl;
    }

    /**
     * @return string
     */
    public function getVideoId()
    {
        return $this->videoId;
    }

    /**
     * @param string $videoId
     */
    public function setVideoId($videoId)
    {
        $this->videoId = $videoId;
    }

    /**
     * @return string
     */
    public function getPattern()
    {
        return $this->pattern;
    }

    /**
     * @param string $pattern
     */
    public function setPattern($pattern)
    {
        $this->pattern = $pattern;
    }

    /**
     * @return EmbedRendererInterface
     */
    public function getRenderer()
    {
        return $this->renderer;
    }

    /**
     * @param EmbedRendererInterface $renderer
     */
    public function setRenderer($renderer)
    {
        $this->renderer = $renderer;
    }

    /**
     * @param int $width
     * @param int $height
     * @param bool $forceAutoplay
     * @param bool $forceSecure
     *
     * @return string
     * @throws NotEmbeddableException
     */
    public function getEmbedCode($width, $height, $forceAutoplay = false, $forceSecure = false)
    {
        if (false == $this->isEmbeddable()) {

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

            throw new NotEmbeddableException();
        }

        return $this->getRenderer()->renderVideoEmbedCode(
            $this->getEmbedUrl($forceAutoplay, $forceSecure),
            $width,
            $height
        );
    }

    /**
     * Switches the protocol scheme between http and https in case you want to force https
     *
     * @param bool|false $forceSecure
     * @return string

Should the return type not be string|false?

     */
    public function getScheme($forceSecure = false)
    {
        if ($forceSecure) {
            return 'https';
        }

        return parse_url($this->rawUrl, PHP_URL_SCHEME);
    }
}

As per coding style, files should not end with a newline character.