Fix showing group name in online list for legacy server.

2 files changed, 16 insertions, 9 deletions

diff --git a/src/gui/onlineplayer.h b/src/gui/onlineplayer.h
index 219a473da..a97c2104e 100644
--- a/src/gui/onlineplayer.h
+++ b/src/gui/onlineplayer.h
@@ -36,12 +36,12 @@ class OnlinePlayer final
                      const signed char level,
                      const GenderT gender,
                      const signed char version,
-                     const unsigned char group) :
+                     const int group) :
             mNick(nick),
             mText(),
+            mGroup(group),
             mStatus(status),
             mLevel(level),
-            mGroup(group),
             mVersion(version),
             mGender(gender),
             mIsGM(false)
@@ -78,12 +78,12 @@ class OnlinePlayer final
 
         std::string mText;
 
+        int mGroup;
+
         unsigned char mStatus;
 
         signed char mLevel;
 
-        unsigned char mGroup;
-
         signed char mVersion;
 
         GenderT mGender;
diff --git a/src/gui/windows/whoisonline.cpp b/src/gui/windows/whoisonline.cpp
index bc16397ed..61e2bc21a 100644
--- a/src/gui/windows/whoisonline.cpp
+++ b/src/gui/windows/whoisonline.cpp
@@ -55,7 +55,9 @@
 #include "net/playerhandler.h"
 #include "net/serverfeatures.h"
 
+#ifdef TMWA_SUPPORT
 #include "resources/db/groupdb.h"
+#endif  // TMWA_SUPPORT
 
 #include "utils/foreach.h"
 #include "utils/gettext.h"
@@ -473,7 +475,7 @@ void WhoIsOnline::loadWebList()
 
                 OnlinePlayer *const player = new OnlinePlayer(nick,
                     CAST_U8(255), level,
-                    Gender::UNSPECIFIED, -1, 0);
+                    Gender::UNSPECIFIED, -1, -1);
                 mOnlinePlayers.insert(player);
                 mOnlineNicks.insert(nick);
 
@@ -872,14 +874,17 @@ void OnlinePlayer::setText(std::string color)
         }
     }
 
-    if (GroupDb::getShowBadge(mGroup))
+#ifdef TMWA_SUPPORT
+    if (mGroup != -1 && GroupDb::getShowBadge(mGroup))
     {
         const std::string name = GroupDb::getName(mGroup);
         mText.append(strprintf("(%s) ", name.c_str()));
     }
-    else if ((mStatus != 255 && ((mStatus & BeingFlag::GM) != 0)) || mIsGM)
+    else
+#endif
     {
-        mText.append("(GM) ");
+        if ((mStatus != 255 && ((mStatus & BeingFlag::GM) != 0)) || mIsGM)
+            mText.append("(GM) ");
     }
 
     if (mLevel > 0)
@@ -890,8 +895,10 @@ void OnlinePlayer::setText(std::string color)
     else if (mGender == Gender::MALE)
         mText.append("\u2642");
 
-    if (GroupDb::getHighlightName(mGroup) && color == "0")
+#ifdef TMWA_SUPPORT
+    if (mGroup != -1 && GroupDb::getHighlightName(mGroup) && color == "0")
         color = "2";
+#endif
 
     if (mStatus > 0 && mStatus != 255)
     {