Issues in HttpRequest.php - All Issues - Inspection of "Major changes. Removed byjg/anydataset and added b..." - byjg/restserver - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9f0cc8...e7ce71 )

by Joao

created 2016-05-11 05:17 UTC

src/HttpRequest.php (2 issues)

Showing only issues like (Show All)

mismatching argument types.

Documentation Minor

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace ByJG\RestServer;

class HttpRequest
{

    protected $get;
    protected $post;
    protected $server;
    protected $session;
    protected $cookie;
    protected $request;

    /**
     *
     * @param array $get
     * @param array $post
     * @param array $server
     * @param array $session
     * @param array $cookie
     */
    public function __construct($get, $post, $server, $session, $cookie)
    {
        $this->get = $get;
        $this->post = $post;
        $this->server = $server;
        $this->session = $session;
        $this->cookie = $cookie;

        $this->request = array_merge($get, $post, $server, $session, $cookie);
    }

    /**
     * Get a parameter passed by GET (the same as $_GET). If not found return false.
     *
     * @param string $value
     * @return string|boolean
     */
    public function get($value)
    {
        if (!isset($this->get[$value])) {
            return false;
        } else {
            return $this->get[$value];
        }
    }

    /**
     * Get a parameter passed by POST (the same as $_POST). If not found return false.
     *
     * @param string $value
     * @return string|boolean
     */
    public function post($value)
    {
        if (!isset($this->post[$value])) {
            return false;
        } else {
            return $this->post[$value];
        }
    }

    /**
     * Get the parameters sent by server (the same as $_SERVER). If not found return false.
     *
     * @param string $value
     * @return string|boolean
     */
    public function server($value)
    {
        if (!isset($this->server[$value])) {
            return false;
        } else {
            return $this->server[$value];
        }
    }

    /**
     * Get a server session value(the same as $_SESSION). If not found return false.
     *
     * @param string $value
     * @return string|boolean
     */
    public function session($value)
    {
        if (!isset($this->session[$value])) {
            return false;
        } else {
            return $this->session[$value];
        }
    }

    /**
     * Get the cookie sent by the client (the same as $_COOKIE). If not found return false.
     *
     * @param string $value
     * @return string|boolean
     */
    public function cookie($value)
    {
        if (!isset($this->cookie[$value])) {
            return false;
        } else {
            return $this->cookie[$value];
        }
    }

    /**
     * Get a value from any of get, post, server, cookie or session. If not found return false.
     *
     * @param string $value
     * @return string|boolean
     */
    public function request($value)
    {
        if (!isset($this->request[$value])) {
            return false;
        } else {
            return $this->request[$value];
        }
    }

    private $payload;

    /**
     * Get the payload passed during the request(the same as php://input). If not found return empty.
     *
     * @return string
     */
    public function payload()
    {
        if (is_null($this->payload)) {
            $this->payload = file_get_contents("php://input");
        }

        return $this->payload;
    }

    /**
     * Use this method to get the CLIENT REQUEST IP.
     * Note that if you behing a Proxy, the variable REMOTE_ADDR will always have the same IP
     * @return string
     */
    public function getRequestIp()
    {
        if ($this->server('HTTP_CLIENT_IP') !== false) {
            $ipaddress = $this->server('HTTP_CLIENT_IP');
        } else if ($this->server('HTTP_X_FORWARDED_FOR') !== false) {
            $ipaddress = $this->server('HTTP_X_FORWARDED_FOR');
        } else if ($this->server('HTTP_X_FORWARDED') !== false) {
            $ipaddress = $this->server('HTTP_X_FORWARDED');
        } else if ($this->server('HTTP_FORWARDED_FOR') !== false) {
            $ipaddress = $this->server('HTTP_FORWARDED_FOR');
        } else if ($this->server('HTTP_FORWARDED') !== false) {
            $ipaddress = $this->server('HTTP_FORWARDED');
        } else if ($this->server('REMOTE_ADDR') !== false) {
            $ipaddress = $this->server('REMOTE_ADDR');
        } else {
            $ipaddress = 'UNKNOWN';
        }

        return $ipaddress;
    }

    /**
     * Use this method to get the SERVER NAME.
     * @param bool $port
     * @param bool $protocol
     * @return string
     */
    public function getRequestServer($port = false, $protocol = false)
    {
        if ($this->server('SERVER_NAME') !== false) {
            $servername = $this->server('SERVER_NAME');
        } else if ($this->server('HTTP_HOST' !== false)) {
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $servername = $this->server('HTTP_HOST');
        } else {
            $servername = $this->server('SERVER_ADDR');
        }

        if ($port && $this->server('SERVER_PORT' !== false)) {
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $servername .= ':' . $this->server('SERVER_PORT');
        }

        if ($protocol) {
            $servername = (($this->server('HTTPS') !== 'off' || $this->server('SERVER_PORT') == 443) ? "https://" : "http://") . $servername;
        }

        return $servername;
    }
}


1		<?php
2
3		namespace ByJG\RestServer;
4
5		class HttpRequest
6		{
7
8		protected $get;
9		protected $post;
10		protected $server;
11		protected $session;
12		protected $cookie;
13		protected $request;
14
15		/**
16		*
17		* @param array $get
18		* @param array $post
19		* @param array $server
20		* @param array $session
21		* @param array $cookie
22		*/
23		public function __construct($get, $post, $server, $session, $cookie)
24		{
25		$this->get = $get;
26		$this->post = $post;
27		$this->server = $server;
28		$this->session = $session;
29		$this->cookie = $cookie;
30
31		$this->request = array_merge($get, $post, $server, $session, $cookie);
32		}
33
34		/**
35		* Get a parameter passed by GET (the same as $_GET). If not found return false.
36		*
37		* @param string $value
38		* @return string\|boolean
39		*/
40	View Code Duplication	public function get($value)
41		{
42		if (!isset($this->get[$value])) {
43		return false;
44		} else {
45		return $this->get[$value];
46		}
47		}
48
49		/**
50		* Get a parameter passed by POST (the same as $_POST). If not found return false.
51		*
52		* @param string $value
53		* @return string\|boolean
54		*/
55	View Code Duplication	public function post($value)
56		{
57		if (!isset($this->post[$value])) {
58		return false;
59		} else {
60		return $this->post[$value];
61		}
62		}
63
64		/**
65		* Get the parameters sent by server (the same as $_SERVER). If not found return false.
66		*
67		* @param string $value
68		* @return string\|boolean
69		*/
70	View Code Duplication	public function server($value)
71		{
72		if (!isset($this->server[$value])) {
73		return false;
74		} else {
75		return $this->server[$value];
76		}
77		}
78
79		/**
80		* Get a server session value(the same as $_SESSION). If not found return false.
81		*
82		* @param string $value
83		* @return string\|boolean
84		*/
85	View Code Duplication	public function session($value)
86		{
87		if (!isset($this->session[$value])) {
88		return false;
89		} else {
90		return $this->session[$value];
91		}
92		}
93
94		/**
95		* Get the cookie sent by the client (the same as $_COOKIE). If not found return false.
96		*
97		* @param string $value
98		* @return string\|boolean
99		*/
100	View Code Duplication	public function cookie($value)
101		{
102		if (!isset($this->cookie[$value])) {
103		return false;
104		} else {
105		return $this->cookie[$value];
106		}
107		}
108
109		/**
110		* Get a value from any of get, post, server, cookie or session. If not found return false.
111		*
112		* @param string $value
113		* @return string\|boolean
114		*/
115	View Code Duplication	public function request($value)
116		{
117		if (!isset($this->request[$value])) {
118		return false;
119		} else {
120		return $this->request[$value];
121		}
122		}
123
124		private $payload;
125
126		/**
127		* Get the payload passed during the request(the same as php://input). If not found return empty.
128		*
129		* @return string
130		*/
131		public function payload()
132		{
133		if (is_null($this->payload)) {
134		$this->payload = file_get_contents("php://input");
135		}
136
137		return $this->payload;
138		}
139
140		/**
141		* Use this method to get the CLIENT REQUEST IP.
142		* Note that if you behing a Proxy, the variable REMOTE_ADDR will always have the same IP
143		* @return string
144		*/
145		public function getRequestIp()
146		{
147		if ($this->server('HTTP_CLIENT_IP') !== false) {
148		$ipaddress = $this->server('HTTP_CLIENT_IP');
149		} else if ($this->server('HTTP_X_FORWARDED_FOR') !== false) {
150		$ipaddress = $this->server('HTTP_X_FORWARDED_FOR');
151		} else if ($this->server('HTTP_X_FORWARDED') !== false) {
152		$ipaddress = $this->server('HTTP_X_FORWARDED');
153		} else if ($this->server('HTTP_FORWARDED_FOR') !== false) {
154		$ipaddress = $this->server('HTTP_FORWARDED_FOR');
155		} else if ($this->server('HTTP_FORWARDED') !== false) {
156		$ipaddress = $this->server('HTTP_FORWARDED');
157		} else if ($this->server('REMOTE_ADDR') !== false) {
158		$ipaddress = $this->server('REMOTE_ADDR');
159		} else {
160		$ipaddress = 'UNKNOWN';
161		}
162
163		return $ipaddress;
164		}
165
166		/**
167		* Use this method to get the SERVER NAME.
168		* @param bool $port
169		* @param bool $protocol
170		* @return string
171		*/
172		public function getRequestServer($port = false, $protocol = false)
173		{
174		if ($this->server('SERVER_NAME') !== false) {
175		$servername = $this->server('SERVER_NAME');
176		} else if ($this->server('HTTP_HOST' !== false)) {
		0 ignored issues – show Documentation introduced 2016-01-19 16:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this `'HTTP_HOST' !== false` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
177		$servername = $this->server('HTTP_HOST');
178		} else {
179		$servername = $this->server('SERVER_ADDR');
180		}
181
182		if ($port && $this->server('SERVER_PORT' !== false)) {
		0 ignored issues – show Documentation introduced 2016-01-19 16:27 UTC by Report Bug Copy Issue Report Show Similar Issues like this `'SERVER_PORT' !== false` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
183		$servername .= ':' . $this->server('SERVER_PORT');
184		}
185
186		if ($protocol) {
187		$servername = (($this->server('HTTPS') !== 'off' \|\| $this->server('SERVER_PORT') == 443) ? "https://" : "http://") . $servername;
188		}
189
190		return $servername;
191		}
192		}
193

byjg / restserver

Push — master ( 9f0cc8...e7ce71 )

src/HttpRequest.php (2 issues)

Showing only issues like (Show All)

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like