Change order of query parameters of Twitch Player URLs. (#5326)

This ensures that it doesn't "fake redirect".

Co-authored-by: Rasmus Karlsson <rasmus.karlsson@pajlada.com>

CHANGELOG.md

@@ -11,6 +11,7 @@
 - Bugfix: Fixed links having `http://` added to the beginning in certain cases. (#5323)
 - Bugfix: Fixed topmost windows from losing their status after opening dialogs on Windows. (#5330)
 - Bugfix: Fixed a gap appearing when using filters on `/watching`. (#5329)
+- Dev: Changed the order of the query parameters for Twitch player URLs. (#5326)
 
 ## 2.5.0-beta.1
 

src/common/Common.hpp

@@ -14,6 +14,9 @@
 
 namespace chatterino {
 
+const inline auto TWITCH_PLAYER_URL =
+    QStringLiteral("https://player.twitch.tv/?channel=%1&parent=twitch.tv");
+
 enum class HighlightState {
     None,
     Highlighted,

src/providers/twitch/TwitchChannel.cpp

@@ -84,8 +84,7 @@ TwitchChannel::TwitchChannel(const QString &name)
     , nameOptions{name, name, name}
     , subscriptionUrl_("https://www.twitch.tv/subs/" + name)
     , channelUrl_("https://twitch.tv/" + name)
-    , popoutPlayerUrl_("https://player.twitch.tv/?parent=twitch.tv&channel=" +
+    , popoutPlayerUrl_(TWITCH_PLAYER_URL.arg(name))
-                       name)
     , bttvEmotes_(std::make_shared<EmoteMap>())
     , ffzEmotes_(std::make_shared<EmoteMap>())
     , seventvEmotes_(std::make_shared<EmoteMap>())

src/singletons/Toasts.cpp

@@ -1,6 +1,7 @@
 #include "Toasts.hpp"
 
 #include "Application.hpp"
+#include "common/Common.hpp"
 #include "common/Literals.hpp"
 #include "common/QLogging.hpp"
 #include "common/Version.hpp"
@@ -177,9 +178,8 @@ public:
             case ToastReaction::OpenInPlayer:
                 if (platform_ == Platform::Twitch)
                 {
-                    QDesktopServices::openUrl(QUrl(
+                    QDesktopServices::openUrl(
-                        u"https://player.twitch.tv/?parent=twitch.tv&channel=" %
+                        QUrl(TWITCH_PLAYER_URL.arg(channelName_)));
-                        channelName_));
                 }
                 break;
             case ToastReaction::OpenInStreamlink: {

src/widgets/splits/Split.cpp

@@ -836,8 +836,7 @@ void Split::openChannelInBrowserPlayer(ChannelPtr channel)
     if (auto *twitchChannel = dynamic_cast<TwitchChannel *>(channel.get()))
     {
         QDesktopServices::openUrl(
-            "https://player.twitch.tv/?parent=twitch.tv&channel=" +
+            QUrl(TWITCH_PLAYER_URL.arg(twitchChannel->getName())));
-            twitchChannel->getName());
     }
 }