fix optional params on MR create

Author: vagrus

lib/Gitlab/Api/MergeRequests.php

@@ -109,21 +109,16 @@ public function show($project_id, $mr_id)
      * @param string $source
      * @param string $target
      * @param string $title
-     * @param int $assignee
-     * @param int $target_project_id
-     * @param string $description
+     * @param array $extraParams
      * @return mixed
      */
-    public function create($project_id, $source, $target, $title, $assignee = null, $target_project_id = null, $description = null)
+    public function create($project_id, $source, $target, $title, array $extraParams = [])
     {
-        return $this->post($this->getProjectPath($project_id, 'merge_requests'), array(
+        return $this->post($this->getProjectPath($project_id, 'merge_requests'), array_merge([
             'source_branch' => $source,
             'target_branch' => $target,
             'title' => $title,
-            'assignee_id' => $assignee,
-            'target_project_id' => $target_project_id,
-            'description' => $description
-        ));
+        ], $extraParams));
     }
 
     /**

test/Gitlab/Tests/Api/MergeRequestsTest.php

@@ -123,9 +123,6 @@ public function shouldCreateMergeRequestWithoutOptionalParams()
                 'title' => 'Merge Request',
                 'target_branch' => 'master',
                 'source_branch' => 'develop',
-                'description' => null,
-                'assignee_id' => null,
-                'target_project_id' => null
             ))
             ->will($this->returnValue($expectedArray))
         ;
@@ -154,7 +151,13 @@ public function shouldCreateMergeRequestWithOptionalParams()
             ->will($this->returnValue($expectedArray))
         ;
 
-        $this->assertEquals($expectedArray, $api->create(1, 'develop', 'master', 'Merge Request', 6, 20, 'Some changes'));
+        $extraParams = [
+            'assignee_id' => 6,
+            'target_project_id' => 20,
+            'description' => 'Some changes',
+        ];
+
+        $this->assertEquals($expectedArray, $api->create(1, 'develop', 'master', 'Merge Request', $extraParams));
     }
 
     /**