Request::getHeaders() - Code Metrics - Inspection of "Merge pull request #43 from Starbugstone/Admin-sec..." - Starbugstone/blogoc - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — Accessing-and-setting-the-site... ( 1806ac )

by Stone

created 2018-08-22 15:56 UTC

Request::getHeaders() A

↳ Parent: Request

Complexity

Conditions	2
Paths	2

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	3
nc	2
nop	0
dl	0
loc	6
rs	10
c	0
b	0
f	0

<?php

namespace Core\Dependency;

/**
 * Class Request
 * @package Core
 *
 * we are dealing with get, post and cookies here, all Superglobals
 * @SuppressWarnings(PHPMD.Superglobals)
 */
class Request
{

    /**
     * gets the data from a get or a post request
     * @param $key
     * @return mixed
     * @throws \Exception
     */
    public function getData($key)
    {
        $requestMethod = $_SERVER['REQUEST_METHOD'];
        if ($requestMethod === 'GET') {
            return $_GET[$key] ?? null;
        }
        if ($requestMethod === 'POST') {
            return $_POST[$key] ?? null;
        }
        throw new \Exception("Unknown Request Method");
    }

    /**
     * checks if the request is a XML HTTP REQUEST
     * @return bool
     */
    public function isXmlRequest()
    {
        if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) === 'xmlhttprequest') {
            return true;
        }
        return false;
    }

    /**
     * gets the referer of the request
     * @return string|null
     */
    public function getReferer()
    {
        return $_SERVER['HTTP_REFERER'] ?? null;
    }

    /**
     * constructs the base url of the site
     * @return string
     */
    public function getBaseUrl(): string
    {
        $host = $_SERVER['HTTP_HOST'];
        $https = !empty($_SERVER['HTTPS']) ? 'https' : 'http';
        return $https . '://' . $host . '/';
    }

    /**
     * Gettint the headers
     * @return array
     */
    public function getHeaders(): array
    {
        if (apache_request_headers() != false) {
            return apache_request_headers();

        }
        return [];
    }

}

1			<?php
2
3			namespace Core\Dependency;
4
5			/**
6			* Class Request
7			* @package Core
8			*
9			* we are dealing with get, post and cookies here, all Superglobals
10			* @SuppressWarnings(PHPMD.Superglobals)
11			*/
12			class Request
13			{
14
15			/**
16			* gets the data from a get or a post request
17			* @param $key
18			* @return mixed
19			* @throws \Exception
20			*/
21			public function getData($key)
22			{
23			$requestMethod = $_SERVER['REQUEST_METHOD'];
24			if ($requestMethod === 'GET') {
25			return $_GET[$key] ?? null;
26			}
27			if ($requestMethod === 'POST') {
28			return $_POST[$key] ?? null;
29			}
30			throw new \Exception("Unknown Request Method");
31			}
32
33			/**
34			* checks if the request is a XML HTTP REQUEST
35			* @return bool
36			*/
37			public function isXmlRequest()
38			{
39			if (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) === 'xmlhttprequest') {
40			return true;
41			}
42			return false;
43			}
44
45			/**
46			* gets the referer of the request
47			* @return string\|null
48			*/
49			public function getReferer()
50			{
51			return $_SERVER['HTTP_REFERER'] ?? null;
52			}
53
54			/**
55			* constructs the base url of the site
56			* @return string
57			*/
58			public function getBaseUrl(): string
59			{
60			$host = $_SERVER['HTTP_HOST'];
61			$https = !empty($_SERVER['HTTPS']) ? 'https' : 'http';
62			return $https . '://' . $host . '/';
63			}
64
65			/**
66			* Gettint the headers
67			* @return array
68			*/
69			public function getHeaders(): array
70			{
71			if (apache_request_headers() != false) {
72			return apache_request_headers();
			1 ignored issue – show Bug Best Practice introduced 2018-08-21 06:25 UTC by Report Bug Copy Issue Report The expression `return apache_request_headers()` could return the type `false` which is incompatible with the type-hinted return `array`. Consider adding an additional type-check to rule them out. Loading history...
73			}
74			return [];
75			}
76
77			}

Starbugstone / blogoc

Push — Accessing-and-setting-the-site... ( 1806ac )

Request::getHeaders() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like