Ajde_Embed_Vimeo

Complexity

Total Complexity

11

Size/Duplication

Total Lines	49
Duplicated Lines	34.69 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	convertUrlToEmbed()	0	7	3
A	getCode()	9	9	1
A	_getVimeoId()	8	11	4
A	getThumbnail()	0	16	3

<?php

class Ajde_Embed_Vimeo extends Ajde_Embed
{
    public function convertUrlToEmbed()
    {
        if (substr($this->_code, 0, 7) == 'http://' || substr($this->_code, 0, 8) == 'https://') {
            $vimid = $this->_getVimeoId();
            $this->_code = '<iframe id="player_'.$vimid.'" src="http://player.vimeo.com/video/'.$vimid.'?title=0&amp;byline=0&amp;portrait=0&amp;api=1&amp;player_id=player_'.$vimid.'" width="400" height="225" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe>';
        }
    }

    public function getCode()

This method seems to be duplicated in your project.

    {
        $this->convertUrlToEmbed();
        $this->_setHeight();
        $this->_setWidth();
        $this->_code = str_replace('portrait=0', 'portrait=0', $this->_code);

        return $this->_code;
    }

    private function _getVimeoId()
    {
        if (substr($this->_code, 0, 7) == 'http://' || substr($this->_code, 0, 8) == 'https://') {

This code seems to be duplicated across your project.

            return str_replace('/', '', parse_url($this->_code, PHP_URL_PATH));
        } else {
            $matches = [];
            preg_match('%video/([0-9]+?)[\/\?\"]%', $this->_code, $matches);

            return isset($matches[1]) ? $matches[1] : null;
        }
    }

    public function getThumbnail()
    {
        $vmid = $this->_getVimeoId();
        if ($vmid) {

The expression $vmid of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            $response = Ajde_Http_Curl::get("http://vimeo.com/api/v2/video/$vmid.php");
            try {
                $hash = unserialize($response);
            } catch (Exception $e) {
                Ajde_Exception_Log::logException(new Ajde_Exception('Could not parse result from Vimeo'));

new \Ajde_Exception('Could not parse result from Vimeo') is of type object<Ajde_Exception>, but the function expects a object<Throwable>.

                return;
            }

            return $hash[0]['thumbnail_large'];
        }
    }
}