Core: Remove some warnings here and there

src/server/game/Battlegrounds/ArenaTeam.cpp

@@ -125,7 +125,7 @@ bool ArenaTeam::AddMember(const uint64& playerGuid)
     }
 
     // Check if player is already in a similar arena team
-    if (player && player->GetArenaTeamId(GetSlot()) || Player::GetArenaTeamIdFromDB(playerGuid, GetType()) != 0)
+    if ((player && player->GetArenaTeamId(GetSlot())) || Player::GetArenaTeamIdFromDB(playerGuid, GetType()) != 0)
     {
         sLog->outError("Arena: Player %s (guid: %u) already has an arena team of type %u", playerName.c_str(), GUID_LOPART(playerGuid), GetType());
         return false;

src/server/game/Entities/Player/Player.cpp

@@ -493,6 +493,7 @@ inline void KillRewarder::_RewardXP(Player* player, float rate)
 {
     uint32 xp(_xp);
     if (_group)
+    {
         // 4.2.1. If player is in group, adjust XP:
         //        * set to 0 if player's level is more than maximum level of not gray member;
         //        * cut XP in half if _isFullXP is false.
@@ -503,6 +504,7 @@ inline void KillRewarder::_RewardXP(Player* player, float rate)
                 uint32(xp * rate / 2) + 1;      // Reward only HALF of XP if some of group members are gray.
         else
             xp = 0;
+    }
     if (xp)
     {
         // 4.2.2. Apply auras modifying rewarded XP (SPELL_AURA_MOD_XP_PCT).

src/server/shared/Database/PreparedStatement.cpp

@@ -345,7 +345,7 @@ std::string MySQLPreparedStatement::getQueryString(const char *query)
         switch (m_stmt->statement_data[i].type)
         {
             case TYPE_BOOL:
-            replace << m_stmt->statement_data[i].data.boolean ? "1" : "0";
+                replace << (m_stmt->statement_data[i].data.boolean ? "1" : "0");
                 break;
             case TYPE_UI8:
             case TYPE_UI16: