joomla-framework / oauth1

src/Client.php

Doc Comments +4 added lines, -4 removed lines

@@ -58,7 +58,7 @@ 
 	/**
 	 * Constructor.
 	 *
-	 * @param   array                   $options      OAuth1 Client options array.
+	 * @param   \Joomla\Registry\Registry                   $options      OAuth1 Client options array.
 	 * @param   Http                    $client       The HTTP client object.
 	 * @param   Input                   $input        The input object
 	 * @param   AbstractWebApplication  $application  The application object
@@ -249,7 +249,7 @@ 
 	 * @param   mixed   $data        The POST request data.
 	 * @param   array   $headers     An array of name-value pairs to include in the header of the request
 	 *
-	 * @return  object  The Response object.
+	 * @return  \Joomla\Http\Response  The Response object.
 	 *
 	 * @since   1.0
 	 * @throws  \DomainException
@@ -318,7 +318,7 @@ 
 	 * Method to validate a response.
 	 *
 	 * @param   string    $url       The request URL.
-	 * @param   Response  $response  The response to validate.
+	 * @param   \Joomla\Http\Response  $response  The response to validate.
 	 *
 	 * @return  void
 	 *
@@ -565,7 +565,7 @@ 
 	 * Set an option for the OAuth1 Client instance.
 	 *
 	 * @param   string  $key    The name of the option to set
-	 * @param   mixed   $value  The option value to set
+	 * @param   string   $value  The option value to set
 	 *
 	 * @return  Client  This object for method chaining
 	 *

Indentation +5 added lines, -5 removed lines

@@ -1,10 +1,10 @@
 <?php
 /**
- * Part of the Joomla Framework OAuth1 Package
- *
- * @copyright  Copyright (C) 2005 - 2015 Open Source Matters, Inc. All rights reserved.
- * @license    GNU General Public License version 2 or later; see LICENSE
- */
+	 * Part of the Joomla Framework OAuth1 Package
+	 *
+	 * @copyright  Copyright (C) 2005 - 2015 Open Source Matters, Inc. All rights reserved.
+	 * @license    GNU General Public License version 2 or later; see LICENSE
+	 */
 
 namespace Joomla\Oauth1;
 

Braces +11 added lines, -22 removed lines

@@ -94,8 +94,7 @@ 
 			if ($response)
 			{
 				return $this->token;
-			}
-			else
+			} else
 			{
 				$this->token = null;
 			}
@@ -105,8 +104,7 @@ 
 		if (strcmp($this->version, '1.0a') === 0)
 		{
 			$verifier = $this->input->get('oauth_verifier');
-		}
-		else
+		} else
 		{
 			$verifier = $this->input->get('oauth_token');
 		}
@@ -164,8 +162,7 @@ 
 			$parameters = array(
 				'oauth_callback' => $this->getOption('callback')
 			);
-		}
-		else
+		} else
 		{
 			$parameters = array();
 		}
@@ -271,8 +268,7 @@ 
 		if (isset($headers['Content-Type']) && strpos($headers['Content-Type'], 'multipart/form-data') !== false || !is_array($data))
 		{
 			$oauth_headers = $parameters;
-		}
-		else
+		} else
 		{
 			// Use all parameters for the signature.
 			$oauth_headers = array_merge($parameters, $data);
@@ -345,8 +341,7 @@ 
 			if (!strcmp($header, 'OAuth '))
 			{
 				$header .= $key . '="' . $this->safeEncode($value) . '"';
-			}
-			else
+			} else
 			{
 				$header .= ', ' . $key . '="' . $value . '"';
 			}
@@ -376,14 +371,12 @@ 
 					if (strpos($url, '?') === false)
 					{
 						$url .= '?' . $key . '=' . $v;
-					}
-					else
+					} else
 					{
 						$url .= '&' . $key . '=' . $v;
 					}
 				}
-			}
-			else
+			} else
 			{
 				if (strpos($value, ' ') !== false)
 				{
@@ -393,8 +386,7 @@ 
 				if (strpos($url, '?') === false)
 				{
 					$url .= '?' . $key . '=' . $value;
-				}
-				else
+				} else
 				{
 					$url .= '&' . $key . '=' . $value;
 				}
@@ -457,8 +449,7 @@ 
 					$v = $this->safeEncode($v);
 					$kv[] = "{$key}={$v}";
 				}
-			}
-			else
+			} else
 			{
 				$value = $this->safeEncode($value);
 				$kv[] = "{$key}={$value}";
@@ -494,16 +485,14 @@ 
 		if (is_array($data))
 		{
 			return array_map(array($this, 'safeEncode'), $data);
-		}
-		elseif (is_scalar($data))
+		} elseif (is_scalar($data))
 		{
 			return str_ireplace(
 				array('+', '%7E'),
 				array(' ', '~'),
 				rawurlencode($data)
 				);
-		}
-		else
+		} else
 		{
 			return '';
 		}

Tests/stubs/ClientInspector.php

Doc Comments +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 	/**
 	 * Mimic verifing credentials.
 	 *
-	 * @return void
+	 * @return boolean
 	 *
 	 * @since 1.0
 	 */

Indentation +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php
 /**
- * @copyright  Copyright (C) 2005 - 2015 Open Source Matters, Inc. All rights reserved.
- * @license    GNU General Public License version 2 or later; see LICENSE
- */
+	 * @copyright  Copyright (C) 2005 - 2015 Open Source Matters, Inc. All rights reserved.
+	 * @license    GNU General Public License version 2 or later; see LICENSE
+	 */
 
 namespace Joomla\OAuth1\Tests;
 

Tests/ClientTest.php

Indentation +12 added lines, -12 removed lines

@@ -99,12 +99,12 @@ 
 	}
 
 	/**
-	* Provides test data.
-	*
-	* @return array
-	*
-	* @since 1.0
-	*/
+	 * Provides test data.
+	 *
+	 * @return array
+	 *
+	 * @since 1.0
+	 */
 	public function seedAuthenticate()
 	{
 		// Token, fail and oauth version.
@@ -258,12 +258,12 @@ 
 	}
 
 	/**
-	* Provides test data.
-	*
-	* @return array
-	*
-	* @since  1.0
-	*/
+	 * Provides test data.
+	 *
+	 * @return array
+	 *
+	 * @since  1.0
+	 */
 	public function seedOauthRequest()
 	{
 		// Method

Spacing +1 added lines, -1 removed lines

@@ -185,7 +185,7 @@
 			TestHelper::setValue($input, 'data', $data);
 
 			// Get mock session
-			$mockSession = $this->getMock('Joomla\\Session\\Session', array( '_start', 'get'));
+			$mockSession = $this->getMock('Joomla\\Session\\Session', array('_start', 'get'));
 
 			if ($fail)
 			{

Braces +3 added lines, -6 removed lines

@@ -136,8 +136,7 @@ 
 			$this->object->setToken($token);
 			$result = $this->object->authenticate();
 			$this->assertEquals($result, $token);
-		}
-		else
+		} else
 		{
 			$this->object->setOption('requestTokenURL', 'https://example.com/request_token');
 			$this->object->setOption('authoriseURL', 'https://example.com/authorize');
@@ -175,8 +174,7 @@ 
 			{
 				TestHelper::setValue($this->object, 'version', $version);
 				$data = array('oauth_verifier' => 'verifier', 'oauth_token' => 'token');
-			}
-			else
+			} else
 			{
 				TestHelper::setValue($this->object, 'version', $version);
 				$data = array('oauth_token' => 'token');
@@ -302,8 +300,7 @@ 
 				$this->object->oauthRequest('www.example.com', $method, array('oauth_token' => '1235'), $data, array('Content-Type' => 'multipart/form-data')),
 				$this->equalTo($returnData)
 				);
-		}
-		else
+		} else
 		{
 			$this->client->expects($this->at(0))
 				->method($method)