From f8f13b44113325723e3ac4c83c84d53ea8227ffd Mon Sep 17 00:00:00 2001 From: okx-code Date: Wed, 28 Feb 2024 00:51:50 +0000 Subject: [PATCH] fix null player name --- .../vg/civcraft/mc/civmodcore/commands/CommandManager.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/plugins/civmodcore-paper/src/main/java/vg/civcraft/mc/civmodcore/commands/CommandManager.java b/plugins/civmodcore-paper/src/main/java/vg/civcraft/mc/civmodcore/commands/CommandManager.java index 603b920cf..f60edf7ad 100644 --- a/plugins/civmodcore-paper/src/main/java/vg/civcraft/mc/civmodcore/commands/CommandManager.java +++ b/plugins/civmodcore-paper/src/main/java/vg/civcraft/mc/civmodcore/commands/CommandManager.java @@ -59,7 +59,9 @@ public class CommandManager extends BukkitCommandManager { public final void init() { // Prepare our list with player names on init. // Load all known players once on initialization, then use a loginlistener to update the existing name set. - Arrays.stream(Bukkit.getOfflinePlayers()).map(OfflinePlayer::getName).forEach(autocompletePlayerNames::add); + Arrays.stream(Bukkit.getOfflinePlayers()).map(OfflinePlayer::getName) + .filter(Objects::nonNull) + .forEach(autocompletePlayerNames::add); /*TODO this may be better solved with a single global listener, but the implications would've needed some checks. This is pretty cheap and works fast.