Sunnysideup\ExternalURLField\ExternalURL

Complexity

Total Complexity

21

Size/Duplication

Total Lines	143
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	1

9 Methods

Rating	Name	Duplication	Size	Complexity
A	scaffoldFormField()	0	6	1
A	Domain()	0	7	2
A	Path()	0	7	2
A	forTemplate()	0	7	2
A	DomainShort()	0	11	2
A	__construct()	0	3	1
A	Nice()	0	15	4
A	saveInto()	0	32	6
A	NoWWW()	0	4	1

<?php

namespace Sunnysideup\ExternalURLField;

use SilverStripe\Core\Config\Config;
use SilverStripe\ORM\FieldType\DBVarchar;
use RuntimeException;

class ExternalURL extends DBVarchar
{
    private static $casting = [
        'Domain' => ExternalURL::class,
        'DomainShort' => 'Varchar',
        'URL' => ExternalURL::class,
        'Path' => 'Varchar',
    ];

    /**
     * 2083 is the lowest common denominator when it comes to url lengths.
     * however, 768 allows searching...
     *
     * @param null|mixed $name
     * @param mixed      $size
     * @param mixed      $options
     */
    public function __construct($name = null, $size = 2083, $options = [])
    {
        parent::__construct($name, $size, $options);
    }

    /**
     * Remove ugly parts of a url to make it nice.
     */
    public function Nice(): string
    {
        if ($this->value) {
            $parts = parse_url($this->URL());
            if ($parts) {

The expression $parts of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                $remove = ['scheme', 'user', 'pass', 'port', 'query', 'fragment'];
                foreach ($remove as $part) {
                    unset($parts[$part]);
                }
            }

            return rtrim(http_build_url($parts), '/');
        }

        return '';
    }

    /**
     * Get just the domain of the url.
     */
    public function Domain()
    {
        if ($this->value) {
            return parse_url($this->URL(), PHP_URL_HOST);
        }

        return '';
    }

    /**
     * Get just the domain of the url without the www subdomain and without https://.
     */
    public function DomainShort()
    {
        if ($this->value) {
            $parsedUrl = parse_url($this->URL());
            $host = $parsedUrl['host'] ?? '';
            // Remove 'www.' if present.
            $domain = preg_replace('/^www\./', '', $host);
            return $domain;
        }

        return '';
    }

    /**
     * Remove the www subdomain, if present.
     */
    public function NoWWW()
    {
        //https://stackoverflow.com/questions/23349257/trim-a-full-string-instead-of-using-trim-with-characters
        return $url = preg_replace('/^(www\.)*/', '', (string) $this->value);

The assignment to $url is dead and can be removed.

    }

    public function Path()
    {
        if ($this->value) {
            return trim((string) parse_url((string) $this->URL(), PHP_URL_PATH), '/');
        }

        return '';
    }

    /**
     * Scaffold the ExternalURLField for this ExternalURL.
     *
     * @param null|mixed $title
     * @param null|mixed $params
     */
    public function scaffoldFormField($title = null, $params = null)
    {
        $field = new ExternalURLField($this->name, $title);
        $field->setMaxLength($this->getSize());

        return $field;
    }

    public function forTemplate()
    {
        if ($this->value) {
            return (string) $this->URL();
        }

        return '';
    }

    public function saveInto($dataObject)
    {
        $url = (string) $this->value;
        if ($url) {
            $config = Config::inst()->get(ExternalURLField::class, 'default_config');
            $defaults = $config['defaultparts'];
            if (!preg_match('#^[a-zA-Z]+://#', $url)) {
                $url = $defaults['scheme'] . '://' . $url;
            }

            $parts = parse_url($url);
            if ($parts) {

The expression $parts of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

                //can't parse url, abort

                foreach (array_keys($parts) as $part) {
                    if (true === $config['removeparts'][$part]) {
                        unset($parts[$part]);
                    }
                }

                // this causes errors!
                // $parts = array_filter($defaults, fn ($default) => ! isset($parts[$part]));

                $this->value = rtrim(http_build_url($defaults, $parts), '/');
            } else {
                $this->value = '';
            }
        } else {
            $this->value = '';

        }
        parent::saveInto($dataObject);
    }
}