@@ -188,7 +188,7 @@ discard block |
||
| 188 | 188 | } |
| 189 | 189 | |
| 190 | 190 | if ($this->_enable_cache && $this->getMethod() === 'GET') { |
| 191 | - $cache_key = urlencode($request_uri) . implode(',', $send_headers); |
|
| 191 | + $cache_key = urlencode($request_uri).implode(',', $send_headers); |
|
| 192 | 192 | if (array_key_exists($cache_key, self::$_cache)) { |
| 193 | 193 | return self::$_cache[$cache_key]; |
| 194 | 194 | } |
@@ -208,7 +208,7 @@ discard block |
||
| 208 | 208 | curl_setopt($ch, CURLOPT_AUTOREFERER, true); |
| 209 | 209 | |
| 210 | 210 | if (!empty($send_headers)) curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers); |
| 211 | - if (!is_null($post_data)) { |
|
| 211 | + if (!is_null($post_data)) { |
|
| 212 | 212 | curl_setopt($ch, CURLOPT_POST, 1); |
| 213 | 213 | curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data); |
| 214 | 214 | } |
@@ -244,9 +244,9 @@ discard block |
||
| 244 | 244 | { |
| 245 | 245 | if (!self::is_absolute_url($url)) { |
| 246 | 246 | $isRelative = true; |
| 247 | - $uri = 'http://dummy.com/' . ltrim($url, '/'); |
|
| 247 | + $uri = 'http://dummy.com/'.ltrim($url, '/'); |
|
| 248 | 248 | throw new InvalidArgumentException('Curl::setGetVar() requires $url to be an absolute url'); |
| 249 | - } else { |
|
| 249 | + }else { |
|
| 250 | 250 | $isRelative = false; |
| 251 | 251 | $uri = $url; |
| 252 | 252 | } |
@@ -254,7 +254,7 @@ discard block |
||
| 254 | 254 | // try to parse uri |
| 255 | 255 | $parts = parse_url($uri); |
| 256 | 256 | if (!$parts) { |
| 257 | - throw new InvalidArgumentException("Can't parse URL: " . $uri); |
|
| 257 | + throw new InvalidArgumentException("Can't parse URL: ".$uri); |
|
| 258 | 258 | } |
| 259 | 259 | |
| 260 | 260 | // Parse params and add new variable |
@@ -270,21 +270,21 @@ discard block |
||
| 270 | 270 | |
| 271 | 271 | if ($user != '') { |
| 272 | 272 | // format in either user:[email protected] or [email protected] |
| 273 | - $user .= (isset($parts['pass']) && $parts['pass'] != '') ? ':' . $parts['pass'] . '@' : '@'; |
|
| 273 | + $user .= (isset($parts['pass']) && $parts['pass'] != '') ? ':'.$parts['pass'].'@' : '@'; |
|
| 274 | 274 | } |
| 275 | 275 | |
| 276 | 276 | $host = (isset($parts['host'])) ? $parts['host'] : ''; |
| 277 | - $port = (isset($parts['port']) && $parts['port'] != '') ? ':' . $parts['port'] : ''; |
|
| 277 | + $port = (isset($parts['port']) && $parts['port'] != '') ? ':'.$parts['port'] : ''; |
|
| 278 | 278 | $path = (isset($parts['path']) && $parts['path'] != '') ? $parts['path'] : ''; |
| 279 | 279 | |
| 280 | 280 | // handle URL params which are existing / new |
| 281 | - $params = ($params) ? '?' . http_build_query($params, null, $separator) : ''; |
|
| 281 | + $params = ($params) ? '?'.http_build_query($params, null, $separator) : ''; |
|
| 282 | 282 | |
| 283 | 283 | // keep fragments (anchors) intact. |
| 284 | - $fragment = (isset($parts['fragment']) && $parts['fragment'] != '') ? '#' . $parts['fragment'] : ''; |
|
| 284 | + $fragment = (isset($parts['fragment']) && $parts['fragment'] != '') ? '#'.$parts['fragment'] : ''; |
|
| 285 | 285 | |
| 286 | 286 | // Recompile URI segments |
| 287 | - $newUri = $scheme . '://' . $user . $host . $port . $path . $params . $fragment; |
|
| 287 | + $newUri = $scheme.'://'.$user.$host.$port.$path.$params.$fragment; |
|
| 288 | 288 | |
| 289 | 289 | if ($isRelative) { |
| 290 | 290 | return str_replace('http://dummy.com/', '', $newUri); |
@@ -301,10 +301,10 @@ discard block |
||
| 301 | 301 | { |
| 302 | 302 | // Strip off the query and fragment parts of the URL before checking |
| 303 | 303 | if (($queryPosition = strpos($url, '?')) !== false) { |
| 304 | - $url = substr($url, 0, $queryPosition - 1); |
|
| 304 | + $url = substr($url, 0, $queryPosition-1); |
|
| 305 | 305 | } |
| 306 | 306 | if (($hashPosition = strpos($url, '#')) !== false) { |
| 307 | - $url = substr($url, 0, $hashPosition - 1); |
|
| 307 | + $url = substr($url, 0, $hashPosition-1); |
|
| 308 | 308 | } |
| 309 | 309 | $colonPosition = strpos($url, ':'); |
| 310 | 310 | $slashPosition = strpos($url, '/'); |
@@ -38,9 +38,15 @@ discard block |
||
| 38 | 38 | */ |
| 39 | 39 | public function __construct(string $url = null, array $data = [], array $headers = []) |
| 40 | 40 | { |
| 41 | - if (!is_null($url)) $this->setURL($url); |
|
| 42 | - foreach ($data as $name => $value) $this->addDataParam($name, $value); |
|
| 43 | - foreach ($headers as $name => $value) $this->addHeader($name, $value); |
|
| 41 | + if (!is_null($url)) { |
|
| 42 | + $this->setURL($url); |
|
| 43 | + } |
|
| 44 | + foreach ($data as $name => $value) { |
|
| 45 | + $this->addDataParam($name, $value); |
|
| 46 | + } |
|
| 47 | + foreach ($headers as $name => $value) { |
|
| 48 | + $this->addHeader($name, $value); |
|
| 49 | + } |
|
| 44 | 50 | } |
| 45 | 51 | |
| 46 | 52 | /** |
@@ -97,7 +103,9 @@ discard block |
||
| 97 | 103 | |
| 98 | 104 | public function removeDataParam(string $name): Curl |
| 99 | 105 | { |
| 100 | - if (array_key_exists($name, $this->_data)) unset($this->_data[$name]); |
|
| 106 | + if (array_key_exists($name, $this->_data)) { |
|
| 107 | + unset($this->_data[$name]); |
|
| 108 | + } |
|
| 101 | 109 | return $this; |
| 102 | 110 | } |
| 103 | 111 | |
@@ -126,7 +134,9 @@ discard block |
||
| 126 | 134 | */ |
| 127 | 135 | public function removeHeader(string $name): Curl |
| 128 | 136 | { |
| 129 | - if (array_key_exists($name, $this->_headers)) unset($this->_headers[$name]); |
|
| 137 | + if (array_key_exists($name, $this->_headers)) { |
|
| 138 | + unset($this->_headers[$name]); |
|
| 139 | + } |
|
| 130 | 140 | return $this; |
| 131 | 141 | } |
| 132 | 142 | |
@@ -170,13 +180,17 @@ discard block |
||
| 170 | 180 | $request_uri = $this->getURL(); |
| 171 | 181 | |
| 172 | 182 | // Headers |
| 173 | - foreach ($this->getHeaders() as $k => $v) $send_headers[] = "{$k}: {$v}"; |
|
| 183 | + foreach ($this->getHeaders() as $k => $v) { |
|
| 184 | + $send_headers[] = "{$k}: {$v}"; |
|
| 185 | + } |
|
| 174 | 186 | |
| 175 | 187 | // Request Data |
| 176 | 188 | switch ($this->getMethod()) { |
| 177 | 189 | case 'GET': |
| 178 | 190 | case 'DELETE': |
| 179 | - foreach ($this->getDataParams() as $name => $value) $request_uri = $this->setGetVar($name, $value, $request_uri); |
|
| 191 | + foreach ($this->getDataParams() as $name => $value) { |
|
| 192 | + $request_uri = $this->setGetVar($name, $value, $request_uri); |
|
| 193 | + } |
|
| 180 | 194 | break; |
| 181 | 195 | |
| 182 | 196 | case 'POST': |
@@ -207,7 +221,9 @@ discard block |
||
| 207 | 221 | curl_setopt($ch, CURLINFO_HEADER_OUT, true); |
| 208 | 222 | curl_setopt($ch, CURLOPT_AUTOREFERER, true); |
| 209 | 223 | |
| 210 | - if (!empty($send_headers)) curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers); |
|
| 224 | + if (!empty($send_headers)) { |
|
| 225 | + curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers); |
|
| 226 | + } |
|
| 211 | 227 | if (!is_null($post_data)) { |
| 212 | 228 | curl_setopt($ch, CURLOPT_POST, 1); |
| 213 | 229 | curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data); |
@@ -217,7 +233,9 @@ discard block |
||
| 217 | 233 | $http_code = curl_getinfo($ch, CURLINFO_HTTP_CODE); |
| 218 | 234 | |
| 219 | 235 | $response = new CurlResponse($http_code, $result); |
| 220 | - if ($this->_enable_cache && isset($cache_key)) self::$_cache[$cache_key] = $response; |
|
| 236 | + if ($this->_enable_cache && isset($cache_key)) { |
|
| 237 | + self::$_cache[$cache_key] = $response; |
|
| 238 | + } |
|
| 221 | 239 | |
| 222 | 240 | return $response; |
| 223 | 241 | } |