Switch to JSON form submission (#22)

* switch from query params to json form body

* update guzzle

* update version

Author: kylekz

README.md

@@ -11,7 +11,7 @@ Refer to the table for PHP version compatibility:
 
 | ChallongePHP Ver. | Compatible PHP |
 |----------|-------------|
-| ^4.0 | 8.0 -8.1 |
+| ^4.0 | 8.0 - 8.1 |
 | ^3.0 | 7.4 - 8.0 |
 | ^2.1 | 7.4 |
 | ^2.0 | 7.4 |
@@ -68,6 +68,6 @@ As the package is fully type-hinted, everything should be self documenting, howe
 
 ## Contact
 - [@Reflexgg](http://twitter.com/Reflexgg)
-- [@Kairuxo](http://twitter.com/Kairuxo)
+- [@rfxkairu](http://twitter.com/rfxkairu)
 
 [Challonge]: <http://api.challonge.com/v1>

composer.lock

@@ -14,7 +14,7 @@ class Challonge
      * ChallongePHP version.
      * Required to pass into Challonge.
      */
-    protected string $version = '3.1';
+    protected string $version = '4.0';
 
     /**
      * PSR-18 compatible HTTP client wrapped in our wrapper.

src/Challonge/Challonge.php

@@ -59,7 +59,7 @@ public function request(string $method, string $uri, array $content = []): array
             $method,
             $base_uri,
             $this->buildHeaders(),
-            \http_build_query($content, '', '&'),
+            json_encode($content),
         );
         $response = $this->client->sendRequest($request);
 
@@ -74,7 +74,7 @@ protected function buildHeaders(): array
     {
         return [
             'Accept' => 'application/json',
-            'Content-Type' => 'application/x-www-form-urlencoded',
+            'Content-Type' => 'application/json',
             'User-Agent' => "ChallongePHP/{$this->version} ChallongePHP (https://github.com/teamreflex/ChallongePHP, {$this->version})",
         ];
     }
@@ -153,6 +153,22 @@ public function setKey(string $key): void
         $this->key = $key;
     }
 
+    /**
+     * @return bool
+     */
+    public function getMapOptions(): bool
+    {
+        return $this->mapOptions;
+    }
+
+    /**
+     * @param bool $mapOptions
+     */
+    public function setMapOptions(bool $mapOptions): void
+    {
+        $this->mapOptions = $mapOptions;
+    }
+
     /**
      * Get the underlying client.
      * @return ClientInterface

src/Challonge/ClientWrapper.php

@@ -297,7 +297,14 @@ public function addParticipant(array $options = []): Participant
      */
     public function bulkAddParticipant(array $options = []): Collection
     {
-        $response = $this->client->request('POST', "tournaments/{$this->id}/participants/bulk_add", $this->client->mapOptions($options, 'participant'));
+        // have to bypass the mapOptions function as the format is a bit different
+        if ($this->client->getMapOptions()) {
+            $options = [
+                'participants' => $options,
+            ];
+        }
+
+        $response = $this->client->request('POST', "tournaments/{$this->id}/participants/bulk_add", $options);
         return Collection::make($response)
             ->map(fn (array $participant) => Participant::fromResponse($this->client, $participant['participant']));
     }

src/Challonge/DTO/Tournament.php

@@ -192,7 +192,10 @@ public function test_tournament_bulkadd_participant(): void
             json_decode(file_get_contents(__DIR__ . '/stubs/tournament_fetch.json'), true)['tournament']
         );
 
-        $response = $tournament->bulkAddParticipant();
+        $response = $tournament->bulkAddParticipant([
+            ['name' => 'Team 1'],
+            ['name' => 'Team 2'],
+        ]);
 
         $this->assertCount(3, $response);
     }