From e8534b42aca7a0788840e4f50358a49066453a67 Mon Sep 17 00:00:00 2001 From: lars Date: Tue, 5 Nov 2024 22:28:11 +0100 Subject: [PATCH 1/2] prevented players from using ac inform command --- .../craftattack/spawn/appliances/acInform/AcInformCommand.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInformCommand.java b/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInformCommand.java index 15707e5..149cd46 100644 --- a/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInformCommand.java +++ b/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInformCommand.java @@ -3,6 +3,7 @@ package eu.mhsl.craftattack.spawn.appliances.acInform; import eu.mhsl.craftattack.spawn.appliance.ApplianceCommand; import org.bukkit.command.Command; import org.bukkit.command.CommandSender; +import org.bukkit.entity.Player; import org.jetbrains.annotations.NotNull; public class AcInformCommand extends ApplianceCommand { @@ -12,7 +13,7 @@ public class AcInformCommand extends ApplianceCommand { @Override protected void execute(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, @NotNull String[] args) throws Exception { -// if(sender instanceof Player) throw new ApplianceCommand.Error("Dieser Command ist nicht für Spieler!"); + if(sender instanceof Player) throw new ApplianceCommand.Error("Dieser Command ist nicht für Spieler!"); getAppliance().processCommand(args); } } From 0a7052b6f5276fe91959f1bd5a4ff9d31fd33998 Mon Sep 17 00:00:00 2001 From: lars Date: Wed, 6 Nov 2024 10:29:20 +0100 Subject: [PATCH 2/2] added error message for missing player name AcInform --- .../eu/mhsl/craftattack/spawn/appliances/acInform/AcInform.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInform.java b/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInform.java index bd976c3..336ad42 100644 --- a/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInform.java +++ b/src/main/java/eu/mhsl/craftattack/spawn/appliances/acInform/AcInform.java @@ -42,7 +42,7 @@ public class AcInform extends Appliance { Component prefix = Component.text("# ", NamedTextColor.DARK_RED); NamedTextColor textColor = NamedTextColor.GRAY; - if(playerName == null || playerName.isBlank()) return; + if(playerName == null || playerName.isBlank()) throw new ApplianceCommand.Error("acinform command needs a player (--playerName)"); if(anticheatName != null && !anticheatName.isBlank()) { component.append(