diff --git a/src/dev/w1zzrd/invtweaks/command/MagnetCommandExecutor.java b/src/dev/w1zzrd/invtweaks/command/MagnetCommandExecutor.java index bed5f37..bf18fba 100644 --- a/src/dev/w1zzrd/invtweaks/command/MagnetCommandExecutor.java +++ b/src/dev/w1zzrd/invtweaks/command/MagnetCommandExecutor.java @@ -48,7 +48,7 @@ public class MagnetCommandExecutor extends ConfigurableCommandExecutor it.getKey().toString().equals(args[0])).findFirst().orElse(null); + assertTrue((targetMaterial = getMaterialMatching(args[0])) != null, String.format(ERR_UNKNOWN, args[0]), sender) + ) return true; assert targetMaterial != null; assert sender instanceof Player; @@ -64,7 +63,7 @@ public class SearchCommandExecutor extends ConfigurableCommandExecutor it.getKey().toString().equals(stringKey)).findFirst().orElse(null); + } } diff --git a/src/dev/w1zzrd/invtweaks/command/SortCommandExecutor.java b/src/dev/w1zzrd/invtweaks/command/SortCommandExecutor.java index 8546614..88f577b 100644 --- a/src/dev/w1zzrd/invtweaks/command/SortCommandExecutor.java +++ b/src/dev/w1zzrd/invtweaks/command/SortCommandExecutor.java @@ -20,6 +20,7 @@ import java.util.logging.Logger; import static dev.w1zzrd.invtweaks.InvTweaksPlugin.LOG_PLUGIN_NAME; import static org.bukkit.Material.*; +import static dev.w1zzrd.invtweaks.command.CommandUtils.*; /** * Handler for executions of /sort command @@ -32,12 +33,10 @@ public class SortCommandExecutor implements CommandExecutor { public boolean onCommand(CommandSender sender, Command command, String label, String[] args) { // Since we rely on targeting an inventory either by what we're looking at or by whom it was called, // there is an implicit dependency on that the command is called by (at the very least) an entity - if (!(sender instanceof Player)) { - logger.info(LOG_PLUGIN_NAME + " Sort command triggered by non-player"); - sender.sendMessage("Command must be run by a player"); - return false; - } + if (assertTrue(sender instanceof Player, "Command must be run by a player", sender)) + return true; + assert sender instanceof Player; final Player player = (Player) sender; logger.fine(LOG_PLUGIN_NAME + " Sort triggered by player " + player.getName());