dirkgroenen / pinterest-api-php

src/Pinterest/Auth/PinterestOAuth.php

Doc Comments +1 added lines

@@ -54,6 +54,7 @@
      *
      * @param  string   $client_id
      * @param  string   $client_secret
+     * @param Request $request
      */
     public function __construct( $client_id, $client_secret, $request )
     {

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace DirkGroenen\Pinterest\Auth;
 
 use DirkGroenen\Pinterest\Transport\Request;
-use DirkGroenen\Pinterest\Exceptions\PinterestException;
 
 class PinterestOAuth {
 

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * Copyright 2015 Dirk Groenen
- *
- * (c) Dirk Groenen <dirk@bitlabs.nl>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * Copyright 2015 Dirk Groenen
+     *
+     * (c) Dirk Groenen <dirk@bitlabs.nl>
+     *
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace DirkGroenen\Pinterest\Auth;
 

Spacing +6 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
      * @param  string   $client_id
      * @param  string   $client_secret
      */
-    public function __construct( $client_id, $client_secret, $request )
+    public function __construct($client_id, $client_secret, $request)
     {
         $this->client_id = $client_id;
         $this->client_secret = $client_secret;
@@ -75,7 +75,7 @@ 
      * @param  string   $redirect_uri
      * @return string
      */
-    public function getLoginUrl( $redirect_uri, $scopes = array("read_public"), $response_type = "code" )
+    public function getLoginUrl($redirect_uri, $scopes = array("read_public"), $response_type = "code")
     {
         $queryparams = array(
             "response_type"     => $response_type,
@@ -87,7 +87,7 @@ 
         );
 
         // Build url and return it
-        return sprintf( "%s?%s", self::AUTH_HOST, http_build_query($queryparams) );
+        return sprintf("%s?%s", self::AUTH_HOST, http_build_query($queryparams));
     }
 
     /**
@@ -98,7 +98,7 @@ 
      */
     private function generateState()
     {
-        return substr( md5( rand() ), 0, 7 );
+        return substr(md5(rand()), 0, 7);
     }
 
     /**
@@ -128,7 +128,7 @@ 
      * @param  string   $code
      * @return array
      */
-    public function getOAuthToken( $code )
+    public function getOAuthToken($code)
     {
         // Build data array
         $data = array(
@@ -151,7 +151,7 @@ 
      * @param  string   $access_token
      * @return void
      */
-    public function setOAuthToken( $access_token )
+    public function setOAuthToken($access_token)
     {
         $this->request->setAccessToken($access_token);
     }

src/Pinterest/Endpoints/Boards.php

Doc Comments +3 added lines, -3 removed lines

@@ -22,7 +22,7 @@ 
      * @param string    $board_id
      * @param array     $data
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Board
+     * @return Board
      */
     public function get( $board_id, array $data = [] )
     {
@@ -36,7 +36,7 @@ 
      * @access public
      * @param  array    $data
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Board
+     * @return Board
      */
     public function create( array $data )
     {
@@ -50,7 +50,7 @@ 
      * @access public
      * @param  string    $board_id
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Board
+     * @return boolean
      */
     public function delete( $board_id )
     {

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace DirkGroenen\Pinterest\Endpoints;
 
 use DirkGroenen\Pinterest\Models\Board;
-use DirkGroenen\Pinterest\Models\Collection;
 
 class Boards extends Endpoint {
 

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php 
 /**
- * Copyright 2015 Dirk Groenen 
- *
- * (c) Dirk Groenen <dirk@bitlabs.nl>
- * 
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * Copyright 2015 Dirk Groenen 
+     *
+     * (c) Dirk Groenen <dirk@bitlabs.nl>
+     * 
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace DirkGroenen\Pinterest\Endpoints;
 

Spacing +8 added lines, -8 removed lines

@@ -24,10 +24,10 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Board
      */
-    public function get( $board_id, array $data = [] )
+    public function get($board_id, array $data = [])
     {
-        $response = $this->request->get( sprintf("boards/%s", $board_id), $data );
-        return new Board( $this->master, $response );
+        $response = $this->request->get(sprintf("boards/%s", $board_id), $data);
+        return new Board($this->master, $response);
     }
 
     /**
@@ -38,10 +38,10 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Board
      */
-    public function create( array $data )
+    public function create(array $data)
     {
-        $response = $this->request->post( "boards", $data );
-        return new Board( $this->master, $response );
+        $response = $this->request->post("boards", $data);
+        return new Board($this->master, $response);
     }
 
     /**
@@ -52,9 +52,9 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Board
      */
-    public function delete( $board_id )
+    public function delete($board_id)
     {
-        $response = $this->request->delete( sprintf("boards/%s", $board_id) );
+        $response = $this->request->delete(sprintf("boards/%s", $board_id));
         return true;
     }
 }
\ No newline at end of file

src/Pinterest/Endpoints/Endpoint.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
     /**
      * Create a new model instance
      *
-     * @param  Transport\Request    $request
+     * @param  Request    $request
      * @param  Pinterest            $master
      * @return void
      */

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php 
 /**
- * Copyright 2015 Dirk Groenen 
- *
- * (c) Dirk Groenen <dirk@bitlabs.nl>
- * 
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * Copyright 2015 Dirk Groenen 
+     *
+     * (c) Dirk Groenen <dirk@bitlabs.nl>
+     * 
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace DirkGroenen\Pinterest\Endpoints;
 

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      * @param  Pinterest            $master
      * @return void
      */
-    public function __construct( Request $request, Pinterest $master )
+    public function __construct(Request $request, Pinterest $master)
     {
         $this->request = $request;
         $this->master = $master;

src/Pinterest/Endpoints/Pins.php

Doc Comments +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
      * @param  string   $pin_id
      * @param array     $data
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Pin
+     * @return Pin
      */
     public function get( $pin_id, array $data = [] )
     {
@@ -37,7 +37,7 @@ 
      * @param  string   $board_id
      * @param array     $data
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Collection
+     * @return Collection
      */
     public function fromBoard( $board_id, array $data = [] )
     {
@@ -51,7 +51,7 @@ 
      * @access public
      * @param  array    $data
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Pin
+     * @return Pin
      */
     public function create( array $data )
     {
@@ -75,7 +75,7 @@ 
      * @param  string   $pin_id
      * @param  array    $data
      * @throws Exceptions/PinterestExceptions
-     * @return Models\Pin
+     * @return Pin
      */
     public function update( $pin_id, array $data )
     {

Spacing +17 added lines, -17 removed lines

@@ -24,10 +24,10 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Pin
      */
-    public function get( $pin_id, array $data = [] )
+    public function get($pin_id, array $data = [])
     {
-        $response = $this->request->get( sprintf("pins/%s", $pin_id), $data );
-        return new Pin( $this->master, $response );
+        $response = $this->request->get(sprintf("pins/%s", $pin_id), $data);
+        return new Pin($this->master, $response);
     }
 
     /**
@@ -39,10 +39,10 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Collection
      */
-    public function fromBoard( $board_id, array $data = [] )
+    public function fromBoard($board_id, array $data = [])
     {
-        $response = $this->request->get( sprintf("boards/%s/pins", $board_id), $data );
-        return new Collection( $this->master, $response, "Pin" );
+        $response = $this->request->get(sprintf("boards/%s/pins", $board_id), $data);
+        return new Collection($this->master, $response, "Pin");
     }
 
     /**
@@ -53,19 +53,19 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Pin
      */
-    public function create( array $data )
+    public function create(array $data)
     {
-        if( array_key_exists("image", $data) ){
-            if(class_exists("\CURLFile")){
+        if (array_key_exists("image", $data)) {
+            if (class_exists("\CURLFile")) {
                 $data["image"] = new \CURLFile($data['image']);
             }
-            else{
+            else {
                 $data["image"] = '@' . $data['image'];  
             }
         }
 
-        $response = $this->request->post( "pins", $data );
-        return new Pin( $this->master, $response );
+        $response = $this->request->post("pins", $data);
+        return new Pin($this->master, $response);
     }
 
     /**
@@ -77,10 +77,10 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return Models\Pin
      */
-    public function update( $pin_id, array $data )
+    public function update($pin_id, array $data)
     {
-        $response = $this->request->update( sprintf("pins/%s", $pin_id), $data );
-        return new Pin( $this->master, $response );
+        $response = $this->request->update(sprintf("pins/%s", $pin_id), $data);
+        return new Pin($this->master, $response);
     }
 
     /**
@@ -91,9 +91,9 @@ 
      * @throws Exceptions/PinterestExceptions
      * @return boolean
      */
-    public function delete( $pin_id )
+    public function delete($pin_id)
     {
-        $response = $this->request->delete( sprintf("pins/%s", $pin_id) );
+        $response = $this->request->delete(sprintf("pins/%s", $pin_id));
         return true;
     }
 }
\ No newline at end of file

Braces +1 added lines, -2 removed lines

@@ -58,8 +58,7 @@
         if( array_key_exists("image", $data) ){
             if(class_exists("\CURLFile")){
                 $data["image"] = new \CURLFile($data['image']);
-            }
-            else{
+            } else{
                 $data["image"] = '@' . $data['image'];  
             }
         }

src/Pinterest/Models/Collection.php

Doc Comments +1 added lines, -1 removed lines

@@ -251,7 +251,7 @@
      * Make the collection items iteratable
      *
      * @access public
-     * @return ArrayIterator
+     * @return \ArrayIterator
      */
     public function getIterator() {
         return new \ArrayIterator($this->items);

Indentation +8 added lines, -8 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 /**
- * Copyright 2015 Dirk Groenen
- *
- * (c) Dirk Groenen <dirk@bitlabs.nl>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * Copyright 2015 Dirk Groenen
+     *
+     * (c) Dirk Groenen <dirk@bitlabs.nl>
+     *
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace DirkGroenen\Pinterest\Models;
 
@@ -80,7 +80,7 @@ 
             $this->items = $items->data;
         }
         else{
-           throw new PinterestException("$items needs to be an instance of Transport\Response or an array.");
+            throw new PinterestException("$items needs to be an instance of Transport\Response or an array.");
         }
 
         // Transform the raw collection data to models

Spacing +13 added lines, -13 removed lines

@@ -61,25 +61,25 @@ 
      * @param  string               $model
      * @throws InvalidModelException
      */
-    public function __construct( Pinterest $master, $items, $model ){
+    public function __construct(Pinterest $master, $items, $model) {
         $this->master = $master;
 
         // Create class path
-        $this->model = ucfirst( strtolower($model) );
+        $this->model = ucfirst(strtolower($model));
 
-        if(!class_exists("\\DirkGroenen\\Pinterest\\Models\\" . $this->model))
+        if (!class_exists("\\DirkGroenen\\Pinterest\\Models\\" . $this->model))
             throw new InvalidModelException;
 
         // Get items and response instance
-        if( is_array($items) ){
+        if (is_array($items)) {
             $this->response = null;
             $this->items = $items;
         }
-        else if( $items instanceof \DirkGroenen\Pinterest\Transport\Response ){
+        else if ($items instanceof \DirkGroenen\Pinterest\Transport\Response) {
             $this->response = $items;
             $this->items = $items->data;
         }
-        else{
+        else {
            throw new PinterestException("$items needs to be an instance of Transport\Response or an array.");
         }
 
@@ -87,10 +87,10 @@ 
         $this->items = $this->buildCollectionModels($this->items);
 
         // Add pagination object
-        if( isset($this->response->page) && !empty($this->response->page['next']) ){
+        if (isset($this->response->page) && !empty($this->response->page['next'])) {
             $this->pagination = $this->response->page;
         }
-        else{
+        else {
             $this->pagination = false;
         }
     }
@@ -117,9 +117,9 @@ 
     {
         $modelcollection = [];
 
-        foreach($items as $item){
+        foreach ($items as $item) {
             $class = new \ReflectionClass("\\DirkGroenen\\Pinterest\\Models\\" . $this->model);
-            $modelcollection[] = $class->newInstanceArgs( [$this->master, $item] );
+            $modelcollection[] = $class->newInstanceArgs([$this->master, $item]);
         }
 
         return $modelcollection;
@@ -133,7 +133,7 @@ 
      */
     public function hasNextPage()
     {
-        return ($this->response != null && isset($this->response->page['next']) );
+        return ($this->response != null && isset($this->response->page['next']));
     }
 
     /**
@@ -143,7 +143,7 @@ 
      * @param  int $index
      * @return Model
      */
-    public function get( $index )
+    public function get($index)
     {
         return $this->items[$index];
     }
@@ -158,7 +158,7 @@ 
     {
         $items = [];
 
-        foreach($this->items as $item){
+        foreach ($this->items as $item) {
             $items[] = $item->toArray();
         }
 

Braces +6 added lines, -8 removed lines

@@ -67,19 +67,18 @@ 
         // Create class path
         $this->model = ucfirst( strtolower($model) );
 
-        if(!class_exists("\\DirkGroenen\\Pinterest\\Models\\" . $this->model))
-            throw new InvalidModelException;
+        if(!class_exists("\\DirkGroenen\\Pinterest\\Models\\" . $this->model)) {
+                    throw new InvalidModelException;
+        }
 
         // Get items and response instance
         if( is_array($items) ){
             $this->response = null;
             $this->items = $items;
-        }
-        else if( $items instanceof \DirkGroenen\Pinterest\Transport\Response ){
+        } else if( $items instanceof \DirkGroenen\Pinterest\Transport\Response ){
             $this->response = $items;
             $this->items = $items->data;
-        }
-        else{
+        } else{
            throw new PinterestException("$items needs to be an instance of Transport\Response or an array.");
         }
 
@@ -89,8 +88,7 @@ 
         // Add pagination object
         if( isset($this->response->page) && !empty($this->response->page['next']) ){
             $this->pagination = $this->response->page;
-        }
-        else{
+        } else{
             $this->pagination = false;
         }
     }

src/Pinterest/Pinterest.php

Doc Comments -1 removed lines

@@ -51,7 +51,6 @@
      * @param  string       $client_id
      * @param  string       $client_secret
      * @param  CurlBuilder  $curlbuilder
-     * @param  string       $redirect_uri
      */
     public function __construct($client_id, $client_secret, $curlbuilder = null)
     {

Spacing +5 added lines, -5 removed lines

@@ -55,11 +55,11 @@ 
      */
     public function __construct($client_id, $client_secret, $curlbuilder = null)
     {
-        if($curlbuilder == null)
+        if ($curlbuilder == null)
             $curlbuilder = new CurlBuilder();
 
         // Create new instance of Transport\Request
-        $this->request = new Request( $curlbuilder );
+        $this->request = new Request($curlbuilder);
 
         // Create and set new instance of the OAuth class
         $this->auth = new PinterestOAuth($client_id, $client_secret, $this->request);
@@ -79,15 +79,15 @@ 
         $class = "\\DirkGroenen\\Pinterest\\Endpoints\\" . ucfirst($endpoint);
 
         // Check if an instance has already been initiated
-        if(!isset($this->cachedEndpoints[$endpoint])){
+        if (!isset($this->cachedEndpoints[$endpoint])) {
             // Check endpoint existence
-            if(!class_exists($class))
+            if (!class_exists($class))
                 throw new InvalidEndpointException;
 
             // Create a reflection of the called class and initialize it
             // with a reference to the request class
             $ref = new \ReflectionClass($class);
-            $obj = $ref->newInstanceArgs([ $this->request, $this ]);
+            $obj = $ref->newInstanceArgs([$this->request, $this]);
 
             $this->cachedEndpoints[$endpoint] = $obj;
         }

Braces +6 added lines, -4 removed lines

@@ -55,8 +55,9 @@ 
      */
     public function __construct($client_id, $client_secret, $curlbuilder = null)
     {
-        if($curlbuilder == null)
-            $curlbuilder = new CurlBuilder();
+        if($curlbuilder == null) {
+                    $curlbuilder = new CurlBuilder();
+        }
 
         // Create new instance of Transport\Request
         $this->request = new Request( $curlbuilder );
@@ -81,8 +82,9 @@ 
         // Check if an instance has already been initiated
         if(!isset($this->cachedEndpoints[$endpoint])){
             // Check endpoint existence
-            if(!class_exists($class))
-                throw new InvalidEndpointException;
+            if(!class_exists($class)) {
+                            throw new InvalidEndpointException;
+            }
 
             // Create a reflection of the called class and initialize it
             // with a reference to the request class

src/Pinterest/Transport/Request.php

Doc Comments +2 added lines, -7 removed lines

@@ -89,7 +89,6 @@ 
      * Make a post request to the given endpoint
      *
      * @access public
-     * @param  string   $endpoint
      * @param  array    $parameters
      * @return [type]
      */
@@ -102,7 +101,6 @@ 
      * Make a delete request to the given endpoint
      *
      * @access public
-     * @param  string   $endpoint
      * @param  array    $parameters
      * @return [type]
      */
@@ -115,7 +113,6 @@ 
      * Make an update request to the given endpoint
      *
      * @access public
-     * @param  string   $endpoint
      * @param  array    $parameters
      * @return [type]
      */
@@ -127,9 +124,7 @@ 
     /**
      * Create a new model instance
      *
-     * @param  Transport\Request    $request
-     * @param  Pinterest            $master
-     * @return void
+     * @return CurlBuilder
      */
     public function getHeaders()
     {
@@ -144,7 +139,7 @@ 
      * @param  string   $apiCall
      * @param  array    $parameters
      * @param  array    $headers
-     * @return mixed
+     * @return Response
      */
     public function execute( $method, $apiCall, array $parameters = array(), $headers = array() )
     {

Spacing +25 added lines, -25 removed lines

@@ -49,7 +49,7 @@ 
      *
      * @param  CurlBuilder   $curlbuilder
      */
-    public function __construct( CurlBuilder $curlbuilder )
+    public function __construct(CurlBuilder $curlbuilder)
     {
         $this->curlbuilder = $curlbuilder;
     }
@@ -60,7 +60,7 @@ 
      * @access public
      * @param  string   $token
      */
-    public function setAccessToken( $token )
+    public function setAccessToken($token)
     {
         $this->access_token = $token;
     }
@@ -73,9 +73,9 @@ 
      * @param  array    $parameters
      * @return [type]
      */
-    public function get( $endpoint, array $parameters = array() )
+    public function get($endpoint, array $parameters = array())
     {
-        if(!empty($parameters)) {
+        if (!empty($parameters)) {
             $path = sprintf("%s/?%s", $endpoint, http_build_query($parameters));
         }
         else {
@@ -93,9 +93,9 @@ 
      * @param  array    $parameters
      * @return [type]
      */
-    public function post( $path, array $parameters = array() )
+    public function post($path, array $parameters = array())
     {
-        return $this->execute("POST", sprintf("%s%s", $this->host, $path), $parameters );
+        return $this->execute("POST", sprintf("%s%s", $this->host, $path), $parameters);
     }
 
     /**
@@ -106,9 +106,9 @@ 
      * @param  array    $parameters
      * @return [type]
      */
-    public function delete( $path, array $parameters = array() )
+    public function delete($path, array $parameters = array())
     {
-        return $this->execute("DELETE", sprintf("%s%s", $this->host, $path) . "/", $parameters );
+        return $this->execute("DELETE", sprintf("%s%s", $this->host, $path) . "/", $parameters);
     }
 
     /**
@@ -119,9 +119,9 @@ 
      * @param  array    $parameters
      * @return [type]
      */
-    public function update( $path, array $parameters = array() )
+    public function update($path, array $parameters = array())
     {
-        return $this->execute("PATCH", sprintf("%s%s", $this->host, $path) . "/", $parameters );
+        return $this->execute("PATCH", sprintf("%s%s", $this->host, $path) . "/", $parameters);
     }
     
     /**
@@ -146,10 +146,10 @@ 
      * @param  array    $headers
      * @return mixed
      */
-    public function execute( $method, $apiCall, array $parameters = array(), $headers = array() )
+    public function execute($method, $apiCall, array $parameters = array(), $headers = array())
     {
         // Check if the access token needs to be added
-        if($this->access_token != null){
+        if ($this->access_token != null) {
             $headers = array_merge($headers, array(
                 "Authorization: Bearer " . $this->access_token,
                 "Content-ype: multipart/form-data",
@@ -160,7 +160,7 @@ 
         $ch = $this->curlbuilder->create();
 
         // Set default options
-        $ch->setOptions( array(
+        $ch->setOptions(array(
             CURLOPT_URL             => $apiCall,
             CURLOPT_HTTPHEADER      => $headers,
             CURLOPT_CONNECTTIMEOUT  => 20,
@@ -170,28 +170,28 @@ 
             CURLOPT_SSL_VERIFYHOST  => false,
             CURLOPT_HEADER          => false,
             CURLINFO_HEADER_OUT     => true
-        ) );
+        ));
 
         switch ($method) {
             case 'POST':
-                $ch->setOptions( array(
+                $ch->setOptions(array(
                     CURLOPT_CUSTOMREQUEST   => 'POST',
                     CURLOPT_POST            => count($parameters),
                     CURLOPT_POSTFIELDS      => $parameters
-                ) );
+                ));
 
-                if(defined('CURLOPT_SAFE_UPLOAD'))
-                    $ch->setOption( CURLOPT_SAFE_UPLOAD, false );
+                if (defined('CURLOPT_SAFE_UPLOAD'))
+                    $ch->setOption(CURLOPT_SAFE_UPLOAD, false);
 
                 break;
             case 'DELETE':
-                $ch->setOption( CURLOPT_CUSTOMREQUEST, "DELETE" );
+                $ch->setOption(CURLOPT_CUSTOMREQUEST, "DELETE");
                 break;
             case 'PATCH':
-                $ch->setOption( CURLOPT_CUSTOMREQUEST, "PATCH" );
+                $ch->setOption(CURLOPT_CUSTOMREQUEST, "PATCH");
                 break;
             default:
-                $ch->setOption( CURLOPT_CUSTOMREQUEST, "GET" );
+                $ch->setOption(CURLOPT_CUSTOMREQUEST, "GET");
                 break;
         }
 
@@ -200,16 +200,16 @@ 
         $response_data = $ch->execute();
 
         // Check if we have a valid response
-        if ( !$response_data || $ch->hasErrors() ) {
-            throw new PinterestException( 'Error: execute() - cURL error: ' . $ch->getErrors(), $ch->getErrorNumber() );
+        if (!$response_data || $ch->hasErrors()) {
+            throw new PinterestException('Error: execute() - cURL error: ' . $ch->getErrors(), $ch->getErrorNumber());
         }
 
         // Initiate the response
         $response = new Response($response_data, $ch);
 
         // Check the response code
-        if ( $response->getResponseCode() >= 400 ) {
-            throw new PinterestException( 'Pinterest error (code: ' . $response->getResponseCode() . ') with message: ' . $response->message, $response->getResponseCode() );
+        if ($response->getResponseCode() >= 400) {
+            throw new PinterestException('Pinterest error (code: ' . $response->getResponseCode() . ') with message: ' . $response->message, $response->getResponseCode());
         }
         $this->headers = $ch->getHeaders();
 

Braces +4 added lines, -4 removed lines

@@ -77,8 +77,7 @@ 
     {
         if(!empty($parameters)) {
             $path = sprintf("%s/?%s", $endpoint, http_build_query($parameters));
-        }
-        else {
+        } else {
             $path = $endpoint;
         }
 
@@ -180,8 +179,9 @@ 
                     CURLOPT_POSTFIELDS      => $parameters
                 ) );
 
-                if(defined('CURLOPT_SAFE_UPLOAD'))
-                    $ch->setOption( CURLOPT_SAFE_UPLOAD, false );
+                if(defined('CURLOPT_SAFE_UPLOAD')) {
+                                    $ch->setOption( CURLOPT_SAFE_UPLOAD, false );
+                }
 
                 break;
             case 'DELETE':

src/Pinterest/Transport/Response.php

Doc Comments +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
     /**
      * Constructor
      *
-     * @param  array        $response
+     * @param  string        $response
      * @param  CurlBuilder  $curl
      * @param  curl    $curl
      */
@@ -75,7 +75,7 @@ 
      *
      * @access public
      * @param  string   $key
-     * @return array
+     * @return boolean
      */
     public function __isset($key)
     {

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace DirkGroenen\Pinterest\Transport;
 
 use DirkGroenen\Pinterest\Utils\CurlBuilder;
-use DirkGroenen\Pinterest\Exceptions\PinterestException;
 
 class Response {
 

Spacing +3 added lines, -3 removed lines

@@ -36,12 +36,12 @@ 
      * @param  CurlBuilder  $curl
      * @param  curl    $curl
      */
-    public function __construct( $response, CurlBuilder $curl )
+    public function __construct($response, CurlBuilder $curl)
     {
         $this->response = $response;
         $this->curl = $curl;
 
-        if( is_string($response) ){
+        if (is_string($response)) {
             $this->response = $this->decodeString($response);
         }
     }
@@ -53,7 +53,7 @@ 
      * @param  string $response
      * @return array
      */
-    private function decodeString( $response )
+    private function decodeString($response)
     {
         return json_decode($response, true);
     }

src/Pinterest/Utils/CurlBuilder.php

Doc Comments +4 added lines, -4 removed lines

@@ -80,7 +80,7 @@ 
      * Execute the curl request
      *
      * @access public
-     * @return mixed
+     * @return false|string
      */
     public function execute()
     {
@@ -91,7 +91,7 @@ 
      * Check if the curl request ended up with errors
      *
      * @access public
-     * @return boolean
+     * @return integer
      */
     public function hasErrors()
     {
@@ -158,7 +158,7 @@ 
      * Parse string headers into array
      *
      * @access private
-     * @param array $headers
+     * @param string $headers
      * @return array
      */
     private function parseHeaders($headers) {
@@ -180,7 +180,7 @@ 
      *
      * @see http://slopjong.de/2012/03/31/curl-follow-locations-with-safe_mode-enabled-or-open_basedir-set/
      * @access private
-     * @return mixed
+     * @return false|string
      */
     private function execFollow() {
         $mr = 5;

Unused Use Statements -2 removed lines

@@ -10,8 +10,6 @@
 
 namespace DirkGroenen\Pinterest\Utils;
 
-use DirkGroenen\Pinterest\Exceptions\PinterestException;
-
 class CurlBuilder {
 
     /**

Spacing +16 added lines, -16 removed lines

@@ -55,9 +55,9 @@ 
      * @param  string   $value
      * @return $this
      */
-    public function setOption( $option, $value )
+    public function setOption($option, $value)
     {
-        curl_setopt($this->curl, $option, $value );
+        curl_setopt($this->curl, $option, $value);
 
         return $this;
     }
@@ -69,7 +69,7 @@ 
      * @param  array   $options
      * @return $this
      */
-    public function setOptions( array $options = [] )
+    public function setOptions(array $options = [])
     {
         curl_setopt_array($this->curl, $options);
 
@@ -163,7 +163,7 @@ 
      */
     private function parseHeaders($headers) {
         $result = array();
-        foreach(explode("\n", $headers) as $row){
+        foreach (explode("\n", $headers) as $row) {
             $header = explode(':', $row, 2);
             if (count($header) == 2)
                 $result[$header[0]] = trim($header[1]);
@@ -186,14 +186,14 @@ 
         $mr = 5;
         $body = null;
 
-        if(ini_get("open_basedir") == "" && ini_get("safe_mode" == "Off")){
+        if (ini_get("open_basedir") == "" && ini_get("safe_mode" == "Off")) {
             curl_setopt($this->curl, CURLOPT_FOLLOWLOCATION, $mr > 0);
             curl_setopt($this->curl, CURLOPT_MAXREDIRS, $mr);
         }
-        else{
+        else {
             $this->setOption(CURLOPT_FOLLOWLOCATION, false);
 
-            if($mr > 0){
+            if ($mr > 0) {
                 $original_url = $this->getInfo(CURLINFO_EFFECTIVE_URL);
                 $newurl = $original_url;
 
@@ -203,7 +203,7 @@ 
                 //curl_setopt($rch, CURLOPT_NOBODY, true);
                 curl_setopt($rch, CURLOPT_FORBID_REUSE, false);
 
-                do{
+                do {
                     curl_setopt($rch, CURLOPT_URL, $newurl);
                     $response = curl_exec($rch);
 
@@ -211,29 +211,29 @@ 
                     $header = substr($response, 0, $header_size);
                     $body = substr($response, $header_size);
 
-                    if(curl_errno($rch)){
+                    if (curl_errno($rch)) {
                         $code = 0;
                     }
-                    else{
+                    else {
                         $code = curl_getinfo($rch, CURLINFO_HTTP_CODE);
 
                         if ($code == 301 || $code == 302) {
                             preg_match('/Location:(.*?)\n/i', $header, $matches);
                             $newurl = trim(array_pop($matches));
                         }
-                        else{
+                        else {
                             $code = 0;
                         }
                     }
-                } while($code && --$mr);
+                } while ($code && --$mr);
 
                 curl_close($rch);
 
-                if(!$mr){
-                    if ($maxredirect === null){
+                if (!$mr) {
+                    if ($maxredirect === null) {
                         trigger_error('Too many redirects.', E_USER_WARNING);
                     }
-                    else{
+                    else {
                         $maxredirect = 0;
                     }
 
@@ -245,7 +245,7 @@ 
             }
         }
 
-        if (!$body){
+        if (!$body) {
             curl_setopt($this->curl, CURLOPT_HEADER, true);
             $response = curl_exec($this->curl);
 

Braces +9 added lines, -12 removed lines

@@ -165,10 +165,11 @@ 
         $result = array();
         foreach(explode("\n", $headers) as $row){
             $header = explode(':', $row, 2);
-            if (count($header) == 2)
-                $result[$header[0]] = trim($header[1]);
-            else
-                $result[] = $header[0];
+            if (count($header) == 2) {
+                            $result[$header[0]] = trim($header[1]);
+            } else {
+                            $result[] = $header[0];
+            }
         }
         return $result;
     }
@@ -189,8 +190,7 @@ 
         if(ini_get("open_basedir") == "" && ini_get("safe_mode" == "Off")){
             curl_setopt($this->curl, CURLOPT_FOLLOWLOCATION, $mr > 0);
             curl_setopt($this->curl, CURLOPT_MAXREDIRS, $mr);
-        }
-        else{
+        } else{
             $this->setOption(CURLOPT_FOLLOWLOCATION, false);
 
             if($mr > 0){
@@ -213,15 +213,13 @@ 
 
                     if(curl_errno($rch)){
                         $code = 0;
-                    }
-                    else{
+                    } else{
                         $code = curl_getinfo($rch, CURLINFO_HTTP_CODE);
 
                         if ($code == 301 || $code == 302) {
                             preg_match('/Location:(.*?)\n/i', $header, $matches);
                             $newurl = trim(array_pop($matches));
-                        }
-                        else{
+                        } else{
                             $code = 0;
                         }
                     }
@@ -232,8 +230,7 @@ 
                 if(!$mr){
                     if ($maxredirect === null){
                         trigger_error('Too many redirects.', E_USER_WARNING);
-                    }
-                    else{
+                    } else{
                         $maxredirect = 0;
                     }