Merge branch '10.0' into 10.1

Author: GrahamCampbell

src/Api/GroupsBoards.php

@@ -7,8 +7,8 @@
 class GroupsBoards extends AbstractApi
 {
     /**
-     * @param int|null $group_id
-     * @param array    $parameters
+     * @param int|string|null $group_id
+     * @param array           $parameters
      *
      * @return mixed
      */

src/HttpClient/Util/QueryStringBuilder.php

@@ -15,19 +15,15 @@ final class QueryStringBuilder
      * Indexed arrays are encoded using empty squared brackets ([]) unlike
      * `http_build_query`.
      *
-     * @param mixed $query
+     * @param array $query
      *
      * @return string
      */
-    public static function build($query)
+    public static function build(array $query)
     {
-        if (!\is_array($query)) {
-            return self::rawurlencode($query);
-        }
-
-        return \implode('&', \array_map(function ($value, $key) {
+        return \sprintf('?%s', \implode('&', \array_map(function ($value, $key) {
             return self::encode($value, $key);
-        }, $query, \array_keys($query)));
+        }, $query, \array_keys($query))));
     }
 
     /**

tests/HttpClient/Util/QueryStringBuilderTest.php

@@ -12,22 +12,16 @@ class QueryStringBuilderTest extends TestCase
     /**
      * @dataProvider queryStringProvider
      *
-     * @param mixed  $query
+     * @param array  $query
      * @param string $expected
      */
-    public function testBuild($query, string $expected): void
+    public function testBuild(array $query, string $expected): void
     {
-        $this->assertSame($expected, QueryStringBuilder::build($query));
+        $this->assertSame(\sprintf('?%s', $expected), QueryStringBuilder::build($query));
     }
 
     public function queryStringProvider()
     {
-        //Scalar value.
-        yield [
-            'a project',
-            'a%20project',
-        ];
-
         //Indexed array.
         yield [
             ['iids' => [88, 86]],