Added ability to specify "Query Parameters" for each API call

Author: aik099

src/Jira/Api.php

@@ -766,7 +766,7 @@ public function api(
 	) {
 		$result = $this->client->sendRequest(
 			$method,
-			$url,
+			$this->addQueryParametersToUrl($method, $url, $data),
 			$data,
 			$this->getEndpoint(),
 			$this->authentication,
@@ -799,6 +799,34 @@ public function api(
 		return false;
 	}
 
+	/**
+	 * Adds the query parameters to the URL.
+	 *
+	 * @param string       $method Request method.
+	 * @param string       $url    URL.
+	 * @param array|string $data   Data.
+	 *
+	 * @return string
+	 */
+	protected function addQueryParametersToUrl($method, $url, array &$data)
+	{
+		if ( !array_key_exists('_query', $data) ) {
+			return $url;
+		}
+
+		$query_parameters = $data['_query'];
+		unset($data['_query']);
+
+		// For GET requests all given parameters end up in Query parameters.
+		if ( $method === self::REQUEST_GET ) {
+			$data = $query_parameters + $data;
+
+			return $url;
+		}
+
+		return $url . (strpos($url, '?') === false ? '?' : ':') . http_build_query($query_parameters);
+	}
+
 	/**
 	 * Downloads attachment.
 	 *

tests/Jira/ApiTest.php

@@ -495,6 +495,74 @@ public function testDeleteWorkLogWithCustomParams()
 		$this->assertEquals(json_decode($response, true), $actual, 'The response is json-decoded.');
 	}
 
+	public function testQueryParametersHandlingForGetRequestMethod()
+	{
+		$this->expectClientCall(
+			Api::REQUEST_GET,
+			'/rest/api/2/something',
+			array(
+				'q_p1' => 'q_p1_v',
+				'q_p2' => 'q_p2_v',
+				'rb_p1' => 'rb_p1_v',
+				'rb_p2' => 'rb_p2_v',
+				'rb_p3' => 'rb_p3_v',
+			),
+			'{}'
+		);
+
+		$this->api->api(
+			Api::REQUEST_GET,
+			'/rest/api/2/something',
+			array(
+				'_query' => array('q_p1' => 'q_p1_v', 'q_p2' => 'q_p2_v'),
+				'rb_p1' => 'rb_p1_v',
+				'rb_p2' => 'rb_p2_v',
+				'rb_p3' => 'rb_p3_v',
+			),
+			true
+		);
+	}
+
+	/**
+	 * @dataProvider queryParametersHandlingForOtherRequestMethodsDataProvider
+	 *
+	 * @param string $request_method Request method.
+	 */
+	public function testQueryParametersHandlingForOtherRequestMethods($request_method)
+	{
+		$this->expectClientCall(
+			$request_method,
+			'/rest/api/2/something?q_p1=q_p1_v&q_p2=q_p2_v',
+			array(
+				'rb_p1' => 'rb_p1_v',
+				'rb_p2' => 'rb_p2_v',
+				'rb_p3' => 'rb_p3_v',
+			),
+			'{}'
+		);
+
+		$this->api->api(
+			$request_method,
+			'/rest/api/2/something',
+			array(
+				'_query' => array('q_p1' => 'q_p1_v', 'q_p2' => 'q_p2_v'),
+				'rb_p1' => 'rb_p1_v',
+				'rb_p2' => 'rb_p2_v',
+				'rb_p3' => 'rb_p3_v',
+			),
+			true
+		);
+	}
+
+	public static function queryParametersHandlingForOtherRequestMethodsDataProvider()
+	{
+		return array(
+			'delete' => array(Api::REQUEST_DELETE),
+			'post' => array(Api::REQUEST_POST),
+			'put' => array(Api::REQUEST_PUT),
+		);
+	}
+
 	/**
 	 * Expects a particular client call.
 	 *