Nip\Router\RequestContext\Traits\HasUrlFunctionsTrait

Complexity

Total Complexity

13

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	1	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	root()	0	3	1
A	getSchemeAndHttpHost()	0	3	1
A	getUri()	0	7	2
A	fullUrl()	0	5	3
A	getHttpHost()	0	11	5
A	url()	0	3	1

<?php

namespace Nip\Router\RequestContext\Traits;

/**
 * Trait HasUrlFunctionsTrait
 * @package Nip\Router\RequestContext\Traits
 */
trait HasUrlFunctionsTrait
{

    /**
     * Get the root URL for the application.
     *
     * @return string
     */
    public function root()
    {
        return rtrim($this->getSchemeAndHttpHost() . $this->getBaseUrl(), '/');

It seems like getBaseUrl() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }


    /**
     * Get the full URL for the request.
     *
     * @return string
     */
    public function fullUrl()
    {
        $query = $this->getQueryString();

It seems like getQueryString() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $question = $this->getBaseUrl() . $this->getPathInfo() == '/' ? '/?' : '?';

It seems like getPathInfo() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $query ? $this->url() . $question . $query : $this->url();
    }

    /**
     * Get the URL (no query string) for the request.
     *
     * @return string
     */
    public function url()
    {
        return rtrim(preg_replace('/\?.*/', '', $this->getUri()), '/');
    }

    /**
     * Generates a normalized URI (URL) for the Request.
     *
     * @return string A normalized URI (URL) for the Request
     *
     * @see getQueryString()
     */
    public function getUri()
    {
        if (null !== $queryString = $this->getQueryString()) {
            $queryString = '?' . $queryString;
        }

        return $this->getSchemeAndHttpHost() . $this->getBaseUrl() . $this->getPathInfo() . $queryString;
    }

    /**
     * @return string
     */
    public function getSchemeAndHttpHost()
    {
        return $this->getScheme() . '://' . $this->getHttpHost();

The method getScheme() does not exist on Nip\Router\RequestContext\Traits\HasUrlFunctionsTrait. Did you maybe mean getSchemeAndHttpHost()?

    }

    /**
     * @return string
     */
    public function getHttpHost()
    {
        $scheme = $this->getScheme();
        $port = '';
        if ('http' === $scheme && 80 != $this->getHttpPort()) {

The method getHttpPort() does not exist on Nip\Router\RequestContext\Traits\HasUrlFunctionsTrait. Did you maybe mean getHttpHost()?

            $port = ':' . $this->getHttpPort();
        } elseif ('https' === $scheme && 443 != $this->getHttpsPort()) {

The method getHttpsPort() does not exist on Nip\Router\RequestContext\Traits\HasUrlFunctionsTrait. Did you maybe mean getHttpHost()?

            $port = ':' . $this->getHttpsPort();
        }

        return $this->getHost() . $port;

It seems like getHost() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}