Bugfix: Use query params for Channel::getMessages instead of json body

dc-Ragnarok · Jul 21, 2024 · 391b19b · 391b19b
1 parent 2ede7cd
commit 391b19b
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/src/Rest/Channel.php b/src/Rest/Channel.php
@@ -100,14 +100,18 @@ public function getMessages(
         string $channelId,
         GetMessagesBuilder $getMessagesBuilder = new GetMessagesBuilder()
     ): ExtendedPromiseInterface {
+        $endpoint = Endpoint::bind(
+            Endpoint::CHANNEL_MESSAGES,
+            $channelId
+        );
+
+        $queryParams = $getMessagesBuilder->get();
+        foreach ($queryParams as $key => $value) {
+            $endpoint->addQuery($key, $value);
+        }
+
         return $this->mapArrayPromise(
-            $this->http->get(
-                Endpoint::bind(
-                    Endpoint::CHANNEL_MESSAGES,
-                    $channelId
-                ),
-                $getMessagesBuilder->get()
-            ),
+            $this->http->get($endpoint),
             Message::class
         )->otherwise($this->logThrowable(...));
     }