Jclyons52\PagePreview\Url

Complexity

Total Complexity

14

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	findFirst()	0	8	2
A	findAll()	0	9	2
A	extract()	0	8	2
A	parse()	0	10	2
B	formatRelativeToAbsolute()	0	20	5

<?php

namespace Jclyons52\PagePreview;

class Url
{
    public $original;

    /**
     * destructured array of url components from parse_url
     * @var array
     */
    public $components;
    
    public function __construct($url)
    {
        $this->original = $url;
        $this->components = $this->parse($url);

It seems like $this->parse($url) can also be of type false. However, the property $components is declared as type array. Maybe add an additional type check?

    }

    public static function findFirst($text)
    {
        if ($urls = self::extract($text)) {
            return new self($urls[0]);
        }
        
        return null;
    }
    
    public static function findAll($text)
    {
        if ($urls = self::extract($text)) {
            return array_map(function ($url) {
                return new self($url);
            }, $urls);
        }
        return null;
    }
    
    public static function extract($text)
    {
        if (preg_match_all('!https?://\S+!', $text, $matches) === 0) {
            return null;
        }
        
        return $matches[0];
    }
    
    public function parse($url)
    {
        $urlComponents = parse_url($url);

        if (filter_var($url, FILTER_VALIDATE_URL) === false) {
            throw new \Exception("url {$url} is invalid");
        }
        
        return $urlComponents;
    }
    
    public function formatRelativeToAbsolute($url)
    {
        if (substr($url, 0, 5) === "data:") {
            return $url;
        }

        $path = array_key_exists('path', $this->components) ? $this->components['path'] : '';

        if (filter_var($url, FILTER_VALIDATE_URL) !== false) {
            return $url;
        }
        if (substr($url, 0, 1) === '/') {
            return 'http://' . $this->components['host'] . $url;
        }

        $host = trim($this->components['host'], '/') . '/';
        $path = trim($path, '/') . '/';

        return 'http://' . $host . $path . $url;
    }
}