mmainstreet / jodel-web

php/Requests/libary/Requests.php

Doc Comments +3 added lines

@@ -226,6 +226,7 @@ 
 	 */
 	/**
 	 * Send a GET request
+	 * @param string $url
 	 */
 	public static function get($url, $headers = array(), $options = array()) {
 		return self::request($url, $headers, null, self::GET, $options);
@@ -263,12 +264,14 @@ 
 	 */
 	/**
 	 * Send a POST request
+	 * @param string $url
 	 */
 	public static function post($url, $headers = array(), $data = array(), $options = array()) {
 		return self::request($url, $headers, $data, self::POST, $options);
 	}
 	/**
 	 * Send a PUT request
+	 * @param string $url
 	 */
 	public static function put($url, $headers = array(), $data = array(), $options = array()) {
 		return self::request($url, $headers, $data, self::PUT, $options);

Spacing +89 added lines, -89 removed lines

@@ -137,12 +137,12 @@ 
 	 */
 	public static function autoloader($class) {
 		// Check that the class starts with "Requests"
-		if (strpos($class, 'Requests') !== 0) {
+		if(strpos($class, 'Requests') !== 0) {
 			return;
 		}
 
 		$file = str_replace('_', '/', $class);
-		if (file_exists(dirname(__FILE__) . '/' . $file . '.php')) {
+		if(file_exists(dirname(__FILE__) . '/' . $file . '.php')) {
 			require_once(dirname(__FILE__) . '/' . $file . '.php');
 		}
 	}
@@ -162,7 +162,7 @@ 
 	 * @param string $transport Transport class to add, must support the Requests_Transport interface
 	 */
 	public static function add_transport($transport) {
-		if (empty(self::$transports)) {
+		if(empty(self::$transports)) {
 			self::$transports = array(
 				'Requests_Transport_cURL',
 				'Requests_Transport_fsockopen',
@@ -186,12 +186,12 @@ 
 		$cap_string = serialize($capabilities);
 
 		// Don't search for a transport if it's already been done for these $capabilities
-		if (isset(self::$transport[$cap_string]) && self::$transport[$cap_string] !== null) {
+		if(isset(self::$transport[$cap_string]) && self::$transport[$cap_string] !== null) {
 			return new self::$transport[$cap_string]();
 		}
 		// @codeCoverageIgnoreEnd
 
-		if (empty(self::$transports)) {
+		if(empty(self::$transports)) {
 			self::$transports = array(
 				'Requests_Transport_cURL',
 				'Requests_Transport_fsockopen',
@@ -199,18 +199,18 @@ 
 		}
 
 		// Find us a working transport
-		foreach (self::$transports as $class) {
-			if (!class_exists($class)) {
+		foreach(self::$transports as $class) {
+			if(!class_exists($class)) {
 				continue;
 			}
 
 			$result = call_user_func(array($class, 'test'), $capabilities);
-			if ($result) {
+			if($result) {
 				self::$transport[$cap_string] = $class;
 				break;
 			}
 		}
-		if (self::$transport[$cap_string] === null) {
+		if(self::$transport[$cap_string] === null) {
 			throw new Requests_Exception('No working transports found', 'notransport', self::$transports);
 		}
 
@@ -355,7 +355,7 @@ 
 	 * @return Requests_Response
 	 */
 	public static function request($url, $headers = array(), $data = array(), $type = self::GET, $options = array()) {
-		if (empty($options['type'])) {
+		if(empty($options['type'])) {
 			$options['type'] = $type;
 		}
 		$options = array_merge(self::get_default_options(), $options);
@@ -364,10 +364,10 @@ 
 
 		$options['hooks']->dispatch('requests.before_request', array(&$url, &$headers, &$data, &$type, &$options));
 
-		if (!empty($options['transport'])) {
+		if(!empty($options['transport'])) {
 			$transport = $options['transport'];
 
-			if (is_string($options['transport'])) {
+			if(is_string($options['transport'])) {
 				$transport = new $transport();
 			}
 		}
@@ -427,29 +427,29 @@ 
 	public static function request_multiple($requests, $options = array()) {
 		$options = array_merge(self::get_default_options(true), $options);
 
-		if (!empty($options['hooks'])) {
+		if(!empty($options['hooks'])) {
 			$options['hooks']->register('transport.internal.parse_response', array('Requests', 'parse_multiple'));
-			if (!empty($options['complete'])) {
+			if(!empty($options['complete'])) {
 				$options['hooks']->register('multiple.request.complete', $options['complete']);
 			}
 		}
 
-		foreach ($requests as $id => &$request) {
-			if (!isset($request['headers'])) {
+		foreach($requests as $id => &$request) {
+			if(!isset($request['headers'])) {
 				$request['headers'] = array();
 			}
-			if (!isset($request['data'])) {
+			if(!isset($request['data'])) {
 				$request['data'] = array();
 			}
-			if (!isset($request['type'])) {
+			if(!isset($request['type'])) {
 				$request['type'] = self::GET;
 			}
-			if (!isset($request['options'])) {
+			if(!isset($request['options'])) {
 				$request['options'] = $options;
 				$request['options']['type'] = $request['type'];
 			}
 			else {
-				if (empty($request['options']['type'])) {
+				if(empty($request['options']['type'])) {
 					$request['options']['type'] = $request['type'];
 				}
 				$request['options'] = array_merge($options, $request['options']);
@@ -458,19 +458,19 @@ 
 			self::set_defaults($request['url'], $request['headers'], $request['data'], $request['type'], $request['options']);
 
 			// Ensure we only hook in once
-			if ($request['options']['hooks'] !== $options['hooks']) {
+			if($request['options']['hooks'] !== $options['hooks']) {
 				$request['options']['hooks']->register('transport.internal.parse_response', array('Requests', 'parse_multiple'));
-				if (!empty($request['options']['complete'])) {
+				if(!empty($request['options']['complete'])) {
 					$request['options']['hooks']->register('multiple.request.complete', $request['options']['complete']);
 				}
 			}
 		}
 		unset($request);
 
-		if (!empty($options['transport'])) {
+		if(!empty($options['transport'])) {
 			$transport = $options['transport'];
 
-			if (is_string($options['transport'])) {
+			if(is_string($options['transport'])) {
 				$transport = new $transport();
 			}
 		}
@@ -479,10 +479,10 @@ 
 		}
 		$responses = $transport->request_multiple($requests, $options);
 
-		foreach ($responses as $id => &$response) {
+		foreach($responses as $id => &$response) {
 			// If our hook got messed with somehow, ensure we end up with the
 			// correct response
-			if (is_string($response)) {
+			if(is_string($response)) {
 				$request = $requests[$id];
 				self::parse_multiple($response, $request);
 				$request['options']['hooks']->dispatch('multiple.request.complete', array(&$response, $id));
@@ -521,7 +521,7 @@ 
 			'verify' => Requests::get_certificate_path(),
 			'verifyname' => true,
 		);
-		if ($multirequest !== false) {
+		if($multirequest !== false) {
 			$defaults['complete'] = null;
 		}
 		return $defaults;
@@ -533,7 +533,7 @@ 
 	 * @return string Default certificate path.
 	 */
 	public static function get_certificate_path() {
-		if ( ! empty( Requests::$certificate_path ) ) {
+		if(!empty(Requests::$certificate_path)) {
 			return Requests::$certificate_path;
 		}
 
@@ -545,7 +545,7 @@ 
 	 *
 	 * @param string $path Certificate path, pointing to a PEM file.
 	 */
-	public static function set_certificate_path( $path ) {
+	public static function set_certificate_path($path) {
 		Requests::$certificate_path = $path;
 	}
 
@@ -560,39 +560,39 @@ 
 	 * @return array $options
 	 */
 	protected static function set_defaults(&$url, &$headers, &$data, &$type, &$options) {
-		if (!preg_match('/^http(s)?:\/\//i', $url, $matches)) {
+		if(!preg_match('/^http(s)?:\/\//i', $url, $matches)) {
 			throw new Requests_Exception('Only HTTP(S) requests are handled.', 'nonhttp', $url);
 		}
 
-		if (empty($options['hooks'])) {
+		if(empty($options['hooks'])) {
 			$options['hooks'] = new Requests_Hooks();
 		}
 
-		if (is_array($options['auth'])) {
+		if(is_array($options['auth'])) {
 			$options['auth'] = new Requests_Auth_Basic($options['auth']);
 		}
-		if ($options['auth'] !== false) {
+		if($options['auth'] !== false) {
 			$options['auth']->register($options['hooks']);
 		}
 
-		if (is_string($options['proxy']) || is_array($options['proxy'])) {
+		if(is_string($options['proxy']) || is_array($options['proxy'])) {
 			$options['proxy'] = new Requests_Proxy_HTTP($options['proxy']);
 		}
-		if ($options['proxy'] !== false) {
+		if($options['proxy'] !== false) {
 			$options['proxy']->register($options['hooks']);
 		}
 
-		if (is_array($options['cookies'])) {
+		if(is_array($options['cookies'])) {
 			$options['cookies'] = new Requests_Cookie_Jar($options['cookies']);
 		}
-		elseif (empty($options['cookies'])) {
+		elseif(empty($options['cookies'])) {
 			$options['cookies'] = new Requests_Cookie_Jar();
 		}
-		if ($options['cookies'] !== false) {
+		if($options['cookies'] !== false) {
 			$options['cookies']->register($options['hooks']);
 		}
 
-		if ($options['idn'] !== false) {
+		if($options['idn'] !== false) {
 			$iri = new Requests_IRI($url);
 			$iri->host = Requests_IDNAEncoder::encode($iri->ihost);
 			$url = $iri->uri;
@@ -601,8 +601,8 @@ 
 		// Massage the type to ensure we support it.
 		$type = strtoupper($type);
 
-		if (!isset($options['data_format'])) {
-			if (in_array($type, array(self::HEAD, self::GET, self::DELETE))) {
+		if(!isset($options['data_format'])) {
+			if(in_array($type, array(self::HEAD, self::GET, self::DELETE))) {
 				$options['data_format'] = 'query';
 			}
 			else {
@@ -627,15 +627,15 @@ 
 	 */
 	protected static function parse_response($headers, $url, $req_headers, $req_data, $options) {
 		$return = new Requests_Response();
-		if (!$options['blocking']) {
+		if(!$options['blocking']) {
 			return $return;
 		}
 
 		$return->raw = $headers;
 		$return->url = $url;
 
-		if (!$options['filename']) {
-			if (($pos = strpos($headers, "\r\n\r\n")) === false) {
+		if(!$options['filename']) {
+			if(($pos = strpos($headers, "\r\n\r\n")) === false) {
 				// Crap!
 				throw new Requests_Exception('Missing header/body separator', 'requests.no_crlf_separator');
 			}
@@ -652,44 +652,44 @@ 
 		$headers = preg_replace('/\n[ \t]/', ' ', $headers);
 		$headers = explode("\n", $headers);
 		preg_match('#^HTTP/(1\.\d)[ \t]+(\d+)#i', array_shift($headers), $matches);
-		if (empty($matches)) {
+		if(empty($matches)) {
 			throw new Requests_Exception('Response could not be parsed', 'noversion', $headers);
 		}
-		$return->protocol_version = (float) $matches[1];
-		$return->status_code = (int) $matches[2];
-		if ($return->status_code >= 200 && $return->status_code < 300) {
+		$return->protocol_version = (float)$matches[1];
+		$return->status_code = (int)$matches[2];
+		if($return->status_code >= 200 && $return->status_code < 300) {
 			$return->success = true;
 		}
 
-		foreach ($headers as $header) {
+		foreach($headers as $header) {
 			list($key, $value) = explode(':', $header, 2);
 			$value = trim($value);
 			preg_replace('#(\s+)#i', ' ', $value);
 			$return->headers[$key] = $value;
 		}
-		if (isset($return->headers['transfer-encoding'])) {
+		if(isset($return->headers['transfer-encoding'])) {
 			$return->body = self::decode_chunked($return->body);
 			unset($return->headers['transfer-encoding']);
 		}
-		if (isset($return->headers['content-encoding'])) {
+		if(isset($return->headers['content-encoding'])) {
 			$return->body = self::decompress($return->body);
 		}
 
 		//fsockopen and cURL compatibility
-		if (isset($return->headers['connection'])) {
+		if(isset($return->headers['connection'])) {
 			unset($return->headers['connection']);
 		}
 
 		$options['hooks']->dispatch('requests.before_redirect_check', array(&$return, $req_headers, $req_data, $options));
 
-		if ($return->is_redirect() && $options['follow_redirects'] === true) {
-			if (isset($return->headers['location']) && $options['redirected'] < $options['redirects']) {
-				if ($return->status_code === 303) {
+		if($return->is_redirect() && $options['follow_redirects'] === true) {
+			if(isset($return->headers['location']) && $options['redirected'] < $options['redirects']) {
+				if($return->status_code === 303) {
 					$options['type'] = self::GET;
 				}
 				$options['redirected']++;
 				$location = $return->headers['location'];
-				if (strpos($location, 'http://') !== 0 && strpos($location, 'https://') !== 0) {
+				if(strpos($location, 'http://') !== 0 && strpos($location, 'https://') !== 0) {
 					// relative redirect, for compatibility make it absolute
 					$location = Requests_IRI::absolutize($url, $location);
 					$location = $location->uri;
@@ -707,7 +707,7 @@ 
 				$redirected->history[] = $return;
 				return $redirected;
 			}
-			elseif ($options['redirected'] >= $options['redirects']) {
+			elseif($options['redirected'] >= $options['redirects']) {
 				throw new Requests_Exception('Too many redirects', 'toomanyredirects', $return);
 			}
 		}
@@ -736,7 +736,7 @@ 
 			$options = $request['options'];
 			$response = self::parse_response($response, $url, $headers, $data, $options);
 		}
-		catch (Requests_Exception $e) {
+		catch(Requests_Exception $e) {
 			$response = $e;
 		}
 	}
@@ -749,7 +749,7 @@ 
 	 * @return string Decoded body
 	 */
 	protected static function decode_chunked($data) {
-		if (!preg_match('/^([0-9a-f]+)(?:;(?:[\w-]*)(?:=(?:(?:[\w-]*)*|"(?:[^\r\n])*"))?)*\r\n/i', trim($data))) {
+		if(!preg_match('/^([0-9a-f]+)(?:;(?:[\w-]*)(?:=(?:(?:[\w-]*)*|"(?:[^\r\n])*"))?)*\r\n/i', trim($data))) {
 			return $data;
 		}
 
@@ -758,15 +758,15 @@ 
 		$decoded = '';
 		$encoded = $data;
 
-		while (true) {
-			$is_chunked = (bool) preg_match('/^([0-9a-f]+)(?:;(?:[\w-]*)(?:=(?:(?:[\w-]*)*|"(?:[^\r\n])*"))?)*\r\n/i', $encoded, $matches);
-			if (!$is_chunked) {
+		while(true) {
+			$is_chunked = (bool)preg_match('/^([0-9a-f]+)(?:;(?:[\w-]*)(?:=(?:(?:[\w-]*)*|"(?:[^\r\n])*"))?)*\r\n/i', $encoded, $matches);
+			if(!$is_chunked) {
 				// Looks like it's not chunked after all
 				return $data;
 			}
 
 			$length = hexdec(trim($matches[1]));
-			if ($length === 0) {
+			if($length === 0) {
 				// Ignore trailer headers
 				return $decoded;
 			}
@@ -775,7 +775,7 @@ 
 			$decoded .= substr($encoded, $chunk_length, $length);
 			$encoded = substr($encoded, $chunk_length + $length + 2);
 
-			if (trim($encoded) === '0' || empty($encoded)) {
+			if(trim($encoded) === '0' || empty($encoded)) {
 				return $decoded;
 			}
 		}
@@ -793,7 +793,7 @@ 
 	 */
 	public static function flatten($array) {
 		$return = array();
-		foreach ($array as $key => $value) {
+		foreach($array as $key => $value) {
 			$return[] = sprintf('%s: %s', $key, $value);
 		}
 		return $return;
@@ -821,21 +821,21 @@ 
 	 * @return string Decompressed string
 	 */
 	public static function decompress($data) {
-		if (substr($data, 0, 2) !== "\x1f\x8b" && substr($data, 0, 2) !== "\x78\x9c") {
+		if(substr($data, 0, 2) !== "\x1f\x8b" && substr($data, 0, 2) !== "\x78\x9c") {
 			// Not actually compressed. Probably cURL ruining this for us.
 			return $data;
 		}
 
-		if (function_exists('gzdecode') && ($decoded = @gzdecode($data)) !== false) {
+		if(function_exists('gzdecode') && ($decoded = @gzdecode($data)) !== false) {
 			return $decoded;
 		}
-		elseif (function_exists('gzinflate') && ($decoded = @gzinflate($data)) !== false) {
+		elseif(function_exists('gzinflate') && ($decoded = @gzinflate($data)) !== false) {
 			return $decoded;
 		}
-		elseif (($decoded = self::compatible_gzinflate($data)) !== false) {
+		elseif(($decoded = self::compatible_gzinflate($data)) !== false) {
 			return $decoded;
 		}
-		elseif (function_exists('gzuncompress') && ($decoded = @gzuncompress($data)) !== false) {
+		elseif(function_exists('gzuncompress') && ($decoded = @gzuncompress($data)) !== false) {
 			return $decoded;
 		}
 
@@ -865,26 +865,26 @@ 
 	public static function compatible_gzinflate($gzData) {
 		// Compressed data might contain a full zlib header, if so strip it for
 		// gzinflate()
-		if (substr($gzData, 0, 3) == "\x1f\x8b\x08") {
+		if(substr($gzData, 0, 3) == "\x1f\x8b\x08") {
 			$i = 10;
 			$flg = ord(substr($gzData, 3, 1));
-			if ($flg > 0) {
-				if ($flg & 4) {
+			if($flg > 0) {
+				if($flg&4) {
 					list($xlen) = unpack('v', substr($gzData, $i, 2));
 					$i = $i + 2 + $xlen;
 				}
-				if ($flg & 8) {
+				if($flg&8) {
 					$i = strpos($gzData, "\0", $i) + 1;
 				}
-				if ($flg & 16) {
+				if($flg&16) {
 					$i = strpos($gzData, "\0", $i) + 1;
 				}
-				if ($flg & 2) {
+				if($flg&2) {
 					$i = $i + 2;
 				}
 			}
 			$decompressed = self::compatible_gzinflate(substr($gzData, $i));
-			if (false !== $decompressed) {
+			if(false !== $decompressed) {
 				return $decompressed;
 			}
 		}
@@ -905,17 +905,17 @@ 
 		// First 2 bytes should be divisible by 0x1F
 		list(, $first_two_bytes) = unpack('n', $gzData);
 
-		if (0x08 == $first_nibble && 0 == ($first_two_bytes % 0x1F)) {
+		if(0x08 == $first_nibble && 0 == ($first_two_bytes % 0x1F)) {
 			$huffman_encoded = true;
 		}
 
-		if ($huffman_encoded) {
-			if (false !== ($decompressed = @gzinflate(substr($gzData, 2)))) {
+		if($huffman_encoded) {
+			if(false !== ($decompressed = @gzinflate(substr($gzData, 2)))) {
 				return $decompressed;
 			}
 		}
 
-		if ("\x50\x4b\x03\x04" == substr($gzData, 0, 4)) {
+		if("\x50\x4b\x03\x04" == substr($gzData, 0, 4)) {
 			// ZIP file format header
 			// Offset 6: 2 bytes, General-purpose field
 			// Offset 26: 2 bytes, filename length
@@ -927,9 +927,9 @@ 
 			// If the file has been compressed on the fly, 0x08 bit is set of
 			// the general purpose field. We can use this to differentiate
 			// between a compressed document, and a ZIP file
-			$zip_compressed_on_the_fly = (0x08 == (0x08 & $general_purpose_flag));
+			$zip_compressed_on_the_fly = (0x08 == (0x08&$general_purpose_flag));
 
-			if (!$zip_compressed_on_the_fly) {
+			if(!$zip_compressed_on_the_fly) {
 				// Don't attempt to decode a compressed zip file
 				return $gzData;
 			}
@@ -937,20 +937,20 @@ 
 			// Determine the first byte of data, based on the above ZIP header
 			// offsets:
 			$first_file_start = array_sum(unpack('v2', substr($gzData, 26, 4)));
-			if (false !== ($decompressed = @gzinflate(substr($gzData, 30 + $first_file_start)))) {
+			if(false !== ($decompressed = @gzinflate(substr($gzData, 30 + $first_file_start)))) {
 				return $decompressed;
 			}
 			return false;
 		}
 
 		// Finally fall back to straight gzinflate
-		if (false !== ($decompressed = @gzinflate($gzData))) {
+		if(false !== ($decompressed = @gzinflate($gzData))) {
 			return $decompressed;
 		}
 
 		// Fallback for all above failing, not expected, but included for
 		// debugging and preventing regressions and to track stats
-		if (false !== ($decompressed = @gzinflate(substr($gzData, 2)))) {
+		if(false !== ($decompressed = @gzinflate(substr($gzData, 2)))) {
 			return $decompressed;
 		}
 
@@ -959,7 +959,7 @@ 
 
 	public static function match_domain($host, $reference) {
 		// Check for a direct match
-		if ($host === $reference) {
+		if($host === $reference) {
 			return true;
 		}
 
@@ -967,10 +967,10 @@ 
 		// Also validates that the host has 3 parts or more, as per Firefox's
 		// ruleset.
 		$parts = explode('.', $host);
-		if (ip2long($host) === false && count($parts) >= 3) {
+		if(ip2long($host) === false && count($parts) >= 3) {
 			$parts[0] = '*';
 			$wildcard = implode('.', $parts);
-			if ($wildcard === $reference) {
+			if($wildcard === $reference) {
 				return true;
 			}
 		}

php/Requests/libary/Requests/IDNAEncoder.php

Doc Comments +1 added lines, -1 removed lines

@@ -357,7 +357,7 @@
 	 * @param int $delta
 	 * @param int $numpoints
 	 * @param bool $firsttime
-	 * @return int New bias
+	 * @return double New bias
 	 */
 	protected static function adapt($delta, $numpoints, $firsttime) {
 #	function adapt(delta,numpoints,firsttime):

Spacing +37 added lines, -37 removed lines

@@ -42,7 +42,7 @@ 
 	 */
 	public static function encode($string) {
 		$parts = explode('.', $string);
-		foreach ($parts as &$part) {
+		foreach($parts as &$part) {
 			$part = self::to_ascii($part);
 		}
 		return implode('.', $parts);
@@ -61,9 +61,9 @@ 
 	 */
 	public static function to_ascii($string) {
 		// Step 1: Check if the string is already ASCII
-		if (self::is_ascii($string)) {
+		if(self::is_ascii($string)) {
 			// Skip to step 7
-			if (strlen($string) < 64) {
+			if(strlen($string) < 64) {
 				return $string;
 			}
 
@@ -75,9 +75,9 @@ 
 
 		// Step 3: UseSTD3ASCIIRules is false, continue
 		// Step 4: Check if it's ASCII now
-		if (self::is_ascii($string)) {
+		if(self::is_ascii($string)) {
 			// Skip to step 7
-			if (strlen($string) < 64) {
+			if(strlen($string) < 64) {
 				return $string;
 			}
 
@@ -85,7 +85,7 @@ 
 		}
 
 		// Step 5: Check ACE prefix
-		if (strpos($string, self::ACE_PREFIX) === 0) {
+		if(strpos($string, self::ACE_PREFIX) === 0) {
 			throw new Requests_Exception('Provided string begins with ACE prefix', 'idna.provided_is_prefixed', $string);
 		}
 
@@ -96,7 +96,7 @@ 
 		$string = self::ACE_PREFIX . $string;
 
 		// Step 8: Check size
-		if (strlen($string) < 64) {
+		if(strlen($string) < 64) {
 			return $string;
 		}
 
@@ -141,30 +141,30 @@ 
 		// Get number of bytes
 		$strlen = strlen($input);
 
-		for ($position = 0; $position < $strlen; $position++) {
+		for($position = 0; $position < $strlen; $position++) {
 			$value = ord($input[$position]);
 
 			// One byte sequence:
-			if ((~$value & 0x80) === 0x80) {
+			if((~$value&0x80) === 0x80) {
 				$character = $value;
 				$length = 1;
 				$remaining = 0;
 			}
 			// Two byte sequence:
-			elseif (($value & 0xE0) === 0xC0) {
-				$character = ($value & 0x1F) << 6;
+			elseif(($value&0xE0) === 0xC0) {
+				$character = ($value&0x1F) << 6;
 				$length = 2;
 				$remaining = 1;
 			}
 			// Three byte sequence:
-			elseif (($value & 0xF0) === 0xE0) {
-				$character = ($value & 0x0F) << 12;
+			elseif(($value&0xF0) === 0xE0) {
+				$character = ($value&0x0F) << 12;
 				$length = 3;
 				$remaining = 2;
 			}
 			// Four byte sequence:
-			elseif (($value & 0xF8) === 0xF0) {
-				$character = ($value & 0x07) << 18;
+			elseif(($value&0xF8) === 0xF0) {
+				$character = ($value&0x07) << 18;
 				$length = 4;
 				$remaining = 3;
 			}
@@ -173,31 +173,31 @@ 
 				throw new Requests_Exception('Invalid Unicode codepoint', 'idna.invalidcodepoint', $value);
 			}
 
-			if ($remaining > 0) {
-				if ($position + $length > $strlen) {
+			if($remaining > 0) {
+				if($position + $length > $strlen) {
 					throw new Requests_Exception('Invalid Unicode codepoint', 'idna.invalidcodepoint', $character);
 				}
-				for ($position++; $remaining > 0; $position++) {
+				for($position++; $remaining > 0; $position++) {
 					$value = ord($input[$position]);
 
 					// If it is invalid, count the sequence as invalid and reprocess the current byte:
-					if (($value & 0xC0) !== 0x80) {
+					if(($value&0xC0) !== 0x80) {
 						throw new Requests_Exception('Invalid Unicode codepoint', 'idna.invalidcodepoint', $character);
 					}
 
-					$character |= ($value & 0x3F) << (--$remaining * 6);
+					$character |= ($value&0x3F) << (--$remaining * 6);
 				}
 				$position--;
 			}
 
-			if (
+			if(
 				// Non-shortest form sequences are invalid
 				   $length > 1 && $character <= 0x7F
 				|| $length > 2 && $character <= 0x7FF
 				|| $length > 3 && $character <= 0xFFFF
 				// Outside of range of ucschar codepoints
 				// Noncharacters
-				|| ($character & 0xFFFE) === 0xFFFE
+				|| ($character&0xFFFE) === 0xFFFE
 				|| $character >= 0xFDD0 && $character <= 0xFDEF
 				|| (
 					// Everything else not in ucschar
@@ -239,8 +239,8 @@ 
 		$codepoints = self::utf8_to_codepoints($input);
 		$extended = array();
 
-		foreach ($codepoints as $char) {
-			if ($char < 128) {
+		foreach($codepoints as $char) {
+			if($char < 128) {
 				// Character is valid ASCII
 				// TODO: this should also check if it's valid for a URL
 				$output .= chr($char);
@@ -249,7 +249,7 @@ 
 			// Check if the character is non-ASCII, but below initial n
 			// This never occurs for Punycode, so ignore in coverage
 			// @codeCoverageIgnoreStart
-			elseif ($char < $n) {
+			elseif($char < $n) {
 				throw new Requests_Exception('Invalid character', 'idna.character_outside_domain', $char);
 			}
 			// @codeCoverageIgnoreEnd
@@ -261,12 +261,12 @@ 
 		sort($extended);
 		$b = $h;
 #		[copy them] followed by a delimiter if b > 0
-		if (strlen($output) > 0) {
+		if(strlen($output) > 0) {
 			$output .= '-';
 		}
 #		{if the input contains a non-basic code point < n then fail}
 #		while h < length(input) do begin
-		while ($h < count($codepoints)) {
+		while($h < count($codepoints)) {
 #			let m = the minimum code point >= n in the input
 			$m = array_shift($extended);
 			//printf('next code point to insert is %s' . PHP_EOL, dechex($m));
@@ -275,31 +275,31 @@ 
 #			let n = m
 			$n = $m;
 #			for each code point c in the input (in order) do begin
-			for ($num = 0; $num < count($codepoints); $num++) {
+			for($num = 0; $num < count($codepoints); $num++) {
 				$c = $codepoints[$num];
 #				if c < n then increment delta, fail on overflow
-				if ($c < $n) {
+				if($c < $n) {
 					$delta++;
 				}
 #				if c == n then begin
-				elseif ($c === $n) {
+				elseif($c === $n) {
 #					let q = delta
 					$q = $delta;
 #					for k = base to infinity in steps of base do begin
-					for ($k = self::BOOTSTRAP_BASE; ; $k += self::BOOTSTRAP_BASE) {
+					for($k = self::BOOTSTRAP_BASE; ; $k += self::BOOTSTRAP_BASE) {
 #						let t = tmin if k <= bias {+ tmin}, or
 #								tmax if k >= bias + tmax, or k - bias otherwise
-						if ($k <= ($bias + self::BOOTSTRAP_TMIN)) {
+						if($k <= ($bias + self::BOOTSTRAP_TMIN)) {
 							$t = self::BOOTSTRAP_TMIN;
 						}
-						elseif ($k >= ($bias + self::BOOTSTRAP_TMAX)) {
+						elseif($k >= ($bias + self::BOOTSTRAP_TMAX)) {
 							$t = self::BOOTSTRAP_TMAX;
 						}
 						else {
 							$t = $k - $bias;
 						}
 #						if q < t then break
-						if ($q < $t) {
+						if($q < $t) {
 							break;
 						}
 #						output the code point for digit t + ((q - t) mod (base - t))
@@ -342,7 +342,7 @@ 
 	protected static function digit_to_char($digit) {
 		// @codeCoverageIgnoreStart
 		// As far as I know, this never happens, but still good to be sure.
-		if ($digit < 0 || $digit > 35) {
+		if($digit < 0 || $digit > 35) {
 			throw new Requests_Exception(sprintf('Invalid digit %d', $digit), 'idna.invalid_digit', $digit);
 		}
 		// @codeCoverageIgnoreEnd
@@ -362,7 +362,7 @@ 
 	protected static function adapt($delta, $numpoints, $firsttime) {
 #	function adapt(delta,numpoints,firsttime):
 #		if firsttime then let delta = delta div damp
-		if ($firsttime) {
+		if($firsttime) {
 			$delta = floor($delta / self::BOOTSTRAP_DAMP);
 		}
 #		else let delta = delta div 2
@@ -375,7 +375,7 @@ 
 		$k = 0;
 #		while delta > ((base - tmin) * tmax) div 2 do begin
 		$max = floor(((self::BOOTSTRAP_BASE - self::BOOTSTRAP_TMIN) * self::BOOTSTRAP_TMAX) / 2);
-		while ($delta > $max) {
+		while($delta > $max) {
 #			let delta = delta div (base - tmin)
 			$delta = floor($delta / (self::BOOTSTRAP_BASE - self::BOOTSTRAP_TMIN));
 #			let k = k + base

php/Requests/libary/Requests/IRI.php

Doc Comments +2 added lines, -1 removed lines

@@ -249,7 +249,7 @@ 
 	 *
 	 * Returns false if $base is not absolute, otherwise an IRI.
 	 *
-	 * @param IRI|string $base (Absolute) Base IRI
+	 * @param string $base (Absolute) Base IRI
 	 * @param IRI|string $relative Relative IRI
 	 * @return IRI|false
 	 */
@@ -984,6 +984,7 @@ 
 	 * Convert an IRI to a URI (or parts thereof)
 	 *
 	 * @param string|bool IRI to convert (or false from {@see get_iri})
+	 * @param false|string $string
 	 * @return string|false URI if IRI is valid, false otherwise.
 	 */
 	protected function to_uri($string) {

Spacing +120 added lines, -120 removed lines

@@ -153,10 +153,10 @@ 
 	 * @param mixed $value Property value
 	 */
 	public function __set($name, $value) {
-		if (method_exists($this, 'set_' . $name)) {
+		if(method_exists($this, 'set_' . $name)) {
 			call_user_func(array($this, 'set_' . $name), $value);
 		}
-		elseif (
+		elseif(
 			   $name === 'iauthority'
 			|| $name === 'iuserinfo'
 			|| $name === 'ihost'
@@ -179,7 +179,7 @@ 
 		// Also why we use array_key_exists below instead of isset()
 		$props = get_object_vars($this);
 
-		if (
+		if(
 			$name === 'iri' ||
 			$name === 'uri' ||
 			$name === 'iauthority' ||
@@ -188,16 +188,16 @@ 
 			$method = 'get_' . $name;
 			$return = $this->$method();
 		}
-		elseif (array_key_exists($name, $props)) {
+		elseif(array_key_exists($name, $props)) {
 			$return = $this->$name;
 		}
 		// host -> ihost
-		elseif (($prop = 'i' . $name) && array_key_exists($prop, $props)) {
+		elseif(($prop = 'i' . $name) && array_key_exists($prop, $props)) {
 			$name = $prop;
 			$return = $this->$prop;
 		}
 		// ischeme -> scheme
-		elseif (($prop = substr($name, 1)) && array_key_exists($prop, $props)) {
+		elseif(($prop = substr($name, 1)) && array_key_exists($prop, $props)) {
 			$name = $prop;
 			$return = $this->$prop;
 		}
@@ -206,7 +206,7 @@ 
 			$return = null;
 		}
 
-		if ($return === null && isset($this->normalization[$this->scheme][$name])) {
+		if($return === null && isset($this->normalization[$this->scheme][$name])) {
 			return $this->normalization[$this->scheme][$name];
 		}
 		else {
@@ -230,7 +230,7 @@ 
 	 * @param string $name Property name
 	 */
 	public function __unset($name) {
-		if (method_exists($this, 'set_' . $name)) {
+		if(method_exists($this, 'set_' . $name)) {
 			call_user_func(array($this, 'set_' . $name), '');
 		}
 	}
@@ -254,25 +254,25 @@ 
 	 * @return IRI|false
 	 */
 	public static function absolutize($base, $relative) {
-		if (!($relative instanceof Requests_IRI)) {
+		if(!($relative instanceof Requests_IRI)) {
 			$relative = new Requests_IRI($relative);
 		}
-		if (!$relative->is_valid()) {
+		if(!$relative->is_valid()) {
 			return false;
 		}
-		elseif ($relative->scheme !== null) {
+		elseif($relative->scheme !== null) {
 			return clone $relative;
 		}
 
-		if (!($base instanceof Requests_IRI)) {
+		if(!($base instanceof Requests_IRI)) {
 			$base = new Requests_IRI($base);
 		}
-		if ($base->scheme === null || !$base->is_valid()) {
+		if($base->scheme === null || !$base->is_valid()) {
 			return false;
 		}
 
-		if ($relative->get_iri() !== '') {
-			if ($relative->iuserinfo !== null || $relative->ihost !== null || $relative->port !== null) {
+		if($relative->get_iri() !== '') {
+			if($relative->iuserinfo !== null || $relative->ihost !== null || $relative->port !== null) {
 				$target = clone $relative;
 				$target->scheme = $base->scheme;
 			}
@@ -282,14 +282,14 @@ 
 				$target->iuserinfo = $base->iuserinfo;
 				$target->ihost = $base->ihost;
 				$target->port = $base->port;
-				if ($relative->ipath !== '') {
-					if ($relative->ipath[0] === '/') {
+				if($relative->ipath !== '') {
+					if($relative->ipath[0] === '/') {
 						$target->ipath = $relative->ipath;
 					}
-					elseif (($base->iuserinfo !== null || $base->ihost !== null || $base->port !== null) && $base->ipath === '') {
+					elseif(($base->iuserinfo !== null || $base->ihost !== null || $base->port !== null) && $base->ipath === '') {
 						$target->ipath = '/' . $relative->ipath;
 					}
-					elseif (($last_segment = strrpos($base->ipath, '/')) !== false) {
+					elseif(($last_segment = strrpos($base->ipath, '/')) !== false) {
 						$target->ipath = substr($base->ipath, 0, $last_segment + 1) . $relative->ipath;
 					}
 					else {
@@ -300,10 +300,10 @@ 
 				}
 				else {
 					$target->ipath = $base->ipath;
-					if ($relative->iquery !== null) {
+					if($relative->iquery !== null) {
 						$target->iquery = $relative->iquery;
 					}
-					elseif ($base->iquery !== null) {
+					elseif($base->iquery !== null) {
 						$target->iquery = $base->iquery;
 					}
 				}
@@ -327,23 +327,23 @@ 
 	protected function parse_iri($iri) {
 		$iri = trim($iri, "\x20\x09\x0A\x0C\x0D");
 		$has_match = preg_match('/^((?P<scheme>[^:\/?#]+):)?(\/\/(?P<authority>[^\/?#]*))?(?P<path>[^?#]*)(\?(?P<query>[^#]*))?(#(?P<fragment>.*))?$/', $iri, $match);
-		if (!$has_match) {
+		if(!$has_match) {
 			throw new Requests_Exception('Cannot parse supplied IRI', 'iri.cannot_parse', $iri);
 		}
 
-		if ($match[1] === '') {
+		if($match[1] === '') {
 			$match['scheme'] = null;
 		}
-		if (!isset($match[3]) || $match[3] === '') {
+		if(!isset($match[3]) || $match[3] === '') {
 			$match['authority'] = null;
 		}
-		if (!isset($match[5])) {
+		if(!isset($match[5])) {
 			$match['path'] = '';
 		}
-		if (!isset($match[6]) || $match[6] === '') {
+		if(!isset($match[6]) || $match[6] === '') {
 			$match['query'] = null;
 		}
-		if (!isset($match[8]) || $match[8] === '') {
+		if(!isset($match[8]) || $match[8] === '') {
 			$match['fragment'] = null;
 		}
 		return $match;
@@ -357,46 +357,46 @@ 
 	 */
 	protected function remove_dot_segments($input) {
 		$output = '';
-		while (strpos($input, './') !== false || strpos($input, '/.') !== false || $input === '.' || $input === '..') {
+		while(strpos($input, './') !== false || strpos($input, '/.') !== false || $input === '.' || $input === '..') {
 			// A: If the input buffer begins with a prefix of "../" or "./",
 			// then remove that prefix from the input buffer; otherwise,
-			if (strpos($input, '../') === 0) {
+			if(strpos($input, '../') === 0) {
 				$input = substr($input, 3);
 			}
-			elseif (strpos($input, './') === 0) {
+			elseif(strpos($input, './') === 0) {
 				$input = substr($input, 2);
 			}
 			// B: if the input buffer begins with a prefix of "/./" or "/.",
 			// where "." is a complete path segment, then replace that prefix
 			// with "/" in the input buffer; otherwise,
-			elseif (strpos($input, '/./') === 0) {
+			elseif(strpos($input, '/./') === 0) {
 				$input = substr($input, 2);
 			}
-			elseif ($input === '/.') {
+			elseif($input === '/.') {
 				$input = '/';
 			}
 			// C: if the input buffer begins with a prefix of "/../" or "/..",
 			// where ".." is a complete path segment, then replace that prefix
 			// with "/" in the input buffer and remove the last segment and its
 			// preceding "/" (if any) from the output buffer; otherwise,
-			elseif (strpos($input, '/../') === 0) {
+			elseif(strpos($input, '/../') === 0) {
 				$input = substr($input, 3);
 				$output = substr_replace($output, '', strrpos($output, '/'));
 			}
-			elseif ($input === '/..') {
+			elseif($input === '/..') {
 				$input = '/';
 				$output = substr_replace($output, '', strrpos($output, '/'));
 			}
 			// D: if the input buffer consists only of "." or "..", then remove
 			// that from the input buffer; otherwise,
-			elseif ($input === '.' || $input === '..') {
+			elseif($input === '.' || $input === '..') {
 				$input = '';
 			}
 			// E: move the first path segment in the input buffer to the end of
 			// the output buffer, including the initial "/" character (if any)
 			// and any subsequent characters up to, but not including, the next
 			// "/" character or the end of the input buffer
-			elseif (($pos = strpos($input, '/', 1)) !== false) {
+			elseif(($pos = strpos($input, '/', 1)) !== false) {
 				$output .= substr($input, 0, $pos);
 				$input = substr_replace($input, '', 0, $pos);
 			}
@@ -431,7 +431,7 @@ 
 		// Now replace any bytes that aren't allowed with their pct-encoded versions
 		$position = 0;
 		$strlen = strlen($string);
-		while (($position += strspn($string, $extra_chars, $position)) < $strlen) {
+		while(($position += strspn($string, $extra_chars, $position)) < $strlen) {
 			$value = ord($string[$position]);
 
 			// Start position
@@ -442,20 +442,20 @@ 
 
 			// No one byte sequences are valid due to the while.
 			// Two byte sequence:
-			if (($value & 0xE0) === 0xC0) {
-				$character = ($value & 0x1F) << 6;
+			if(($value&0xE0) === 0xC0) {
+				$character = ($value&0x1F) << 6;
 				$length = 2;
 				$remaining = 1;
 			}
 			// Three byte sequence:
-			elseif (($value & 0xF0) === 0xE0) {
-				$character = ($value & 0x0F) << 12;
+			elseif(($value&0xF0) === 0xE0) {
+				$character = ($value&0x0F) << 12;
 				$length = 3;
 				$remaining = 2;
 			}
 			// Four byte sequence:
-			elseif (($value & 0xF8) === 0xF0) {
-				$character = ($value & 0x07) << 18;
+			elseif(($value&0xF8) === 0xF0) {
+				$character = ($value&0x07) << 18;
 				$length = 4;
 				$remaining = 3;
 			}
@@ -466,14 +466,14 @@ 
 				$remaining = 0;
 			}
 
-			if ($remaining) {
-				if ($position + $length <= $strlen) {
-					for ($position++; $remaining; $position++) {
+			if($remaining) {
+				if($position + $length <= $strlen) {
+					for($position++; $remaining; $position++) {
 						$value = ord($string[$position]);
 
 						// Check that the byte is valid, then add it to the character:
-						if (($value & 0xC0) === 0x80) {
-							$character |= ($value & 0x3F) << (--$remaining * 6);
+						if(($value&0xC0) === 0x80) {
+							$character |= ($value&0x3F) << (--$remaining * 6);
 						}
 						// If it is invalid, count the sequence as invalid and reprocess the current byte:
 						else {
@@ -490,7 +490,7 @@ 
 			}
 
 			// Percent encode anything invalid or not in ucschar
-			if (
+			if(
 				// Invalid sequences
 				!$valid
 				// Non-shortest form sequences are invalid
@@ -499,7 +499,7 @@ 
 				|| $length > 3 && $character <= 0xFFFF
 				// Outside of range of ucschar codepoints
 				// Noncharacters
-				|| ($character & 0xFFFE) === 0xFFFE
+				|| ($character&0xFFFE) === 0xFFFE
 				|| $character >= 0xFDD0 && $character <= 0xFDEF
 				|| (
 					// Everything else not in ucschar
@@ -515,11 +515,11 @@ 
 				)
 			) {
 				// If we were a character, pretend we weren't, but rather an error.
-				if ($valid) {
+				if($valid) {
 					$position--;
 				}
 
-				for ($j = $start; $j <= $position; $j++) {
+				for($j = $start; $j <= $position; $j++) {
 					$string = substr_replace($string, sprintf('%%%02X', ord($string[$j])), $j, 1);
 					$j += 2;
 					$position += 2;
@@ -552,11 +552,11 @@ 
 		$remaining = 0;
 
 		// Loop over each and every byte, and set $value to its value
-		for ($i = 1, $len = count($bytes); $i < $len; $i++) {
+		for($i = 1, $len = count($bytes); $i < $len; $i++) {
 			$value = hexdec($bytes[$i]);
 
 			// If we're the first byte of sequence:
-			if (!$remaining) {
+			if(!$remaining) {
 				// Start position
 				$start = $i;
 
@@ -564,25 +564,25 @@ 
 				$valid = true;
 
 				// One byte sequence:
-				if ($value <= 0x7F) {
+				if($value <= 0x7F) {
 					$character = $value;
 					$length = 1;
 				}
 				// Two byte sequence:
-				elseif (($value & 0xE0) === 0xC0) {
-					$character = ($value & 0x1F) << 6;
+				elseif(($value&0xE0) === 0xC0) {
+					$character = ($value&0x1F) << 6;
 					$length = 2;
 					$remaining = 1;
 				}
 				// Three byte sequence:
-				elseif (($value & 0xF0) === 0xE0) {
-					$character = ($value & 0x0F) << 12;
+				elseif(($value&0xF0) === 0xE0) {
+					$character = ($value&0x0F) << 12;
 					$length = 3;
 					$remaining = 2;
 				}
 				// Four byte sequence:
-				elseif (($value & 0xF8) === 0xF0) {
-					$character = ($value & 0x07) << 18;
+				elseif(($value&0xF8) === 0xF0) {
+					$character = ($value&0x07) << 18;
 					$length = 4;
 					$remaining = 3;
 				}
@@ -595,9 +595,9 @@ 
 			// Continuation byte:
 			else {
 				// Check that the byte is valid, then add it to the character:
-				if (($value & 0xC0) === 0x80) {
+				if(($value&0xC0) === 0x80) {
 					$remaining--;
-					$character |= ($value & 0x3F) << ($remaining * 6);
+					$character |= ($value&0x3F) << ($remaining * 6);
 				}
 				// If it is invalid, count the sequence as invalid and reprocess the current byte as the start of a sequence:
 				else {
@@ -608,9 +608,9 @@ 
 			}
 
 			// If we've reached the end of the current byte sequence, append it to Unicode::$data
-			if (!$remaining) {
+			if(!$remaining) {
 				// Percent encode anything invalid or not in iunreserved
-				if (
+				if(
 					// Invalid sequences
 					!$valid
 					// Non-shortest form sequences are invalid
@@ -621,7 +621,7 @@ 
 					|| $character < 0x2D
 					|| $character > 0xEFFFD
 					// Noncharacters
-					|| ($character & 0xFFFE) === 0xFFFE
+					|| ($character&0xFFFE) === 0xFFFE
 					|| $character >= 0xFDD0 && $character <= 0xFDEF
 					// Everything else not in iunreserved (this is all BMP)
 					|| $character === 0x2F
@@ -631,12 +631,12 @@ 
 					|| $character > 0x7E && $character < 0xA0
 					|| $character > 0xD7FF && $character < 0xF900
 				) {
-					for ($j = $start; $j <= $i; $j++) {
+					for($j = $start; $j <= $i; $j++) {
 						$string .= '%' . strtoupper($bytes[$j]);
 					}
 				}
 				else {
-					for ($j = $start; $j <= $i; $j++) {
+					for($j = $start; $j <= $i; $j++) {
 						$string .= chr(hexdec($bytes[$j]));
 					}
 				}
@@ -645,8 +645,8 @@ 
 
 		// If we have any bytes left over they are invalid (i.e., we are
 		// mid-way through a multi-byte sequence)
-		if ($remaining) {
-			for ($j = $start; $j < $len; $j++) {
+		if($remaining) {
+			for($j = $start; $j < $len; $j++) {
 				$string .= '%' . strtoupper($bytes[$j]);
 			}
 		}
@@ -655,25 +655,25 @@ 
 	}
 
 	protected function scheme_normalization() {
-		if (isset($this->normalization[$this->scheme]['iuserinfo']) && $this->iuserinfo === $this->normalization[$this->scheme]['iuserinfo']) {
+		if(isset($this->normalization[$this->scheme]['iuserinfo']) && $this->iuserinfo === $this->normalization[$this->scheme]['iuserinfo']) {
 			$this->iuserinfo = null;
 		}
-		if (isset($this->normalization[$this->scheme]['ihost']) && $this->ihost === $this->normalization[$this->scheme]['ihost']) {
+		if(isset($this->normalization[$this->scheme]['ihost']) && $this->ihost === $this->normalization[$this->scheme]['ihost']) {
 			$this->ihost = null;
 		}
-		if (isset($this->normalization[$this->scheme]['port']) && $this->port === $this->normalization[$this->scheme]['port']) {
+		if(isset($this->normalization[$this->scheme]['port']) && $this->port === $this->normalization[$this->scheme]['port']) {
 			$this->port = null;
 		}
-		if (isset($this->normalization[$this->scheme]['ipath']) && $this->ipath === $this->normalization[$this->scheme]['ipath']) {
+		if(isset($this->normalization[$this->scheme]['ipath']) && $this->ipath === $this->normalization[$this->scheme]['ipath']) {
 			$this->ipath = '';
 		}
-		if (isset($this->ihost) && empty($this->ipath)) {
+		if(isset($this->ihost) && empty($this->ipath)) {
 			$this->ipath = '/';
 		}
-		if (isset($this->normalization[$this->scheme]['iquery']) && $this->iquery === $this->normalization[$this->scheme]['iquery']) {
+		if(isset($this->normalization[$this->scheme]['iquery']) && $this->iquery === $this->normalization[$this->scheme]['iquery']) {
 			$this->iquery = null;
 		}
-		if (isset($this->normalization[$this->scheme]['ifragment']) && $this->ifragment === $this->normalization[$this->scheme]['ifragment']) {
+		if(isset($this->normalization[$this->scheme]['ifragment']) && $this->ifragment === $this->normalization[$this->scheme]['ifragment']) {
 			$this->ifragment = null;
 		}
 	}
@@ -686,14 +686,14 @@ 
 	 */
 	public function is_valid() {
 		$isauthority = $this->iuserinfo !== null || $this->ihost !== null || $this->port !== null;
-		if ($this->ipath !== '' &&
+		if($this->ipath !== '' &&
 			(
 				$isauthority && $this->ipath[0] !== '/' ||
 				(
 					$this->scheme === null &&
 					!$isauthority &&
 					strpos($this->ipath, ':') !== false &&
-					(strpos($this->ipath, '/') === false ? true : strpos($this->ipath, ':') < strpos($this->ipath, '/'))
+					(strpos($this->ipath, '/') === false?true : strpos($this->ipath, ':') < strpos($this->ipath, '/'))
 				)
 			)
 		) {
@@ -712,14 +712,14 @@ 
 	 */
 	protected function set_iri($iri) {
 		static $cache;
-		if (!$cache) {
+		if(!$cache) {
 			$cache = array();
 		}
 
-		if ($iri === null) {
+		if($iri === null) {
 			return true;
 		}
-		if (isset($cache[$iri])) {
+		if(isset($cache[$iri])) {
 			list($this->scheme,
 				 $this->iuserinfo,
 				 $this->ihost,
@@ -731,7 +731,7 @@ 
 			return $return;
 		}
 
-		$parsed = $this->parse_iri((string) $iri);
+		$parsed = $this->parse_iri((string)$iri);
 
 		$return = $this->set_scheme($parsed['scheme'])
 			&& $this->set_authority($parsed['authority'])
@@ -758,10 +758,10 @@ 
 	 * @return bool
 	 */
 	protected function set_scheme($scheme) {
-		if ($scheme === null) {
+		if($scheme === null) {
 			$this->scheme = null;
 		}
-		elseif (!preg_match('/^[A-Za-z][0-9A-Za-z+\-.]*$/', $scheme)) {
+		elseif(!preg_match('/^[A-Za-z][0-9A-Za-z+\-.]*$/', $scheme)) {
 			$this->scheme = null;
 			return false;
 		}
@@ -780,17 +780,17 @@ 
 	 */
 	protected function set_authority($authority) {
 		static $cache;
-		if (!$cache) {
+		if(!$cache) {
 			$cache = array();
 		}
 
-		if ($authority === null) {
+		if($authority === null) {
 			$this->iuserinfo = null;
 			$this->ihost = null;
 			$this->port = null;
 			return true;
 		}
-		if (isset($cache[$authority])) {
+		if(isset($cache[$authority])) {
 			list($this->iuserinfo,
 				 $this->ihost,
 				 $this->port,
@@ -800,16 +800,16 @@ 
 		}
 
 		$remaining = $authority;
-		if (($iuserinfo_end = strrpos($remaining, '@')) !== false) {
+		if(($iuserinfo_end = strrpos($remaining, '@')) !== false) {
 			$iuserinfo = substr($remaining, 0, $iuserinfo_end);
 			$remaining = substr($remaining, $iuserinfo_end + 1);
 		}
 		else {
 			$iuserinfo = null;
 		}
-		if (($port_start = strpos($remaining, ':', strpos($remaining, ']'))) !== false) {
+		if(($port_start = strpos($remaining, ':', strpos($remaining, ']'))) !== false) {
 			$port = substr($remaining, $port_start + 1);
-			if ($port === false || $port === '') {
+			if($port === false || $port === '') {
 				$port = null;
 			}
 			$remaining = substr($remaining, 0, $port_start);
@@ -837,7 +837,7 @@ 
 	 * @return bool
 	 */
 	protected function set_userinfo($iuserinfo) {
-		if ($iuserinfo === null) {
+		if($iuserinfo === null) {
 			$this->iuserinfo = null;
 		}
 		else {
@@ -856,12 +856,12 @@ 
 	 * @return bool
 	 */
 	protected function set_host($ihost) {
-		if ($ihost === null) {
+		if($ihost === null) {
 			$this->ihost = null;
 			return true;
 		}
-		if (substr($ihost, 0, 1) === '[' && substr($ihost, -1) === ']') {
-			if (Requests_IPv6::check_ipv6(substr($ihost, 1, -1))) {
+		if(substr($ihost, 0, 1) === '[' && substr($ihost, -1) === ']') {
+			if(Requests_IPv6::check_ipv6(substr($ihost, 1, -1))) {
 				$this->ihost = '[' . Requests_IPv6::compress(substr($ihost, 1, -1)) . ']';
 			}
 			else {
@@ -877,8 +877,8 @@ 
 			// as that can add unescaped characters.
 			$position = 0;
 			$strlen = strlen($ihost);
-			while (($position += strcspn($ihost, 'ABCDEFGHIJKLMNOPQRSTUVWXYZ%', $position)) < $strlen) {
-				if ($ihost[$position] === '%') {
+			while(($position += strcspn($ihost, 'ABCDEFGHIJKLMNOPQRSTUVWXYZ%', $position)) < $strlen) {
+				if($ihost[$position] === '%') {
 					$position += 3;
 				}
 				else {
@@ -903,13 +903,13 @@ 
 	 * @return bool
 	 */
 	protected function set_port($port) {
-		if ($port === null) {
+		if($port === null) {
 			$this->port = null;
 			return true;
 		}
 
-		if (strspn($port, '0123456789') === strlen($port)) {
-			$this->port = (int) $port;
+		if(strspn($port, '0123456789') === strlen($port)) {
+			$this->port = (int)$port;
 			$this->scheme_normalization();
 			return true;
 		}
@@ -926,21 +926,21 @@ 
 	 */
 	protected function set_path($ipath) {
 		static $cache;
-		if (!$cache) {
+		if(!$cache) {
 			$cache = array();
 		}
 
-		$ipath = (string) $ipath;
+		$ipath = (string)$ipath;
 
-		if (isset($cache[$ipath])) {
-			$this->ipath = $cache[$ipath][(int) ($this->scheme !== null)];
+		if(isset($cache[$ipath])) {
+			$this->ipath = $cache[$ipath][(int)($this->scheme !== null)];
 		}
 		else {
 			$valid = $this->replace_invalid_with_pct_encoding($ipath, '!$&\'()*+,;=@:/');
 			$removed = $this->remove_dot_segments($valid);
 
 			$cache[$ipath] = array($valid, $removed);
-			$this->ipath = ($this->scheme !== null) ? $removed : $valid;
+			$this->ipath = ($this->scheme !== null)?$removed : $valid;
 		}
 		$this->scheme_normalization();
 		return true;
@@ -953,7 +953,7 @@ 
 	 * @return bool
 	 */
 	protected function set_query($iquery) {
-		if ($iquery === null) {
+		if($iquery === null) {
 			$this->iquery = null;
 		}
 		else {
@@ -970,7 +970,7 @@ 
 	 * @return bool
 	 */
 	protected function set_fragment($ifragment) {
-		if ($ifragment === null) {
+		if($ifragment === null) {
 			$this->ifragment = null;
 		}
 		else {
@@ -987,18 +987,18 @@ 
 	 * @return string|false URI if IRI is valid, false otherwise.
 	 */
 	protected function to_uri($string) {
-		if (!is_string($string)) {
+		if(!is_string($string)) {
 			return false;
 		}
 
 		static $non_ascii;
-		if (!$non_ascii) {
+		if(!$non_ascii) {
 			$non_ascii = implode('', range("\x80", "\xFF"));
 		}
 
 		$position = 0;
 		$strlen = strlen($string);
-		while (($position += strcspn($string, $non_ascii, $position)) < $strlen) {
+		while(($position += strcspn($string, $non_ascii, $position)) < $strlen) {
 			$string = substr_replace($string, sprintf('%%%02X', ord($string[$position])), $position, 1);
 			$position += 3;
 			$strlen += 2;
@@ -1013,22 +1013,22 @@ 
 	 * @return string
 	 */
 	protected function get_iri() {
-		if (!$this->is_valid()) {
+		if(!$this->is_valid()) {
 			return false;
 		}
 
 		$iri = '';
-		if ($this->scheme !== null) {
+		if($this->scheme !== null) {
 			$iri .= $this->scheme . ':';
 		}
-		if (($iauthority = $this->get_iauthority()) !== null) {
+		if(($iauthority = $this->get_iauthority()) !== null) {
 			$iri .= '//' . $iauthority;
 		}
 		$iri .= $this->ipath;
-		if ($this->iquery !== null) {
+		if($this->iquery !== null) {
 			$iri .= '?' . $this->iquery;
 		}
-		if ($this->ifragment !== null) {
+		if($this->ifragment !== null) {
 			$iri .= '#' . $this->ifragment;
 		}
 
@@ -1050,18 +1050,18 @@ 
 	 * @return string
 	 */
 	protected function get_iauthority() {
-		if ($this->iuserinfo === null && $this->ihost === null && $this->port === null) {
+		if($this->iuserinfo === null && $this->ihost === null && $this->port === null) {
 			return null;
 		}
 
 		$iauthority = '';
-		if ($this->iuserinfo !== null) {
+		if($this->iuserinfo !== null) {
 			$iauthority .= $this->iuserinfo . '@';
 		}
-		if ($this->ihost !== null) {
+		if($this->ihost !== null) {
 			$iauthority .= $this->ihost;
 		}
-		if ($this->port !== null) {
+		if($this->port !== null) {
 			$iauthority .= ':' . $this->port;
 		}
 		return $iauthority;
@@ -1074,7 +1074,7 @@ 
 	 */
 	protected function get_authority() {
 		$iauthority = $this->get_iauthority();
-		if (is_string($iauthority)) {
+		if(is_string($iauthority)) {
 			return $this->to_uri($iauthority);
 		}
 		else {

php/Requests/libary/Requests/Session.php

Doc Comments +1 added lines, -1 removed lines

@@ -232,7 +232,7 @@
 	 *
 	 * @param array $request Request data (same form as {@see request_multiple})
 	 * @param boolean $merge_options Should we merge options as well?
-	 * @return array Request data
+	 * @return string Request data
 	 */
 	protected function merge_request($request, $merge_options = true) {
 		if ($this->url !== null) {

Spacing +10 added lines, -10 removed lines

@@ -69,7 +69,7 @@ 
 		$this->data = $data;
 		$this->options = $options;
 
-		if (empty($this->options['cookies'])) {
+		if(empty($this->options['cookies'])) {
 			$this->options['cookies'] = new Requests_Cookie_Jar();
 		}
 	}
@@ -81,7 +81,7 @@ 
 	 * @return mixed|null Property value, null if none found
 	 */
 	public function __get($key) {
-		if (isset($this->options[$key])) {
+		if(isset($this->options[$key])) {
 			return $this->options[$key];
 		}
 
@@ -113,7 +113,7 @@ 
 	 * @param string $key Property key
 	 */
 	public function __unset($key) {
-		if (isset($this->options[$key])) {
+		if(isset($this->options[$key])) {
 			unset($this->options[$key]);
 		}
 	}
@@ -215,7 +215,7 @@ 
 	 * @return array Responses (either Requests_Response or a Requests_Exception object)
 	 */
 	public function request_multiple($requests, $options = array()) {
-		foreach ($requests as $key => $request) {
+		foreach($requests as $key => $request) {
 			$requests[$key] = $this->merge_request($request, false);
 		}
 
@@ -235,26 +235,26 @@ 
 	 * @return array Request data
 	 */
 	protected function merge_request($request, $merge_options = true) {
-		if ($this->url !== null) {
+		if($this->url !== null) {
 			$request['url'] = Requests_IRI::absolutize($this->url, $request['url']);
 			$request['url'] = $request['url']->uri;
 		}
 
-		if (empty($request['headers'])) {
+		if(empty($request['headers'])) {
 			$request['headers'] = array();
 		}
 		$request['headers'] = array_merge($this->headers, $request['headers']);
 
-		if (empty($request['data'])) {
-			if (is_array($this->data)) {
+		if(empty($request['data'])) {
+			if(is_array($this->data)) {
 				$request['data'] = $this->data;
 			}
 		}
-		elseif (is_array($request['data']) && is_array($this->data)) {
+		elseif(is_array($request['data']) && is_array($this->data)) {
 			$request['data'] = array_merge($this->data, $request['data']);
 		}
 
-		if ($merge_options !== false) {
+		if($merge_options !== false) {
 			$request['options'] = array_merge($this->options, $request['options']);
 
 			// Disallow forcing the type, as that's a per request setting

php/Requests/libary/Requests/Auth.php

Spacing +1 added lines, -1 removed lines

@@ -29,5 +29,5 @@
 	 * @see Requests_Hooks::register
 	 * @param Requests_Hooks $hooks Hook system
 	 */
-	public function register(Requests_Hooks &$hooks);
+	public function register(Requests_Hooks&$hooks);
 }
\ No newline at end of file

Doc Comments +1 added lines

@@ -28,6 +28,7 @@
 	 *
 	 * @see Requests_Hooks::register
 	 * @param Requests_Hooks $hooks Hook system
+	 * @return void
 	 */
 	public function register(Requests_Hooks &$hooks);
 }
\ No newline at end of file

php/Location.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-class Location{
+class Location {
 
     public $cityName;
 
@@ -37,7 +37,7 @@ 
     {
         $this->lng = $lng;
     }
-    public function toArray(){
+    public function toArray() {
         return array(
             "city" => $this->getCityName(),
             "country" => 'DE',

php/Requests/libary/Requests/Cookie.php

Doc Comments +2 added lines, -1 removed lines

@@ -62,7 +62,7 @@ 
 	 *
 	 * @param string $name
 	 * @param string $value
-	 * @param array|Requests_Utility_CaseInsensitiveDictionary $attributes Associative array of attribute data
+	 * @param Requests_Utility_CaseInsensitiveDictionary $attributes Associative array of attribute data
 	 */
 	public function __construct($name, $value, $attributes = array(), $flags = array(), $reference_time = null) {
 		$this->name = $name;
@@ -377,6 +377,7 @@ 
 	 * specifies some of this handling, but not in a thorough manner.
 	 *
 	 * @param string Cookie header value (from a Set-Cookie header)
+	 * @param integer $reference_time
 	 * @return Requests_Cookie Parsed cookie object
 	 */
 	public static function parse($string, $name = '', $reference_time = null) {

Spacing +43 added lines, -43 removed lines

@@ -77,7 +77,7 @@ 
 		$this->flags = array_merge($default_flags, $flags);
 
 		$this->reference_time = time();
-		if ($reference_time !== null) {
+		if($reference_time !== null) {
 			$this->reference_time = $reference_time;
 		}
 
@@ -97,12 +97,12 @@ 
 		// If a cookie has both the Max-Age and the Expires attribute, the Max-
 		// Age attribute has precedence and controls the expiration date of the
 		// cookie.
-		if (isset($this->attributes['max-age'])) {
+		if(isset($this->attributes['max-age'])) {
 			$max_age = $this->attributes['max-age'];
 			return $max_age < $this->reference_time;
 		}
 
-		if (isset($this->attributes['expires'])) {
+		if(isset($this->attributes['expires'])) {
 			$expires = $this->attributes['expires'];
 			return $expires < $this->reference_time;
 		}
@@ -117,11 +117,11 @@ 
 	 * @return boolean Whether the cookie is valid for the given URI
 	 */
 	public function uri_matches(Requests_IRI $uri) {
-		if (!$this->domain_matches($uri->host)) {
+		if(!$this->domain_matches($uri->host)) {
 			return false;
 		}
 
-		if (!$this->path_matches($uri->path)) {
+		if(!$this->path_matches($uri->path)) {
 			return false;
 		}
 
@@ -135,37 +135,37 @@ 
 	 * @return boolean Whether the cookie is valid for the given domain
 	 */
 	public function domain_matches($string) {
-		if (!isset($this->attributes['domain'])) {
+		if(!isset($this->attributes['domain'])) {
 			// Cookies created manually; cookies created by Requests will set
 			// the domain to the requested domain
 			return true;
 		}
 
 		$domain_string = $this->attributes['domain'];
-		if ($domain_string === $string) {
+		if($domain_string === $string) {
 			// The domain string and the string are identical.
 			return true;
 		}
 
 		// If the cookie is marked as host-only and we don't have an exact
 		// match, reject the cookie
-		if ($this->flags['host-only'] === true) {
+		if($this->flags['host-only'] === true) {
 			return false;
 		}
 
-		if (strlen($string) <= strlen($domain_string)) {
+		if(strlen($string) <= strlen($domain_string)) {
 			// For obvious reasons, the string cannot be a suffix if the domain
 			// is shorter than the domain string
 			return false;
 		}
 
-		if (substr($string, -1 * strlen($domain_string)) !== $domain_string) {
+		if(substr($string, -1 * strlen($domain_string)) !== $domain_string) {
 			// The domain string should be a suffix of the string.
 			return false;
 		}
 
 		$prefix = substr($string, 0, strlen($string) - strlen($domain_string));
-		if (substr($prefix, -1) !== '.') {
+		if(substr($prefix, -1) !== '.') {
 			// The last character of the string that is not included in the
 			// domain string should be a %x2E (".") character.
 			return false;
@@ -184,12 +184,12 @@ 
 	 * @return boolean Whether the cookie is valid for the given path
 	 */
 	public function path_matches($request_path) {
-		if (empty($request_path)) {
+		if(empty($request_path)) {
 			// Normalize empty path to root
 			$request_path = '/';
 		}
 
-		if (!isset($this->attributes['path'])) {
+		if(!isset($this->attributes['path'])) {
 			// Cookies created manually; cookies created by Requests will set
 			// the path to the requested path
 			return true;
@@ -197,19 +197,19 @@ 
 
 		$cookie_path = $this->attributes['path'];
 
-		if ($cookie_path === $request_path) {
+		if($cookie_path === $request_path) {
 			// The cookie-path and the request-path are identical.
 			return true;
 		}
 
-		if (strlen($request_path) > strlen($cookie_path) && substr($request_path, 0, strlen($cookie_path)) === $cookie_path) {
-			if (substr($cookie_path, -1) === '/') {
+		if(strlen($request_path) > strlen($cookie_path) && substr($request_path, 0, strlen($cookie_path)) === $cookie_path) {
+			if(substr($cookie_path, -1) === '/') {
 				// The cookie-path is a prefix of the request-path, and the last
 				// character of the cookie-path is %x2F ("/").
 				return true;
 			}
 
-			if (substr($request_path, strlen($cookie_path), 1) === '/') {
+			if(substr($request_path, strlen($cookie_path), 1) === '/') {
 				// The cookie-path is a prefix of the request-path, and the
 				// first character of the request-path that is not included in
 				// the cookie-path is a %x2F ("/") character.
@@ -226,15 +226,15 @@ 
 	 * @return boolean Whether the cookie was successfully normalized
 	 */
 	public function normalize() {
-		foreach ($this->attributes as $key => $value) {
+		foreach($this->attributes as $key => $value) {
 			$orig_value = $value;
 			$value = $this->normalize_attribute($key, $value);
-			if ($value === null) {
+			if($value === null) {
 				unset($this->attributes[$key]);
 				continue;
 			}
 
-			if ($value !== $orig_value) {
+			if($value !== $orig_value) {
 				$this->attributes[$key] = $value;
 			}
 		}
@@ -252,15 +252,15 @@ 
 	 * @return mixed Value if available, or null if the attribute value is invalid (and should be skipped)
 	 */
 	protected function normalize_attribute($name, $value) {
-		switch (strtolower($name)) {
+		switch(strtolower($name)) {
 			case 'expires':
 				// Expiration parsing, as per RFC 6265 section 5.2.1
-				if (is_int($value)) {
+				if(is_int($value)) {
 					return $value;
 				}
 
 				$expiry_time = strtotime($value);
-				if ($expiry_time === false) {
+				if($expiry_time === false) {
 					return null;
 				}
 
@@ -268,17 +268,17 @@ 
 
 			case 'max-age':
 				// Expiration parsing, as per RFC 6265 section 5.2.2
-				if (is_int($value)) {
+				if(is_int($value)) {
 					return $value;
 				}
 
 				// Check that we have a valid age
-				if (!preg_match('/^-?\d+$/', $value)) {
+				if(!preg_match('/^-?\d+$/', $value)) {
 					return null;
 				}
 
-				$delta_seconds = (int) $value;
-				if ($delta_seconds <= 0) {
+				$delta_seconds = (int)$value;
+				if($delta_seconds <= 0) {
 					$expiry_time = 0;
 				}
 				else {
@@ -289,7 +289,7 @@ 
 
 			case 'domain':
 				// Domain normalization, as per RFC 6265 section 5.2.3
-				if ($value[0] === '.') {
+				if($value[0] === '.') {
 					$value = substr($value, 1);
 				}
 
@@ -332,11 +332,11 @@ 
 	 */
 	public function format_for_set_cookie() {
 		$header_value = $this->format_for_header();
-		if (!empty($this->attributes)) {
+		if(!empty($this->attributes)) {
 			$parts = array();
-			foreach ($this->attributes as $key => $value) {
+			foreach($this->attributes as $key => $value) {
 				// Ignore non-associative attributes
-				if (is_numeric($key)) {
+				if(is_numeric($key)) {
 					$parts[] = $value;
 				}
 				else {
@@ -383,10 +383,10 @@ 
 		$parts = explode(';', $string);
 		$kvparts = array_shift($parts);
 
-		if (!empty($name)) {
+		if(!empty($name)) {
 			$value = $string;
 		}
-		elseif (strpos($kvparts, '=') === false) {
+		elseif(strpos($kvparts, '=') === false) {
 			// Some sites might only have a value without the equals separator.
 			// Deviate from RFC 6265 and pretend it was actually a blank name
 			// (`=foo`)
@@ -404,9 +404,9 @@ 
 		// Attribute key are handled case-insensitively
 		$attributes = new Requests_Utility_CaseInsensitiveDictionary();
 
-		if (!empty($parts)) {
-			foreach ($parts as $part) {
-				if (strpos($part, '=') === false) {
+		if(!empty($parts)) {
+			foreach($parts as $part) {
+				if(strpos($part, '=') === false) {
 					$part_key = $part;
 					$part_value = true;
 				}
@@ -433,16 +433,16 @@ 
 	 */
 	public static function parse_from_headers(Requests_Response_Headers $headers, Requests_IRI $origin = null, $time = null) {
 		$cookie_headers = $headers->getValues('Set-Cookie');
-		if (empty($cookie_headers)) {
+		if(empty($cookie_headers)) {
 			return array();
 		}
 
 		$cookies = array();
-		foreach ($cookie_headers as $header) {
+		foreach($cookie_headers as $header) {
 			$parsed = self::parse($header, '', $time);
 
 			// Default domain/path attributes
-			if (empty($parsed->attributes['domain']) && !empty($origin)) {
+			if(empty($parsed->attributes['domain']) && !empty($origin)) {
 				$parsed->attributes['domain'] = $origin->host;
 				$parsed->flags['host-only'] = true;
 			}
@@ -451,17 +451,17 @@ 
 			}
 
 			$path_is_valid = (!empty($parsed->attributes['path']) && $parsed->attributes['path'][0] === '/');
-			if (!$path_is_valid && !empty($origin)) {
+			if(!$path_is_valid && !empty($origin)) {
 				$path = $origin->path;
 
 				// Default path normalization as per RFC 6265 section 5.1.4
-				if (substr($path, 0, 1) !== '/') {
+				if(substr($path, 0, 1) !== '/') {
 					// If the uri-path is empty or if the first character of
 					// the uri-path is not a %x2F ("/") character, output
 					// %x2F ("/") and skip the remaining steps.
 					$path = '/';
 				}
-				elseif (substr_count($path, '/') === 1) {
+				elseif(substr_count($path, '/') === 1) {
 					// If the uri-path contains no more than one %x2F ("/")
 					// character, output %x2F ("/") and skip the remaining
 					// step.
@@ -477,7 +477,7 @@ 
 			}
 
 			// Reject invalid cookie domains
-			if (!empty($origin) && !$parsed->domain_matches($origin->host)) {
+			if(!empty($origin) && !$parsed->domain_matches($origin->host)) {
 				continue;
 			}
 

php/Requests/libary/Requests/Exception/Transport/cURL.php

Doc Comments +3 added lines

@@ -29,6 +29,9 @@
 	 */
 	protected $reason = 'Unknown';
 
+	/**
+	 * @param string $message
+	 */
 	public function __construct($message, $type, $data = null, $code = 0) {
 		if ($type !== null) {
 			$this->type = $type;

Spacing +3 added lines, -3 removed lines

@@ -30,15 +30,15 @@
 	protected $reason = 'Unknown';
 
 	public function __construct($message, $type, $data = null, $code = 0) {
-		if ($type !== null) {
+		if($type !== null) {
 			$this->type = $type;
 		}
 
-		if ($code !== null) {
+		if($code !== null) {
 			$this->code = $code;
 		}
 
-		if ($message !== null) {
+		if($message !== null) {
 			$this->reason = $message;
 		}
 

php/Requests/libary/Requests/Hooker.php

Doc Comments +1 added lines

@@ -19,6 +19,7 @@
 	 * @param string $hook Hook name
 	 * @param callback $callback Function/method to call on event
 	 * @param int $priority Priority number. <0 is executed earlier, >0 is executed later
+	 * @return void
 	 */
 	public function register($hook, $callback, $priority = 0);