LogalDeveloper/Snowbrawl
0
Fork 0
Code Issues 4 Actions Releases Activity

Fixed concurrent modification when clearing all boss bars.

Browse Source
This commit is contained in:
Logan Fick
2022-06-04 12:03:58 -04:00
parent 08f689debf
commit 4c9a5b2282
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/main/java/dev/logal/snowbrawl/managers/BossBarManager.java
View File

@@ -18,6 +18,7 @@ import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerQuitEvent; import org.bukkit.event.player.PlayerQuitEvent;
import java.util.HashMap; import java.util.HashMap;
import java.util.List;
/** /**
* Keeps track of boss bars on a per-player basis. * Keeps track of boss bars on a per-player basis.
@@ -78,8 +79,8 @@ public class BossBarManager implements Listener {
* Clears all boss bars owned by this boss bar manager. * Clears all boss bars owned by this boss bar manager.
*/ */
public void clearAllBossBars(){ public void clearAllBossBars(){
// TODO: There was probably a ConcurrentModificationException caused here during a server shut down at one point. final List<Player> players = this.bossBars.keySet().stream().toList(); // Make copy of list of players to avoid concurrent modification.
for (Player player : this.bossBars.keySet()){ for (Player player : players){
this.clearBossBar(player); this.clearBossBar(player);
} }
} }