System\Http\Request

Complexity

Total Complexity

31

Size/Duplication

Total Lines	282
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Importance

Changes

0

18 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	prepareUrl()	0	18	3
A	getValueOfRequest()	0	12	2
A	get()	0	7	2
A	post()	0	7	2
A	file()	0	7	2
A	setPost()	0	4	1
A	setGet()	0	4	1
A	setFile()	0	4	1
A	server()	0	4	1
A	method()	0	4	1
A	referer()	0	4	1
A	baseUrl()	0	4	1
A	url()	0	4	1
A	host()	0	4	1
A	isRequestToAdminManagement()	0	8	1
A	isMatchingRequestMethod()	0	18	5
A	canRequestContinue()	0	16	4

<?php

namespace System\Http;

use System\Application;

class Request
{
    /**
     * Application Object
     *
     * @var \System\Application
     */
    private $app;

    /**
     * Url
     *
     * @var string
     */
    private $url;

    /**
     * Base Url
     *
     * @var string
     */
    private $baseUrl;

    /**
     * Host
     *
     * @var string
     */
    private $host;

    /**
     * Constructor
     *
     * @param \System\Application $app
     */
    public function __construct(Application $app)
    {
        $this->app = $app;
    }

    /**
     * Prepare url
     *
     * @return void
     */
    public function prepareUrl()
    {
        $script = dirname($this->server('SCRIPT_NAME'));

        $requestUri = $this->server('REQUEST_URI');

        if (strpos($requestUri, '?')) {
            list($requestUri) = explode('?', $requestUri);
        }

        $this->url = cleanUrl($script, $requestUri);

        $REQUEST_PROTOCOL = $this->app->http->isSecure() ? 'https' : 'http';

The property http does not exist on object<System\Application>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->host = $REQUEST_PROTOCOL . '://' . $this->server('HTTP_HOST');

        $this->baseUrl = $this->host . $requestUri;
    }

    /**
     * Get value from spcefic request type
     *
     * @param array $requestType
     * @param string $key
     * @return mixed
     */
    private function getValueOfRequest($requestType, $key)
    {
        $value = array_get($requestType, $key);

        if (is_array($value)) {
            $value = array_filter($value);
        } else {
            $value = trim($value);
        }

        return $value;
    }

    /**
     * Get value from $_GET by the given key
     *
     * @param string $key
     * @return mixed
     */
    public function get($key = null)
    {
        if ($key) {

The expression $key 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.

            return $this->getValueOfRequest($_GET, $key);
        }
        return $_GET;
    }

    /**
     * Get value from $_POST by the given key
     *
     * @param string $key
     * @return mixed
     */
    public function post($key = null)
    {
        if ($key) {

The expression $key 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.

            return $this->getValueOfRequest($_POST, $key);
        }
        return $_POST;
    }

    /**
     * Get value from $_FILES by the given key
     *
     * @return mixed
     */
    public function file($key = null)
    {
        if ($key) {
            return $this->getValueOfRequest($_FILES, $key);
        }
        return $_FILES;
    }

    /**
     * Set value To $_POST For the given key
     *
     * @param string $key
     * @param mixed $value
     * @return void
     */
    public function setPost($key, $value)
    {
        $_POST[$key] = $value;
    }

    /**
     * Set value To $_GET For the given key
     *
     * @param string $key
     * @param mixed $value
     * @return void
     */
    public function setGet($key, $value)
    {
        $_GET[$key] = $value;
    }

    /**
     * Set value To $_FILES For the given key
     *
     * @param string $key
     * @param mixed $value
     * @return void
     */
    public function setFile($key, $value)
    {
        $_FILES[$key] = $value;
    }

    /**
     * Get value from $_SERVER by the given key
     *
     * @param string $key
     * @return string
     */
    public function server($key)
    {
        return array_get($_SERVER, $key);
    }

    /**
     * Get current request method
     *
     * @return string
     */
    public function method()
    {
        return $this->server('REQUEST_METHOD');
    }

    /**
     * Get the referer link
     *
     * @return string
     */
    public function referer()
    {
        return $this->server('HTTP_REFERER');
    }

    /**
     * Get full url of the script
     *
     * @return string
     */
    public function baseUrl()
    {
        return $this->baseUrl;
    }

    /**
     * Get only relative url (clean url)
     *
     * @return string
     */
    public function url()
    {
        return $this->url;
    }

    /**
     * Get only host
     *
     * @return string
     */
    public function host()
    {
        return $this->host;
    }

    /**
     * Check if the request to the admin panel
     *
     * @return bool
     */
    public function isRequestToAdminManagement()
    {
        $url = $this->url;
        $url = ltrim($url, '/');
        $url = explode('/', $url)[0];

        return $url == 'admin';
    }

    /**
     * Check the request method
     *
     * @return bool
     */
    public function isMatchingRequestMethod($methods = ['GET'])
    {
        if (!is_array($methods)) {
            $methods = [$methods];
        }

        if (empty($methods)) {
            $methods = ['GET'];
        }

        foreach ($methods as $method) {
            if ($this->method() == strtoupper($method)) {
                return true;
            }
        }

        return false;
    }

    /**
     * Check if the request can be Continued
     * @property object $load
     * @return bool
     */
    public function canRequestContinue($middlewares)
    {
        if (empty($middlewares)) {
            return true;
        }

        foreach ($middlewares as $middleware) {
            $output = $this->app->load->middleware($middleware)->handle();

The property load does not exist on object<System\Application>. Since you implemented __get, maybe consider adding a @property annotation.

            if (!$output) {
                return false;
            }
        }

        return true;
    }
}