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)

Indentation +7 added lines, -7 removed lines

@@ -12,13 +12,13 @@
 class Exception extends \Exception
 {
     /**
-    * __Construct function.
-    *
-    * Redefine the exception so message isn't optional
-    *
-    * @access public
-    * @return void
-    */
+     * __Construct function.
+     *
+     * Redefine the exception so message isn't optional
+     *
+     * @access public
+     * @return void
+     */
     public function __construct($message, $code = 0, Exception $previous = null)
     {
         // Make sure everything is assigned properly

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);

Braces +1 added lines, -2 removed lines

@@ -14,8 +14,7 @@
         try
         {
             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

Indentation +7 added lines, -7 removed lines

@@ -21,13 +21,13 @@
 
 
     /**
-    * __construct function.
-    *
-    * Instantiates the main class.
-    * Creates a client which is passed to the request construct.
-    *
-    * @access public
-    */
+     * __construct function.
+     *
+     * Instantiates the main class.
+     * Creates a client which is passed to the request construct.
+     *
+     * @access public
+     */
     public function __construct( $auth_string = '' )
     {
         $client = new Client($auth_string);

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

Indentation +7 added lines, -7 removed lines

@@ -1,13 +1,13 @@
 <?php
 namespace QuickPay\API;
 /**
- * @class       QuickPay_Client
- * @since       1.0.0
- * @package     QuickPay
- * @category    Class
- * @author      Patrick Tolvstein, Perfect Solution ApS
- * @docs        http://tech.quickpay.net/api/
- */
+     * @class       QuickPay_Client
+     * @since       1.0.0
+     * @package     QuickPay
+     * @category    Class
+     * @author      Patrick Tolvstein, Perfect Solution ApS
+     * @docs        http://tech.quickpay.net/api/
+     */
 class Client
 {
     /**

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

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@ 
 <?php
 namespace QuickPay\API;
 /**
- * @class         QuickPay_Response
- * @since        1.0.0
- * @package        QuickPay
- * @category    Class
- * @author         Patrick Tolvstein, Perfect Solution ApS
- * @docs        http://tech.quickpay.net/api/
- */
+     * @class         QuickPay_Response
+     * @since        1.0.0
+     * @package        QuickPay
+     * @category    Class
+     * @author         Patrick Tolvstein, Perfect Solution ApS
+     * @docs        http://tech.quickpay.net/api/
+     */
 class Response
 {
     /**
@@ -85,7 +85,7 @@ 
         return [
             $this->status_code,
             ['sent' => $sent_headers,
-             'received' => $this->received_headers,
+                'received' => $this->received_headers,
             ],
             $this->response_data,
         ];

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;
         }
 

QuickPay/API/Constants.php

Indentation +7 added lines, -7 removed lines

@@ -1,13 +1,13 @@
 <?php
 namespace QuickPay\API;
 /**
- * @class       QuickPay_Constants
- * @since       1.0.0
- * @package     QuickPay
- * @category    Class
- * @author      Patrick Tolvstein, Perfect Solution ApS
- * @docs        http://tech.quickpay.net/api/
- */
+     * @class       QuickPay_Constants
+     * @since       1.0.0
+     * @package     QuickPay
+     * @category    Class
+     * @author      Patrick Tolvstein, Perfect Solution ApS
+     * @docs        http://tech.quickpay.net/api/
+     */
 class Constants
 {
     /**