From d4eccf478f8a5542e7012e2480bd87306068aafa Mon Sep 17 00:00:00 2001
From: Teriuihi <teriuihi@alttd.com>
Date: Sun, 20 Oct 2024 19:42:31 +0200
Subject: [PATCH] Add check for existing viewers before adding them

Updated the addViewer method to verify if a player is already in the viewers list before adding. Adjusted the order of operations to ensure the scoreboard is updated correctly before adding the player to the viewers.
---
 .../java/com/alttd/halloween/leaderboard/Leaderboard.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/main/java/com/alttd/halloween/leaderboard/Leaderboard.java b/src/main/java/com/alttd/halloween/leaderboard/Leaderboard.java
index f7b4b73..95aacb8 100644
--- a/src/main/java/com/alttd/halloween/leaderboard/Leaderboard.java
+++ b/src/main/java/com/alttd/halloween/leaderboard/Leaderboard.java
@@ -88,11 +88,13 @@ public class Leaderboard {
     }
 
     public void addViewer(Player player) {
-        if (viewers.put(player.getUniqueId(), player) == null) {
+        if (viewers.containsKey(player.getUniqueId())) {
             return; //Player was already in viewers list
         }
         protocolManager.sendServerPacket(player, title.entitySpawnPacket());
         lines.forEach(line -> protocolManager.sendServerPacket(player, line.entitySpawnPacket()));
+        updateScoreboard(player);
+        viewers.put(player.getUniqueId(), player);
     }
 
     public void removeViewer(Player player) {