From f7932b621f5e620545a52339f37715e1571e3bca Mon Sep 17 00:00:00 2001 From: Benjamin Date: Fri, 11 Mar 2022 11:06:33 +0100 Subject: [PATCH] ignore irrelevant players --- .../java/com/benjocraeft/sharehealth/Commands.java | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/main/java/com/benjocraeft/sharehealth/Commands.java b/src/main/java/com/benjocraeft/sharehealth/Commands.java index 91881a7..7ca40ce 100644 --- a/src/main/java/com/benjocraeft/sharehealth/Commands.java +++ b/src/main/java/com/benjocraeft/sharehealth/Commands.java @@ -1,7 +1,6 @@ package com.benjocraeft.sharehealth; import org.bukkit.Bukkit; -import org.bukkit.OfflinePlayer; import org.bukkit.command.Command; import org.bukkit.command.CommandSender; import org.bukkit.command.TabExecutor; @@ -10,7 +9,6 @@ import org.bukkit.util.StringUtil; import java.util.*; import java.util.function.BiConsumer; -import java.util.function.Consumer; public class Commands implements TabExecutor { @@ -78,7 +76,7 @@ public class Commands implements TabExecutor { } - + //This is a mess, no idea how to expand it for 3 part commands @Override public List onTabComplete(CommandSender commandSender, Command command, String s, String[] strings) { List list = new ArrayList<>(); @@ -142,15 +140,19 @@ public class Commands implements TabExecutor { private void commandSetLogging(CommandSender sender, boolean hasLogging){ if (sender instanceof Player){ - Player player = (Player) sender; + if (!Sharehealth.GetPlayers().contains(sender)) + return; - Sharehealth.Instance.onLoggingUpdated(player, hasLogging); - player.sendMessage("Logging settings updated."); + Sharehealth.Instance.onLoggingUpdated((Player) sender, hasLogging); + sender.sendMessage("Logging settings updated."); } } private void commandGetLogging(CommandSender sender){ if (sender instanceof Player){ + if (!Sharehealth.GetPlayers().contains(sender)) + return; + String message = "Logging enabled: " + Sharehealth.Instance.getLogging((Player) sender); sender.sendMessage(message); }