HungerGameSSS

[4.1] — Bug-Fix Patch 1

Fixed

Critical

• C-03 · arena.max-players default value (config.yml) Changed the default from 1000 to 24. The previous value caused spawn-circle radii of ~1500 blocks, placing cage pistons in unloaded chunks and silently failing block placement — players were teleported into the void at match start.

• C-02 · ArenaStartEvent fired twice per match (MatchService.java) Removed the premature callEvent(new ArenaStartEvent(arena)) call inside startMatch(). The event is now fired exactly once, inside releasePlayers() when the state transitions to PLAYING and the game actually begins. Previously, any listener hooked to ArenaStartEvent (scoreboards, rewards, third-party integrations) would execute twice.

• C-05 · DeathNote — non-thread-safe HashMap in passive task (DeathNote.java) Replaced HashMap with ConcurrentHashMap for both progressMap and targetMap. These maps are written by the 1-tick passive BukkitRunnable and concurrently removed by onQuit / onDeath event callbacks.

• C-04 · Arena.spectators — non-thread-safe ArrayList (Arena.java) Changed to Collections.synchronizedList(new ArrayList<>()). The spectator list is modified on the main thread (handleDeath) and iterated from async contexts (checkWin, scoreboard updates).

High

• H-05 · DatabaseManager.loadArena() did not mark players as alive (DatabaseManager.java) After loading player UUIDs on server reload, GamePlayer.setAlive(true) is now called for each loaded player. Without this, checkWin() saw zero alive players immediately after reload and ended the match in the same tick.

• H-02 · ShadowBlade.revealPlayer() showed vanished player to entire server (ShadowBlade.java) revealPlayer() now iterates only the players in the same arena instead of calling getOnlinePlayers() server-wide. In multi-arena setups, players in other arenas would receive a spurious showPlayer() call. Both armorMap and vanishedPlayers upgraded from HashMap to ConcurrentHashMap to eliminate race conditions between the timed-reveal BukkitRunnable, onDeath, and onQuit callbacks.

Medium

• M-07 · Wrong permission node in BlockBreakListener (BlockBreakListener.java) Changed hg.admin to hoplitebr.admin to match the permission declared in plugin.yml. Admins with hoplitebr.admin could not break blocks inside an arena during WAITING / STARTING / ENDING phases.