Request::getServer() - Code Metrics - Inspection of "Fix error in Request::init() on PHP 7.4 when parse..." - canax/request - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 7073ad...8f5ab8 )

by Mikael

created 2020-04-22 14:50 UTC

Request::getServer() A

↳ Parent: Request

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	4
cts	4
cp	1
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	2
crap	2

<?php

namespace Anax\Request;

/**
 * Storing information from the request and calculating related essentials.
 *
 */
class Request
{
    /**
     * @var string $requestUri Request URI from $_SERVER.
     * @var string $scriptName Scriptname from $_SERVER, actual scriptname part.
     * @var string $path       Scriptname from $_SERVER, path-part.
     */
    private $requestUri;
    private $scriptName;
    private $path;



    /**
     * @var string $route      The route.
     * @var array  $routeParts The route as an array.
     */
    private $route;
    private $routeParts;



    /**
     * @var string $currentUrl Current url.
     * @var string $siteUrl    Url to this site, http://dbwebb.se.
     * @var string $baseUrl    Url to root dir,
     *                         siteUrl . /some/installation/directory/.
     */
    private $currentUrl;
    private $siteUrl;
    private $baseUrl;



    /**
     * @var array $server Mapped to $_SERVER.
     * @var array $get    Mapped to $_GET.
     * @var array $post   Mapped to $_POST.
     * @var array $body   Mapped to request body, defaults to php://input.
     */
    private $server;
    private $get;
    private $post;
    private $body;



    /**
     * Constructor.
     */
    public function __construct()
    {
        $this->setGlobals();
    }



    /**
     * Read info from the globals.
     *
     * @param array $globals use to initiate globals with values.
     *
     * @return void
     */
    public function setGlobals($globals = [])
    {
        $this->server = [];
        $this->get = [];
        $this->post = [];

        $this->server = isset($globals["server"])
            ? array_merge($_SERVER, $globals["server"])
            : $_SERVER;

        $this->get = isset($globals["get"])
            ? array_merge($_GET, $globals["get"])
            : $_GET;

        $this->post = isset($globals["post"])
            ? array_merge($_POST, $globals["post"])
            : $_POST;
    }



    /**
     * Read info from the globals.
     *
     * @param array $globals use to initiate globals with values.
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     *
     * @return void
     */
    public function unsetGlobals()
    {
        $this->server = [];
        $this->get = [];
        $this->post = [];
    }



    /**
     * Init the request class by reading information from the request.
     *
     * @return $this
     */
    public function init()
    {
        $this->requestUri = rawurldecode($this->getServer("REQUEST_URI"));
        $scriptName = rawurldecode($this->getServer("SCRIPT_NAME"));
        $this->path = rtrim(dirname($scriptName), "/");
        $this->scriptName = basename($scriptName);

        // The route and its parts
        $this->extractRoute();

        // Prepare to create siteUrl and baseUrl by using currentUrl
        $this->currentUrl = $this->getCurrentUrl();
        $parts = parse_url($this->currentUrl);
        if ($parts === false) {
            $this->siteUrl = null;
            $this->baseUrl = null;
            return $this;
        }

        // Build the url from its parts
        $this->siteUrl = "{$parts["scheme"]}://{$parts["host"]}"
            . (isset($parts["port"])
                ? ":{$parts["port"]}"
                : "");
        $this->baseUrl = $this->siteUrl . $this->path;

        return $this;
    }



    /**
     * Get site url including scheme, host and port.
     *
     * @return string
     */
    public function getSiteUrl()
    {
        return $this->siteUrl;
    }



    /**
     * Get base url including site url and path to current index.php.
     *
     * @return string
     */
    public function getBaseUrl()
    {
        return $this->baseUrl;
    }



    /**
     * Get script name, index.php or other.
     *
     * @return string
     */
    public function getScriptName()
    {
        return $this->scriptName;
    }



    /**
     * Get route path parts in an array.
     *
     * @return array with route in its parts
     */
    public function getRouteParts()
    {
        return $this->routeParts;
    }



    /**
     * Get route path as a string.
     *
     * @return string as the current extracted route
     */
    public function getRoute()
    {
        return $this->route;
    }



    /**
     * Get the request method.
     *
     * @return string as the request method
     */
    public function getMethod()
    {
        return $this->getServer("REQUEST_METHOD");
    }



    /**
     * Extract the part containing the route.
     *
     * @todo Should be private, or useful in test?
     *
     * @return string as the current extracted route
     */
    public function extractRoute()
    {
        $requestUri = $this->requestUri;
        $scriptPath = $this->path;
        $scriptFile = $this->scriptName;

        // Compare REQUEST_URI and SCRIPT_NAME as long they match,
        // leave the rest as current request.
        $i = 0;
        $len = min(strlen($requestUri), strlen($scriptPath));
        while ($i < $len
               && $requestUri[$i] == $scriptPath[$i]
        ) {
            $i++;
        }
        $route = trim(substr($requestUri, $i), "/");

        // Does the request start with script-name - remove it.
        $len1 = strlen($route);
        $len2 = strlen($scriptFile);

        if ($len2 <= $len1
            && substr_compare($scriptFile, $route, 0, $len2, true) === 0
        ) {
            $route = substr($route, $len2 + 1);
        }

        // Remove the ?-part from the query when analysing controller/metod/arg1/arg2
        $queryPos = strpos($route, "?");
        if ($queryPos !== false) {
            $route = substr($route, 0, $queryPos);
        }

        $route = ($route === false) ? "" : $route;

        $this->route = $route;
        $this->routeParts = explode("/", trim($route, "/"));

        return $this->route;
    }



    /**
     * Get the current url.
     *
     * @param boolean $queryString attach query string, default is true.
     *
     * @return string as current url.
     */
    public function getCurrentUrl($queryString = true)
    {
        $port  = $this->getServer("SERVER_PORT");
        $https = $this->getServer("HTTPS") == "on" ? true : false;

        $scheme = $https
            ? "https"
            : $this->getServer("REQUEST_SCHEME", "http");

        $server = $this->getServer("SERVER_NAME")
            ?: $this->getServer("HTTP_HOST");

        $port  = ($port === "80")
            ? ""
            : (($port == 443 && $https)
                ? ""
                : ":" . $port);

        $uri = rawurldecode($this->getServer("REQUEST_URI"));
        $uri = $queryString
            ? rtrim($uri, "/")
            : rtrim(strtok($uri, "?"), "/");

        $url  = htmlspecialchars($scheme) . "://";
        $url .= htmlspecialchars($server)
            . $port . htmlspecialchars(rawurldecode($uri));

        return $url;
    }



    /**
     * Get a value from the _SERVER array and use default if it is not set.
     *
     * @param string $key     to check if it exists in the $_SERVER variable,
     *                        or empty to get whole array.
     * @param mixed  $default value to return as default
     *
     * @return mixed
     */
    public function getServer($key = null, $default = null)
    {
        if ($key) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            return $this->server[$key] ?? $default;
        }

        return $this->server;
    }



    /**
     * Set variable in the server array.
     *
     * @param mixed  $key   the key an the , or an key-value array
     * @param string $value the value of the key
     *
     * @return self
     */
    public function setServer($key, $value = null) : object

    {
        if (is_array($key)) {
            $this->server = array_merge($this->server, $key);
        } else {
            $this->server[$key] = $value;
        }
        return $this;
    }



    /**
     * Check if the value from the _GET array exists.
     *
     * @param string $key     to check if it exists in the $_GET variable
     *
     * @return boolean
     */
    public function hasGet($key)
    {
        return array_key_exists($key, $this->get);
    }



    /**
     * Get a value from the _GET array and use default if it is not set.
     *
     * @param string $key     to check if it exists in the $_GET variable,
     *                        or empty to get whole array.
     * @param mixed  $default value to return as default
     *
     * @return mixed
     */
    public function getGet($key = null, $default = null)

    {
        if ($key) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            return $this->get[$key] ?? $default;
        }

        return $this->get;
    }



    /**
     * Set variable in the get array.
     *
     * @param mixed  $key   the key an the value, or an key-value array
     * @param string $value the value of the key
     *
     * @return self
     */
    public function setGet($key, $value = null) : object

    {
        if (is_array($key)) {
            $this->get = array_merge($this->get, $key);
        } else {
            $this->get[$key] = $value;
        }
        return $this;
    }



    /**
     * Get a value from the _POST array and use default if it is not set.
     *
     * @param string $key     to check if it exists in the $_POST variable,
     *                        or empty to get whole array.
     * @param mixed  $default value to return as default
     *
     * @return mixed
     */
    public function getPost($key = null, $default = null)

    {
        if ($key) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            return $this->post[$key] ?? $default;
        }

        return $this->post;
    }



    /**
     * Set variable in the post array.
     *
     * @param mixed  $key   the key an the value, or an key-value array
     * @param string $value the value of the key
     *
     * @return self
     */
    public function setPost($key, $value = null) : object

    {
        if (is_array($key)) {
            $this->post = array_merge($this->post, $key);
        } else {
            $this->post[$key] = $value;
        }
        return $this;
    }



    /**
     * Set the request body (useful for unit testing).
     *
     * @return self
     */
    public function setBody($body)
    {
        $this->body = $body;
    }



    /**
     * Get the request body.
     *
     * @return mixed
     */
    public function getBody()
    {
        return isset($this->body)
            ? $this->body
            : file_get_contents("php://input");
    }



    /**
     * Get the request body from the HTTP request and treat it as
     * JSON data.
     *
     * @throws Anax\Request\Exception when request body is invalid JSON.
     *
     * @return mixed as the JSON converted content.
     */
    public function getBodyAsJson()
    {
        $entry = json_decode($this->getBody(), true);
        if (is_null($entry)) {
            throw new Exception("Could not read HTTP request body as JSON.");
        }
        return $entry;
    }
}


1			<?php
2
3			namespace Anax\Request;
4
5			/**
6			* Storing information from the request and calculating related essentials.
7			*
8			*/
9			class Request
10			{
11			/**
12			* @var string $requestUri Request URI from $_SERVER.
13			* @var string $scriptName Scriptname from $_SERVER, actual scriptname part.
14			* @var string $path Scriptname from $_SERVER, path-part.
15			*/
16			private $requestUri;
17			private $scriptName;
18			private $path;
19
20
21
22			/**
23			* @var string $route The route.
24			* @var array $routeParts The route as an array.
25			*/
26			private $route;
27			private $routeParts;
28
29
30
31			/**
32			* @var string $currentUrl Current url.
33			* @var string $siteUrl Url to this site, http://dbwebb.se.
34			* @var string $baseUrl Url to root dir,
35			* siteUrl . /some/installation/directory/.
36			*/
37			private $currentUrl;
38			private $siteUrl;
39			private $baseUrl;
40
41
42
43			/**
44			* @var array $server Mapped to $_SERVER.
45			* @var array $get Mapped to $_GET.
46			* @var array $post Mapped to $_POST.
47			* @var array $body Mapped to request body, defaults to php://input.
48			*/
49			private $server;
50			private $get;
51			private $post;
52			private $body;
53
54
55
56			/**
57			* Constructor.
58			*/
59	31		public function __construct()
60			{
61	31		$this->setGlobals();
62	31		}
63
64
65
66			/**
67			* Read info from the globals.
68			*
69			* @param array $globals use to initiate globals with values.
70			*
71			* @return void
72			*/
73	31		public function setGlobals($globals = [])
74			{
75	31		$this->server = [];
76	31		$this->get = [];
77	31		$this->post = [];
78
79	31		$this->server = isset($globals["server"])
80	27		? array_merge($_SERVER, $globals["server"])
81	31		: $_SERVER;
82
83	31		$this->get = isset($globals["get"])
84	1		? array_merge($_GET, $globals["get"])
85	31		: $_GET;
86
87	31		$this->post = isset($globals["post"])
88	1		? array_merge($_POST, $globals["post"])
89	31		: $_POST;
90	31		}
91
92
93
94			/**
95			* Read info from the globals.
96			*
97			* @param array $globals use to initiate globals with values.
			0 ignored issues – show Bug introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report There is no parameter named `$globals`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
98			*
99			* @return void
100			*/
101	4		public function unsetGlobals()
102			{
103	4		$this->server = [];
104	4		$this->get = [];
105	4		$this->post = [];
106	4		}
107
108
109
110			/**
111			* Init the request class by reading information from the request.
112			*
113			* @return $this
114			*/
115	11		public function init()
116			{
117	11		$this->requestUri = rawurldecode($this->getServer("REQUEST_URI"));
118	11		$scriptName = rawurldecode($this->getServer("SCRIPT_NAME"));
119	11		$this->path = rtrim(dirname($scriptName), "/");
120	11		$this->scriptName = basename($scriptName);
121
122			// The route and its parts
123	11		$this->extractRoute();
124
125			// Prepare to create siteUrl and baseUrl by using currentUrl
126	11		$this->currentUrl = $this->getCurrentUrl();
127	11		$parts = parse_url($this->currentUrl);
128	11		if ($parts === false) {
129	1		$this->siteUrl = null;
130	1		$this->baseUrl = null;
131	1		return $this;
132			}
133
134			// Build the url from its parts
135	10		$this->siteUrl = "{$parts["scheme"]}://{$parts["host"]}"
136	10		. (isset($parts["port"])
137	2		? ":{$parts["port"]}"
138	10		: "");
139	10		$this->baseUrl = $this->siteUrl . $this->path;
140
141	10		return $this;
142			}
143
144
145
146			/**
147			* Get site url including scheme, host and port.
148			*
149			* @return string
150			*/
151	4		public function getSiteUrl()
152			{
153	4		return $this->siteUrl;
154			}
155
156
157
158			/**
159			* Get base url including site url and path to current index.php.
160			*
161			* @return string
162			*/
163	4		public function getBaseUrl()
164			{
165	4		return $this->baseUrl;
166			}
167
168
169
170			/**
171			* Get script name, index.php or other.
172			*
173			* @return string
174			*/
175			public function getScriptName()
176			{
177			return $this->scriptName;
178			}
179
180
181
182			/**
183			* Get route path parts in an array.
184			*
185			* @return array with route in its parts
186			*/
187			public function getRouteParts()
188			{
189			return $this->routeParts;
190			}
191
192
193
194			/**
195			* Get route path as a string.
196			*
197			* @return string as the current extracted route
198			*/
199	6		public function getRoute()
200			{
201	6		return $this->route;
202			}
203
204
205
206			/**
207			* Get the request method.
208			*
209			* @return string as the request method
210			*/
211	1		public function getMethod()
212			{
213	1		return $this->getServer("REQUEST_METHOD");
214			}
215
216
217
218			/**
219			* Extract the part containing the route.
220			*
221			* @todo Should be private, or useful in test?
222			*
223			* @return string as the current extracted route
224			*/
225	11		public function extractRoute()
226			{
227	11		$requestUri = $this->requestUri;
228	11		$scriptPath = $this->path;
229	11		$scriptFile = $this->scriptName;
230
231			// Compare REQUEST_URI and SCRIPT_NAME as long they match,
232			// leave the rest as current request.
233	11		$i = 0;
234	11		$len = min(strlen($requestUri), strlen($scriptPath));
235	11		while ($i < $len
236	11		&& $requestUri[$i] == $scriptPath[$i]
237			) {
238	10		$i++;
239			}
240	11		$route = trim(substr($requestUri, $i), "/");
241
242			// Does the request start with script-name - remove it.
243	11		$len1 = strlen($route);
244	11		$len2 = strlen($scriptFile);
245
246	11		if ($len2 <= $len1
247	11		&& substr_compare($scriptFile, $route, 0, $len2, true) === 0
248			) {
249	10		$route = substr($route, $len2 + 1);
250			}
251
252			// Remove the ?-part from the query when analysing controller/metod/arg1/arg2
253	11		$queryPos = strpos($route, "?");
254	11		if ($queryPos !== false) {
255			$route = substr($route, 0, $queryPos);
256			}
257
258	11		$route = ($route === false) ? "" : $route;
259
260	11		$this->route = $route;
261	11		$this->routeParts = explode("/", trim($route, "/"));
262
263	11		return $this->route;
264			}
265
266
267
268			/**
269			* Get the current url.
270			*
271			* @param boolean $queryString attach query string, default is true.
272			*
273			* @return string as current url.
274			*/
275	26		public function getCurrentUrl($queryString = true)
276			{
277	26		$port = $this->getServer("SERVER_PORT");
278	26		$https = $this->getServer("HTTPS") == "on" ? true : false;
279
280	26		$scheme = $https
281	6		? "https"
282	26		: $this->getServer("REQUEST_SCHEME", "http");
283
284	26		$server = $this->getServer("SERVER_NAME")
285	26		?: $this->getServer("HTTP_HOST");
286
287	26		$port = ($port === "80")
288	15		? ""
289	11		: (($port == 443 && $https)
290	3		? ""
291	26		: ":" . $port);
292
293	26		$uri = rawurldecode($this->getServer("REQUEST_URI"));
294	26		$uri = $queryString
295	26		? rtrim($uri, "/")
296	26		: rtrim(strtok($uri, "?"), "/");
297
298	26		$url = htmlspecialchars($scheme) . "://";
299	26		$url .= htmlspecialchars($server)
300	26		. $port . htmlspecialchars(rawurldecode($uri));
301
302	26		return $url;
303			}
304
305
306
307			/**
308			* Get a value from the _SERVER array and use default if it is not set.
309			*
310			* @param string $key to check if it exists in the $_SERVER variable,
311			* or empty to get whole array.
312			* @param mixed $default value to return as default
313			*
314			* @return mixed
315			*/
316	29		public function getServer($key = null, $default = null)
317			{
318	29		if ($key) {
			0 ignored issues – show Bug Best Practice introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report 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. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
319	28		return $this->server[$key] ?? $default;
320			}
321
322	2		return $this->server;
323			}
324
325
326
327			/**
328			* Set variable in the server array.
329			*
330			* @param mixed $key the key an the , or an key-value array
331			* @param string $value the value of the key
332			*
333			* @return self
334			*/
335	20	View Code Duplication	public function setServer($key, $value = null) : object
			0 ignored issues – show Duplication introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
336			{
337	20		if (is_array($key)) {
338	1		$this->server = array_merge($this->server, $key);
339			} else {
340	20		$this->server[$key] = $value;
341			}
342	20		return $this;
343			}
344
345
346
347			/**
348			* Check if the value from the _GET array exists.
349			*
350			* @param string $key to check if it exists in the $_GET variable
351			*
352			* @return boolean
353			*/
354	1		public function hasGet($key)
355			{
356	1		return array_key_exists($key, $this->get);
357			}
358
359
360
361			/**
362			* Get a value from the _GET array and use default if it is not set.
363			*
364			* @param string $key to check if it exists in the $_GET variable,
365			* or empty to get whole array.
366			* @param mixed $default value to return as default
367			*
368			* @return mixed
369			*/
370	2	View Code Duplication	public function getGet($key = null, $default = null)
			0 ignored issues – show Duplication introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
371			{
372	2		if ($key) {
			0 ignored issues – show Bug Best Practice introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report 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. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
373	1		return $this->get[$key] ?? $default;
374			}
375
376	2		return $this->get;
377			}
378
379
380
381			/**
382			* Set variable in the get array.
383			*
384			* @param mixed $key the key an the value, or an key-value array
385			* @param string $value the value of the key
386			*
387			* @return self
388			*/
389	1	View Code Duplication	public function setGet($key, $value = null) : object
			0 ignored issues – show Duplication introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
390			{
391	1		if (is_array($key)) {
392	1		$this->get = array_merge($this->get, $key);
393			} else {
394	1		$this->get[$key] = $value;
395			}
396	1		return $this;
397			}
398
399
400
401			/**
402			* Get a value from the _POST array and use default if it is not set.
403			*
404			* @param string $key to check if it exists in the $_POST variable,
405			* or empty to get whole array.
406			* @param mixed $default value to return as default
407			*
408			* @return mixed
409			*/
410	2	View Code Duplication	public function getPost($key = null, $default = null)
			0 ignored issues – show Duplication introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
411			{
412	2		if ($key) {
			0 ignored issues – show Bug Best Practice introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report 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. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
413	1		return $this->post[$key] ?? $default;
414			}
415
416	2		return $this->post;
417			}
418
419
420
421			/**
422			* Set variable in the post array.
423			*
424			* @param mixed $key the key an the value, or an key-value array
425			* @param string $value the value of the key
426			*
427			* @return self
428			*/
429	1	View Code Duplication	public function setPost($key, $value = null) : object
			0 ignored issues – show Duplication introduced 2019-11-07 14:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
430			{
431	1		if (is_array($key)) {
432	1		$this->post = array_merge($this->post, $key);
433			} else {
434	1		$this->post[$key] = $value;
435			}
436	1		return $this;
437			}
438
439
440
441			/**
442			* Set the request body (useful for unit testing).
443			*
444			* @return self
445			*/
446			public function setBody($body)
447			{
448			$this->body = $body;
449			}
450
451
452
453			/**
454			* Get the request body.
455			*
456			* @return mixed
457			*/
458			public function getBody()
459			{
460			return isset($this->body)
461			? $this->body
462			: file_get_contents("php://input");
463			}
464
465
466
467			/**
468			* Get the request body from the HTTP request and treat it as
469			* JSON data.
470			*
471			* @throws Anax\Request\Exception when request body is invalid JSON.
472			*
473			* @return mixed as the JSON converted content.
474			*/
475			public function getBodyAsJson()
476			{
477			$entry = json_decode($this->getBody(), true);
478			if (is_null($entry)) {
479			throw new Exception("Could not read HTTP request body as JSON.");
480			}
481			return $entry;
482			}
483			}
484

canax / request

Push — master ( 7073ad...8f5ab8 )

Request::getServer() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like