WebServCo\Framework\Traits\RequestUrlTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getAppUrl()	0	10	2
A	getShortUrl()	0	6	1
A	queryToString()	0	10	3
A	getUrl()	0	11	3

<?php
namespace WebServCo\Framework\Traits;

trait RequestUrlTrait
{
    public function getAppUrl()
    {
        if (\WebServCo\Framework\Framework::isCLI()) {
            return false;
        }
        return $this->getSchema() .

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

        '://' .
        $this->getHost() .

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

        $this->path .

The property path does not exist on WebServCo\Framework\Traits\RequestUrlTrait. Did you maybe forget to declare it?

        DIRECTORY_SEPARATOR;
    }
    
    public function getShortUrl()
    {
        $url = $this->getAppUrl();
        $url .= $this->getTarget();

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

        $url .= $this->getSuffix();

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

        return $url;
    }
    
    public function getUrl($removeParameters = [])
    {
        $url = $this->getShortUrl();
        $query = $this->getQuery();

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

        foreach ($removeParameters as $item) {
            if (array_key_exists($item, $query)) {
                unset($query[$item]);
            }
        }
        $url .= $this->queryToString($query);
        return $url;
    }
    
    public function queryToString($query = [])
    {
        if (empty($query)) {
            return false;
        }
        $queries = [];
        foreach ($query as $k => $v) {
            $queries[] = sprintf('%s=%s', $k, $v);
        }
        return '?' . implode('&', $queries);
    }
}