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

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

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
      */

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

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

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

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() )
     {

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(!class_exists("\CURLFile") && defined('CURLOPT_SAFE_UPLOAD'))
-                    $ch->setOption( CURLOPT_SAFE_UPLOAD, false );
+                if(!class_exists("\CURLFile") && 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)
     {

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;

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