QuickPay / quickpay-php-client

QuickPay/API/Exception.php

Doc Comments +1 added lines

@@ -17,6 +17,7 @@
     * Redefine the exception so message isn't optional
     *
     * @access public
+    * @param string $message
     * @return void
     */
     public function __construct($message, $code = 0, Exception $previous = null)

QuickPay/API/Request.php

Doc Comments +2 added lines

@@ -27,6 +27,7 @@ 
      * Instantiates the object
      *
      * @access public
+     * @param Client $client
      * @return object
      */
     public function __construct( $client )
@@ -68,6 +69,7 @@ 
      * Performs an API POST request
      *
      * @access public
+     * @param string $path
      * @return Response
      */
     public function post( $path, $form = array() )

Spacing +10 added lines, -10 removed lines

@@ -29,7 +29,7 @@ 
      * @access public
      * @return object
      */
-    public function __construct( $client )
+    public function __construct($client)
     {
         $this->client = $client;
     }
@@ -44,11 +44,11 @@ 
      * @param  array  $query
      * @return Response
      */
-    public function get( $path, $query = array() )
+    public function get($path, $query = array())
     {
         // Add query parameters to $path?
         if ($query) {
-            if (strpos($path, '?') === false ) {
+            if (strpos($path, '?') === false) {
                 $path .= '?' . http_build_query($query);
             } else {
                 $path .= ini_get('arg_separator.output') . http_build_query($query);
@@ -70,7 +70,7 @@ 
      * @access public
      * @return Response
      */
-    public function post( $path, $form = array() )
+    public function post($path, $form = array())
     {
         // Set the request params
         $this->set_url($path);
@@ -87,7 +87,7 @@ 
      * @access public
      * @return Response
      */
-    public function put( $path, $form = array() )
+    public function put($path, $form = array())
     {
         // Set the request params
         $this->set_url($path);
@@ -104,7 +104,7 @@ 
      * @access public
      * @return Response
      */
-    public function patch( $path, $form = array() )
+    public function patch($path, $form = array())
     {
         // Set the request params
         $this->set_url($path);
@@ -121,7 +121,7 @@ 
      * @access public
      * @return Response
      */
-    public function delete( $path, $form = array() )
+    public function delete($path, $form = array())
     {
         // Set the request params
         $this->set_url($path);
@@ -138,7 +138,7 @@ 
      * @access protected
      * @return void
      */
-    protected function set_url( $params )
+    protected function set_url($params)
     {
         curl_setopt($this->client->ch, CURLOPT_URL, Constants::API_URL . trim($params, '/'));
     }
@@ -153,13 +153,13 @@ 
      * @param  array  $form
      * @return Response
      */
-    protected function execute( $request_type, $form = array() )
+    protected function execute($request_type, $form = array())
     {
         // Set the HTTP request type
         curl_setopt($this->client->ch, CURLOPT_CUSTOMREQUEST, $request_type);
 
         // If additional data is delivered, we will send it along with the API request
-        if (is_array($form) && ! empty($form)) {
+        if (is_array($form) && !empty($form)) {
             curl_setopt($this->client->ch, CURLOPT_POSTFIELDS, http_build_query($form));
         }
 

QuickPay/Tests/api/ExceptionTest.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
         {
             throw new Exception($this->testMessage, $this->testCode);
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             $this->assertEquals($e->getMessage(), $this->testMessage);
             $this->assertEquals($e->getCode(), $this->testCode);

QuickPay/QuickPay.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     *
     * @access public
     */
-    public function __construct( $auth_string = '' )
+    public function __construct($auth_string = '')
     {
         $client = new Client($auth_string);
         $this->request = new Request($client);

QuickPay/API/Client.php

Spacing +3 added lines, -3 removed lines

@@ -31,10 +31,10 @@ 
      *
      * @access public
      */
-    public function __construct( $auth_string = '' )
+    public function __construct($auth_string = '')
     {
         // Check if lib cURL is enabled
-        if (!function_exists('curl_init') ) {
+        if (!function_exists('curl_init')) {
             throw new Exception('Lib cURL must be enabled on the server');
         }
 
@@ -54,7 +54,7 @@ 
      */
     public function shutdown()
     {
-        if (!empty($this->ch) ) {
+        if (!empty($this->ch)) {
             curl_close($this->ch);
         }
     }

QuickPay/API/Response.php

Spacing +6 added lines, -6 removed lines

@@ -48,7 +48,7 @@ 
      * @param string $received_headers the headers received
      * @param string $response_data    the http response body
      */
-    public function __construct( $status_code, $sent_headers, $received_headers, $response_data )
+    public function __construct($status_code, $sent_headers, $received_headers, $response_data)
     {
         $this->status_code = $status_code;
         $this->sent_headers = $sent_headers;
@@ -66,10 +66,10 @@ 
      * @param  boolan $keep_authorization_value Normally the value of the Authorization: header is masked. True keeps the sent value.
      * @return array            [integer, string[], string]
      */
-    public function as_raw( $keep_authorization_value = false )
+    public function as_raw($keep_authorization_value = false)
     {
         // To avoid unintentional logging of credentials the default is to mask the value of the Authorization: header
-        if ($keep_authorization_value ) {
+        if ($keep_authorization_value) {
             $sent_headers = $this->sent_headers;
         } else {
             // Avoid dependency on mbstring
@@ -100,7 +100,7 @@ 
      */
     public function as_array()
     {
-        if ($response = json_decode($this->response_data, true) ) {
+        if ($response = json_decode($this->response_data, true)) {
             return $response;
         }
 
@@ -116,7 +116,7 @@ 
      */
     public function as_object()
     {
-        if ($response = json_decode($this->response_data) ) {
+        if ($response = json_decode($this->response_data)) {
             return $response;
         }
 
@@ -144,7 +144,7 @@ 
      */
     public function is_success()
     {
-        if ($this->status_code > 299 ) {
+        if ($this->status_code > 299) {
             return false;
         }