RicardoFiorani\Adapter\AbstractServiceAdapter

Complexity

Total Complexity

13

Size/Duplication

Total Lines	131
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

96.55%

Importance

Changes	5
Bugs	1	Features	0

11 Methods

Rating	Name	Duplication	Size	Complexity
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	__construct()	0	6	1
A	getEmbedCode()	0	8	2
A	getScheme()	0	4	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 $autoplay
     *
     * @return string
     *
     * @throws NotEmbeddableException
     */
    public function getEmbedCode($width, $height, $autoplay = false, $secure = 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($autoplay, $secure), $width, $height);
    }

    /**
     * Switches the protocol scheme between http and https
     *
     * @param bool|false $secure
     * @return string
     */
    public function getScheme($secure = false)
    {
        return ($secure ? 'https' : 'http');
    }
}

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