fix: register who has a hotbar enabled inside directly from the HotbarItem

Signed-off-by: Cristóbal Veas <[email protected]>

Author: zetastormy

src/main/java/team/devblook/akropolis/module/modules/hotbar/HotbarItem.java

@@ -121,6 +121,7 @@ public void giveItem(Player player) {
         }
 
         player.getInventory().setItem(slot, newItem);
+        hotbarManager.getPlayers().add(player.getUniqueId());
     }
 
     public void removeItem(Player player) {
@@ -135,6 +136,8 @@ public void removeItem(Player player) {
         if (keyValueInItem != null && keyValueInItem.equals(keyValue)) {
             inventory.remove(itemInSlot);
         }
+
+        hotbarManager.getPlayers().remove(player.getUniqueId());
     }
 
     @EventHandler

src/main/java/team/devblook/akropolis/module/modules/hotbar/HotbarManager.java

@@ -129,14 +129,12 @@ public void giveItemsToPlayer(Player player) {
         if (inDisabledWorld(player.getLocation())) return;
 
         hotbarItems.forEach(hotbarItem -> hotbarItem.giveItem(player));
-        players.add(player.getUniqueId());
     }
 
     public void removeItemsFromPlayer(Player player) {
         if (inDisabledWorld(player.getLocation())) return;
 
         hotbarItems.forEach(hotbarItem -> hotbarItem.removeItem(player));
-        players.remove(player.getUniqueId());
     }
 
     public boolean hasHotbar(UUID playerUuid) {
@@ -146,4 +144,8 @@ public boolean hasHotbar(UUID playerUuid) {
     public List<HotbarItem> getHotbarItems() {
         return hotbarItems;
     }
+
+    public Set<UUID> getPlayers() {
+        return players;
+    }
 }