Apply fixes from StyleCI (#545)

Author: GrahamCampbell

lib/Gitlab/Api/Groups.php

@@ -95,7 +95,7 @@ public function transfer($group_id, $project_id)
     }
 
     /**
-     * @param integer $id
+     * @param int $id
      * @param array $parameters
      * @return mixed
      */

lib/Gitlab/Api/Issues.php

@@ -308,10 +308,10 @@ public function awardEmoji($project_id, $issue_iid)
     }
 
     /**
-    * @param int|string $project_id
-    * @param int $issue_iid
-    * @return mixed
-    */
+     * @param int|string $project_id
+     * @param int $issue_iid
+     * @return mixed
+     */
     public function closedByMergeRequests($project_id, $issue_iid)
     {
         return $this->get($this->getProjectPath($project_id, 'issues/'.$this->encodePath($issue_iid)).'/closed_by');
@@ -326,7 +326,7 @@ public function showParticipants($project_id, $issue_iid)
     {
         return $this->get($this->getProjectPath($project_id, 'issues/' .$this->encodePath($issue_iid)).'/participants');
     }
-  
+
     /**
      * {@inheritDoc}
      */

lib/Gitlab/Api/ProjectNamespaces.php

@@ -21,7 +21,7 @@ public function all(array $parameters = [])
     }
 
     /**
-     * @param integer|string $namespace_id
+     * @param int|string $namespace_id
      * @return mixed
      */
     public function show($namespace_id)

lib/Gitlab/Api/Projects.php

@@ -277,7 +277,7 @@ public function cancelPipeline($project_id, $pipeline_id)
     {
         return $this->post($this->getProjectPath($project_id, 'pipelines/'.$this->encodePath($pipeline_id)).'/cancel');
     }
-    
+
     /**
      * @param int|string $project_id
      * @param int $pipeline_id
@@ -287,9 +287,9 @@ public function deletePipeline($project_id, $pipeline_id)
     {
         return $this->delete($this->getProjectPath($project_id, 'pipelines/'.$this->encodePath($pipeline_id)));
     }
-    
+
     /**
-     * @param integer $project_id
+     * @param int $project_id
      * @param array $parameters
      * @return mixed
      */
@@ -835,7 +835,7 @@ public function deployment($project_id, $deployment_id)
     {
         return $this->get($this->getProjectPath($project_id, 'deployments/'.$this->encodePath($deployment_id)));
     }
-    
+
     /**
      * @param mixed $project_id
      * @param array $parameters
@@ -863,7 +863,7 @@ public function addShare($project_id, array $parameters = [])
 
         return $this->post($this->getProjectPath($project_id, 'share'), $resolver->resolve($parameters));
     }
-    
+
     /**
      * @param mixed $project_id
      * @param int $group_id
@@ -918,7 +918,7 @@ public function removeBadge($project_id, $badge_id)
      * @param int $badge_id
      * @param array $parameters
      * @return mixed
-    */
+     */
     public function updateBadge($project_id, $badge_id, array $parameters = array())
     {
         return $this->put($this->getProjectPath($project_id, 'badges/' . $this->encodePath($badge_id)), $parameters);

lib/Gitlab/Model/AbstractModel.php

@@ -109,7 +109,8 @@ public function __get($property)
         if (!in_array($property, static::$properties)) {
             throw new RuntimeException(sprintf(
                 'Property "%s" does not exist for %s object',
-                $property, get_called_class()
+                $property,
+                get_called_class()
             ));
         }
 

lib/Gitlab/Model/Note.php

@@ -7,7 +7,7 @@
 /**
  * Class Note
  *
- * @property-read integer $id
+ * @property-read int $id
  * @property-read User $author
  * @property-read string $body
  * @property-read string $created_at

lib/Gitlab/Model/Project.php

@@ -781,8 +781,15 @@ public function mergeRequest($id)
      */
     public function createMergeRequest($source, $target, $title, $assignee = null, $description = null)
     {
-        $data = $this->client->mergeRequests()->create($this->id, $source, $target, $title, $assignee, $this->id,
-            $description);
+        $data = $this->client->mergeRequests()->create(
+            $this->id,
+            $source,
+            $target,
+            $title,
+            $assignee,
+            $this->id,
+            $description
+        );
 
         return MergeRequest::fromArray($this->getClient(), $this, $data);
     }

test/Gitlab/Tests/Api/AbstractApiTest.php

@@ -33,7 +33,7 @@ public function shouldPrepareBodyWithCleanArrays()
             ]
         );
 
-        $this->assertEquals($expectedBody, urldecode((string)$stream));
+        $this->assertEquals($expectedBody, urldecode((string) $stream));
     }
 
     protected function getAbstractApiMock(array $methods = [])

test/Gitlab/Tests/Api/GroupBoardsTest.php

@@ -40,7 +40,7 @@ public function shouldShowIssueBoard()
 
         $this->assertEquals($expectedArray, $api->show(1, 2));
     }
-    
+
     /**
      * @test
      */
@@ -123,7 +123,7 @@ public function shouldGetAllLists()
             ->with('groups/1/boards/2/lists')
             ->will($this->returnValue($expectedArray))
         ;
-    
+
         $this->assertEquals($expectedArray, $api->allLists(1, 2));
     }
 
@@ -143,14 +143,14 @@ public function shouldGetList()
                 'position' => 3
             )
         );
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
             ->with('groups/1/boards/2/lists/3')
             ->will($this->returnValue($expectedArray))
         ;
-    
+
         $this->assertEquals($expectedArray, $api->showList(1, 2, 3));
     }
 
@@ -170,7 +170,7 @@ public function shouldCreateList()
                 'position' => 3
             )
         );
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('post')
@@ -197,14 +197,14 @@ public function shouldUpdateList()
                 'position' => 1
             )
         );
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('put')
             ->with('groups/5/boards/2/lists/3', array('position' => 1))
             ->will($this->returnValue($expectedArray))
         ;
-    
+
         $this->assertEquals($expectedArray, $api->updateList(5, 2, 3, 1));
     }
 
@@ -214,14 +214,14 @@ public function shouldUpdateList()
     public function shouldDeleteList()
     {
         $expectedBool = true;
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('delete')
             ->with('groups/1/boards/2/lists/3')
             ->will($this->returnValue($expectedBool))
         ;
-    
+
         $this->assertEquals($expectedBool, $api->deleteList(1, 2, 3));
     }
 

test/Gitlab/Tests/Api/IssueBoardsTest.php

@@ -40,7 +40,7 @@ public function shouldShowIssueBoard()
 
         $this->assertEquals($expectedArray, $api->show(1, 2));
     }
-    
+
     /**
      * @test
      */
@@ -123,7 +123,7 @@ public function shouldGetAllLists()
             ->with('projects/1/boards/2/lists')
             ->will($this->returnValue($expectedArray))
         ;
-    
+
         $this->assertEquals($expectedArray, $api->allLists(1, 2));
     }
 
@@ -143,14 +143,14 @@ public function shouldGetList()
                 'position' => 3
             )
         );
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('get')
             ->with('projects/1/boards/2/lists/3')
             ->will($this->returnValue($expectedArray))
         ;
-    
+
         $this->assertEquals($expectedArray, $api->showList(1, 2, 3));
     }
 
@@ -170,7 +170,7 @@ public function shouldCreateList()
                 'position' => 3
             )
         );
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('post')
@@ -197,14 +197,14 @@ public function shouldUpdateList()
                 'position' => 1
             )
         );
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('put')
             ->with('projects/5/boards/2/lists/3', array('position' => 1))
             ->will($this->returnValue($expectedArray))
         ;
-    
+
         $this->assertEquals($expectedArray, $api->updateList(5, 2, 3, 1));
     }
 
@@ -214,14 +214,14 @@ public function shouldUpdateList()
     public function shouldDeleteList()
     {
         $expectedBool = true;
-    
+
         $api = $this->getApiMock();
         $api->expects($this->once())
             ->method('delete')
             ->with('projects/1/boards/2/lists/3')
             ->will($this->returnValue($expectedBool))
         ;
-    
+
         $this->assertEquals($expectedBool, $api->deleteList(1, 2, 3));
     }