diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/TotalFreedomMod.java b/src/main/java/me/StevenLawson/TotalFreedomMod/TotalFreedomMod.java index 57f32ed..9d3b055 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/TotalFreedomMod.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/TotalFreedomMod.java @@ -5,10 +5,9 @@ import me.StevenLawson.TotalFreedomMod.admin.AdminList; import me.StevenLawson.TotalFreedomMod.announcer.Announcer; import me.StevenLawson.TotalFreedomMod.ban.BanManager; import me.StevenLawson.TotalFreedomMod.ban.PermbanList; +import me.StevenLawson.TotalFreedomMod.commands.Commands; import me.StevenLawson.TotalFreedomMod.discord.bridge.DiscordBridge; import me.StevenLawson.TotalFreedomMod.command.CommandBlocker; -import me.StevenLawson.TotalFreedomMod.commands.CommandHandler; -import me.StevenLawson.TotalFreedomMod.commands.CommandLoader; import me.StevenLawson.TotalFreedomMod.commands.FreedomCommand; import me.StevenLawson.TotalFreedomMod.config.ConfigurationEntry; import me.StevenLawson.TotalFreedomMod.deprecated.bridge.BukkitTelnetBridge; @@ -23,7 +22,6 @@ import me.StevenLawson.TotalFreedomMod.world.AdminWorld; import me.StevenLawson.TotalFreedomMod.world.FlatlandsWorld; import me.StevenLawson.TotalFreedomMod.world.ProtectedArea; import org.bukkit.World; -import org.bukkit.command.CommandSender; import org.bukkit.configuration.InvalidConfigurationException; import org.bukkit.configuration.file.FileConfiguration; import org.bukkit.configuration.file.YamlConfiguration; @@ -176,6 +174,8 @@ public class TotalFreedomMod extends JavaPlugin { // Heartbeat new HeartBeat(plugin).runTaskTimer(plugin, HEARTBEAT_RATE * 20L, HEARTBEAT_RATE * 20L); + Commands.registerCommands(this); + // Start services HTTPDManager.start(); @@ -200,7 +200,6 @@ public class TotalFreedomMod extends JavaPlugin { @Override public void run() { - CommandLoader.scan(); CommandBlocker.load(); // Add spawnpoints later - https://github.com/TotalFreedom/TotalFreedomMod/issues/438 @@ -222,12 +221,6 @@ public class TotalFreedomMod extends JavaPlugin { Log.info("Plugin disabled"); } - @Override - public boolean onCommand(CommandSender sender, org.bukkit.command.Command cmd, String commandLabel, String[] args) - { - return CommandHandler.handleCommand(sender, cmd, commandLabel, args); - } - public FileConfiguration getCustomConfig() { return this.customConfig; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/command/CommandBlocker.java b/src/main/java/me/StevenLawson/TotalFreedomMod/command/CommandBlocker.java index db138b3..65f82c1 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/command/CommandBlocker.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/command/CommandBlocker.java @@ -2,10 +2,10 @@ package me.StevenLawson.TotalFreedomMod.command; import me.StevenLawson.TotalFreedomMod.Log; import me.StevenLawson.TotalFreedomMod.admin.AdminList; -import me.StevenLawson.TotalFreedomMod.commands.CommandLoader; import me.StevenLawson.TotalFreedomMod.config.ConfigurationEntry; import me.StevenLawson.TotalFreedomMod.util.Utilities; import org.apache.commons.lang3.StringUtils; +import org.bukkit.Bukkit; import org.bukkit.ChatColor; import org.bukkit.command.Command; import org.bukkit.command.CommandMap; @@ -34,7 +34,7 @@ public class CommandBlocker { BLOCKED_COMMANDS.clear(); - final CommandMap commandMap = CommandLoader.getCommandMap(); + final CommandMap commandMap = Utilities.getField(Bukkit.getServer().getPluginManager(), "commandMap"); if (commandMap == null) { Log.severe("Error loading commandMap."); diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandHandler.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandHandler.java deleted file mode 100644 index e389eb8..0000000 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandHandler.java +++ /dev/null @@ -1,78 +0,0 @@ -package me.StevenLawson.TotalFreedomMod.commands; - -import me.StevenLawson.TotalFreedomMod.Log; -import me.StevenLawson.TotalFreedomMod.TotalFreedomMod; -import org.apache.commons.lang3.StringUtils; -import org.bukkit.ChatColor; -import org.bukkit.command.Command; -import org.bukkit.command.CommandSender; -import org.bukkit.entity.Player; - -public class CommandHandler { - public static final String COMMAND_PATH = FreedomCommand.class.getPackage().getName(); // "me.StevenLawson.TotalFreedomMod.Commands"; - public static final String COMMAND_PREFIX = "Command_"; - - public static boolean handleCommand(CommandSender sender, Command cmd, String commandLabel, String[] args) { - final Player playerSender; - final boolean senderIsConsole; - - if (sender instanceof Player) { - senderIsConsole = false; - playerSender = (Player) sender; - - Log.info(String.format("[PLAYER_COMMAND] %s (%s): /%s %s", - playerSender.getName(), - ChatColor.stripColor(playerSender.getDisplayName()), - commandLabel, - StringUtils.join(args, " ")), true); - } - else - { - senderIsConsole = true; - playerSender = null; - - Log.info(String.format("[CONSOLE_COMMAND] %s: /%s %s", - sender.getName(), - commandLabel, - StringUtils.join(args, " ")), true); - } - - final FreedomCommand dispatcher; - try - { - final ClassLoader classLoader = TotalFreedomMod.class.getClassLoader(); - dispatcher = (FreedomCommand) classLoader.loadClass(String.format("%s.%s%s", - COMMAND_PATH, - COMMAND_PREFIX, - cmd.getName().toLowerCase())).newInstance(); - dispatcher.setup(TotalFreedomMod.plugin, sender, dispatcher.getClass()); - } - catch (Exception ex) - { - Log.severe("Could not load command: " + cmd.getName()); - Log.severe(ex); - - sender.sendMessage(ChatColor.RED + "Command Error! Could not load command: " + cmd.getName()); - return true; - } - - if (!dispatcher.senderHasPermission()) - { - sender.sendMessage(FreedomCommand.MSG_NO_PERMS); - return true; - } - - try - { - return dispatcher.run(sender, playerSender, cmd, commandLabel, args, senderIsConsole); - } - catch (Exception ex) - { - Log.severe("Command Error: " + commandLabel); - Log.severe(ex); - sender.sendMessage(ChatColor.RED + "Command Error: " + ex.getMessage()); - } - - return true; - } -} diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandLoader.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandLoader.java deleted file mode 100644 index 7e75899..0000000 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandLoader.java +++ /dev/null @@ -1,324 +0,0 @@ -package me.StevenLawson.TotalFreedomMod.commands; - -import me.StevenLawson.TotalFreedomMod.Log; -import me.StevenLawson.TotalFreedomMod.TotalFreedomMod; -import me.StevenLawson.TotalFreedomMod.util.Utilities; -import org.bukkit.Bukkit; -import org.bukkit.command.*; -import org.bukkit.plugin.Plugin; - -import java.io.IOException; -import java.security.CodeSource; -import java.util.*; -import java.util.regex.Matcher; -import java.util.regex.Pattern; -import java.util.zip.ZipEntry; -import java.util.zip.ZipInputStream; - -public class CommandLoader { - public static final Pattern COMMAND_PATTERN; - private static final List COMMAND_LIST; - - static { - COMMAND_PATTERN = Pattern.compile(CommandHandler.COMMAND_PATH.replace('.', '/') + "/(" + CommandHandler.COMMAND_PREFIX + "[^\\$]+)\\.class"); - COMMAND_LIST = new ArrayList(); - } - - private CommandLoader() { - throw new AssertionError(); - } - - public static void scan() - { - CommandMap commandMap = getCommandMap(); - if (commandMap == null) - { - Log.severe("Error loading commandMap."); - return; - } - COMMAND_LIST.clear(); - COMMAND_LIST.addAll(getCommands()); - - for (TFM_CommandInfo commandInfo : COMMAND_LIST) - { - TFM_DynamicCommand dynamicCommand = new TFM_DynamicCommand(commandInfo); - - Command existing = commandMap.getCommand(dynamicCommand.getName()); - if (existing != null) - { - unregisterCommand(existing, commandMap); - } - - commandMap.register(TotalFreedomMod.plugin.getDescription().getName(), dynamicCommand); - } - - Log.info("TFM commands loaded."); - } - - public static void unregisterCommand(String commandName) - { - CommandMap commandMap = getCommandMap(); - if (commandMap != null) - { - Command command = commandMap.getCommand(commandName.toLowerCase()); - if (command != null) - { - unregisterCommand(command, commandMap); - } - } - } - - public static void unregisterCommand(Command command, CommandMap commandMap) - { - try - { - command.unregister(commandMap); - HashMap knownCommands = getKnownCommands(commandMap); - if (knownCommands != null) - { - knownCommands.remove(command.getName()); - for (String alias : command.getAliases()) - { - knownCommands.remove(alias); - } - } - } - catch (Exception ex) - { - Log.severe(ex); - } - } - - @SuppressWarnings("unchecked") - public static CommandMap getCommandMap() - { - final Object commandMap = Utilities.getField(Bukkit.getServer().getPluginManager(), "commandMap"); - if (commandMap != null) - { - if (commandMap instanceof CommandMap) - { - return (CommandMap) commandMap; - } - } - return null; - } - - @SuppressWarnings("unchecked") - public static HashMap getKnownCommands(CommandMap commandMap) - { - Object knownCommands = Utilities.getField(commandMap, "knownCommands"); - if (knownCommands != null) - { - if (knownCommands instanceof HashMap) - { - return (HashMap) knownCommands; - } - } - return null; - } - - private static List getCommands() - { - List commandList = new ArrayList(); - - try - { - CodeSource codeSource = TotalFreedomMod.class.getProtectionDomain().getCodeSource(); - if (codeSource != null) - { - ZipInputStream zip = new ZipInputStream(codeSource.getLocation().openStream()); - ZipEntry zipEntry; - while ((zipEntry = zip.getNextEntry()) != null) - { - String entryName = zipEntry.getName(); - Matcher matcher = COMMAND_PATTERN.matcher(entryName); - if (matcher.find()) - { - try - { - Class commandClass = Class.forName(CommandHandler.COMMAND_PATH + "." + matcher.group(1)); - - CommandPermissions commandPermissions = commandClass.getAnnotation(CommandPermissions.class); - CommandParameters commandParameters = commandClass.getAnnotation(CommandParameters.class); - - if (commandPermissions != null && commandParameters != null) - { - TFM_CommandInfo commandInfo = new TFM_CommandInfo( - commandClass, - matcher.group(1).split("_")[1], - commandPermissions.level(), - commandPermissions.source(), - commandPermissions.blockHostConsole(), - commandParameters.description(), - commandParameters.usage(), - commandParameters.aliases()); - - commandList.add(commandInfo); - } - } - catch (ClassNotFoundException ex) - { - Log.severe(ex); - } - } - } - } - } - catch (IOException ex) - { - Log.severe(ex); - } - - return commandList; - } - - public static class TFM_CommandInfo - { - private final String commandName; - private final Class commandClass; - private final AdminLevel level; - private final SourceType source; - private final boolean blockHostConsole; - private final String description; - private final String usage; - private final List aliases; - - public TFM_CommandInfo(Class commandClass, String commandName, AdminLevel level, SourceType source, boolean blockHostConsole, String description, String usage, String aliases) { - this.commandName = commandName; - this.commandClass = commandClass; - this.level = level; - this.source = source; - this.blockHostConsole = blockHostConsole; - this.description = description; - this.usage = usage; - this.aliases = ("".equals(aliases) ? new ArrayList() : Arrays.asList(aliases.split(","))); - } - - public List getAliases() - { - return Collections.unmodifiableList(aliases); - } - - public Class getCommandClass() - { - return commandClass; - } - - public String getCommandName() - { - return commandName; - } - - public String getDescription() - { - return description; - } - - public String getDescriptionPermissioned() - { - String _description = description; - - switch (this.getLevel()) - { - case SENIOR: - _description = "Senior " + (this.getSource() == SourceType.ONLY_CONSOLE ? "Console" : "") + " Command - " + _description; - break; - case SUPER: - _description = "Superadmin Command - " + _description; - break; - case OP: - _description = "OP Command - " + _description; - break; - } - - return _description; - } - - public AdminLevel getLevel() { - return level; - } - - public SourceType getSource() - { - return source; - } - - public String getUsage() - { - return usage; - } - - public boolean getBlockHostConsole() - { - return blockHostConsole; - } - - @Override - public String toString() - { - StringBuilder sb = new StringBuilder(); - sb.append("commandName: ").append(commandName); - sb.append("\ncommandClass: ").append(commandClass.getName()); - sb.append("\nlevel: ").append(level); - sb.append("\nsource: ").append(source); - sb.append("\nblock_host_console: ").append(blockHostConsole); - sb.append("\ndescription: ").append(description); - sb.append("\nusage: ").append(usage); - sb.append("\naliases: ").append(aliases); - return sb.toString(); - } - } - - public static class TFM_DynamicCommand extends Command implements PluginIdentifiableCommand - { - private final TFM_CommandInfo commandInfo; - - private TFM_DynamicCommand(TFM_CommandInfo commandInfo) - { - super(commandInfo.getCommandName(), commandInfo.getDescriptionPermissioned(), commandInfo.getUsage(), commandInfo.getAliases()); - - this.commandInfo = commandInfo; - } - - @Override - public boolean execute(CommandSender sender, String commandLabel, String[] args) - { - boolean success = false; - - if (!getPlugin().isEnabled()) - { - return false; - } - - try - { - success = getPlugin().onCommand(sender, this, commandLabel, args); - } - catch (Throwable ex) - { - throw new CommandException("Unhandled exception executing command '" + commandLabel + "' in plugin " + getPlugin().getDescription().getFullName(), ex); - } - - if (!success && getUsage().length() > 0) - { - for (String line : getUsage().replace("", commandLabel).split("\n")) - { - sender.sendMessage(line); - } - } - - return success; - } - - @Override - public Plugin getPlugin() - { - return TotalFreedomMod.plugin; - } - - public TFM_CommandInfo getCommandInfo() - { - return commandInfo; - } - } -} diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandParameters.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandParameters.java index d943d0e..3ce6cbf 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandParameters.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/CommandParameters.java @@ -1,14 +1,10 @@ package me.StevenLawson.TotalFreedomMod.commands; -import java.lang.annotation.Retention; -import java.lang.annotation.RetentionPolicy; +// This is only here because of Command_tfm -@Retention(RetentionPolicy.RUNTIME) public @interface CommandParameters { String description(); String usage(); - - String aliases() default ""; // "alias1,alias2,alias3" - no spaces } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminmode.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminmode.java index 7c75dae..467322e 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminmode.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminmode.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH, blockHostConsole = true) -@CommandParameters(description = "Close server to non-superadmins.", usage = "/ [on | off]") public class Command_adminmode extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminworld.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminworld.java index 8e5a6df..8a24564 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminworld.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_adminworld.java @@ -9,7 +9,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.OP, source = SourceType.BOTH) -@CommandParameters(description = "Go to the AdminWorld.", usage = "/ [guest < list | purge | add | remove > | time | weather ]") public class Command_adminworld extends FreedomCommand { private enum CommandMode { TELEPORT, GUEST, TIME, WEATHER @@ -63,19 +62,19 @@ public class Command_adminworld extends FreedomCommand { if (adminWorld == null || sender_p.getWorld() == adminWorld) { - playerMsg("Going to the main world."); + playerMsg(sender, "Going to the main world."); sender_p.teleport(server.getWorlds().get(0).getSpawnLocation()); } else { if (AdminWorld.getInstance().canAccessWorld(sender_p)) { - playerMsg("Going to the AdminWorld."); + playerMsg(sender, "Going to the AdminWorld."); AdminWorld.getInstance().sendToWorld(sender_p); } else { - playerMsg("You don't have permission to access the AdminWorld."); + playerMsg(sender, "You don't have permission to access the AdminWorld."); } } @@ -87,7 +86,7 @@ public class Command_adminworld extends FreedomCommand { { if ("list".equalsIgnoreCase(args[1])) { - playerMsg("AdminWorld guest list: " + AdminWorld.getInstance().guestListToString()); + playerMsg(sender, "AdminWorld guest list: " + AdminWorld.getInstance().guestListToString()); } else if ("purge".equalsIgnoreCase(args[1])) { @@ -120,7 +119,7 @@ public class Command_adminworld extends FreedomCommand { } else { - playerMsg("Could not add player to guest list."); + playerMsg(sender, "Could not add player to guest list."); } } else if ("remove".equals(args[1])) @@ -132,7 +131,7 @@ public class Command_adminworld extends FreedomCommand { } else { - playerMsg("Can't find guest entry for: " + args[2]); + playerMsg(sender, "Can't find guest entry for: " + args[2]); } } else @@ -153,11 +152,11 @@ public class Command_adminworld extends FreedomCommand { if (timeOfDay != null) { AdminWorld.getInstance().setTimeOfDay(timeOfDay); - playerMsg("AdminWorld time set to: " + timeOfDay.name()); + playerMsg(sender, "AdminWorld time set to: " + timeOfDay.name()); } else { - playerMsg("Invalid time of day. Can be: sunrise, noon, sunset, midnight"); + playerMsg(sender, "Invalid time of day. Can be: sunrise, noon, sunset, midnight"); } } else @@ -177,11 +176,11 @@ public class Command_adminworld extends FreedomCommand { if (weatherMode != null) { AdminWorld.getInstance().setWeatherMode(weatherMode); - playerMsg("AdminWorld weather set to: " + weatherMode.name()); + playerMsg(sender, "AdminWorld weather set to: " + weatherMode.name()); } else { - playerMsg("Invalid weather mode. Can be: off, rain, storm"); + playerMsg(sender, "Invalid weather mode. Can be: off, rain, storm"); } } else diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_blockcmd.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_blockcmd.java index 26c3475..3e796cc 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_blockcmd.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_blockcmd.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Block all commands for a specific player.", usage = "/ >", aliases = "blockcommands,blockcommand") public class Command_blockcmd extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -28,7 +27,7 @@ public class Command_blockcmd extends FreedomCommand { playerdata.setCommandsBlocked(false); } } - playerMsg("Unblocked commands for " + counter + " players."); + playerMsg(sender, "Unblocked commands for " + counter + " players."); return true; } @@ -36,13 +35,13 @@ public class Command_blockcmd extends FreedomCommand { if (player == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND); return true; } if (AdminList.isSuperAdmin(player)) { - playerMsg(player.getName() + " is a Superadmin, and cannot have their commands blocked."); + playerMsg(sender, player.getName() + " is a Superadmin, and cannot have their commands blocked."); return true; } @@ -51,7 +50,7 @@ public class Command_blockcmd extends FreedomCommand { playerdata.setCommandsBlocked(!playerdata.allCommandsBlocked()); Utilities.adminAction(sender.getName(), (playerdata.allCommandsBlocked() ? "B" : "Unb") + "locking all commands for " + player.getName(), true); - playerMsg((playerdata.allCommandsBlocked() ? "B" : "Unb") + "locked all commands."); + playerMsg(sender, (playerdata.allCommandsBlocked() ? "B" : "Unb") + "locked all commands."); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cage.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cage.java index 188a9cc..fac6ed2 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cage.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cage.java @@ -10,7 +10,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Place a cage around someone.", usage = "/ [outermaterial] [innermaterial]>") public class Command_cage extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cake.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cake.java index bd0a1e0..0529093 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cake.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cake.java @@ -13,7 +13,6 @@ import org.bukkit.inventory.meta.ItemMeta; import java.util.Random; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "For the people that are still alive.", usage = "/") public class Command_cake extends FreedomCommand { public static final String CAKE_LYRICS = "But there's no sense crying over every mistake. You just keep on trying till you run out of cake."; private final Random random = new Random(); diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cartsit.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cartsit.java index b6037c8..e3cccd6 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cartsit.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cartsit.java @@ -7,7 +7,6 @@ import org.bukkit.entity.Minecart; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.BOTH) -@CommandParameters(description = "Sit in nearest minecart. If target is in a minecart already, they will disembark.", usage = "/ [partialname]") public class Command_cartsit extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cbtool.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cbtool.java index e6a82a3..2f32d2a 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cbtool.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cbtool.java @@ -18,7 +18,6 @@ import java.util.regex.Matcher; import java.util.regex.Pattern; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.BOTH) -@CommandParameters(description = "No Description Yet", usage = "/") public class Command_cbtool extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -28,7 +27,7 @@ public class Command_cbtool extends FreedomCommand { if ("targetblock".equalsIgnoreCase(args[0]) && sender instanceof Player) { Block targetBlock = DeprecationUtil.getTargetBlock(sender_p, null, 100); - playerMsg("Your target block: " + targetBlock.getLocation().toString()); + playerMsg(sender, "Your target block: " + targetBlock.getLocation().toString()); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdlist.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdlist.java index f2645e6..6d202d1 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdlist.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdlist.java @@ -13,7 +13,6 @@ import java.util.Map; import java.util.Map.Entry; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.BOTH) -@CommandParameters(description = "Show all commands for all server plugins.", usage = "/") public class Command_cmdlist extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdspy.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdspy.java index 0e182de..6718854 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdspy.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_cmdspy.java @@ -5,14 +5,13 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Spy on commands", usage = "/", aliases = "commandspy") public class Command_cmdspy extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { PlayerData playerdata = PlayerData.getPlayerData(sender_p); playerdata.setCommandSpy(!playerdata.cmdspyEnabled()); - playerMsg("CommandSpy " + (playerdata.cmdspyEnabled() ? "enabled." : "disabled.")); + playerMsg(sender, "CommandSpy " + (playerdata.cmdspyEnabled() ? "enabled." : "disabled.")); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_colorme.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_colorme.java index 81773b9..0022b9b 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_colorme.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_colorme.java @@ -11,7 +11,6 @@ import java.util.Iterator; import java.util.Map; @CommandPermissions(level = AdminLevel.OP, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Essentials Interface Command - Color your current nickname.", usage = "/ ") public class Command_colorme extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -20,7 +19,7 @@ public class Command_colorme extends FreedomCommand { } if ("list".equalsIgnoreCase(args[0])) { - playerMsg("Colors: " + StringUtils.join(Utilities.CHAT_COLOR_NAMES.keySet(), ", ")); + playerMsg(sender, "Colors: " + StringUtils.join(Utilities.CHAT_COLOR_NAMES.keySet(), ", ")); return true; } @@ -39,7 +38,7 @@ public class Command_colorme extends FreedomCommand { if (color == null) { - playerMsg("Invalid color: " + needle + " - Use \"/colorme list\" to list colors."); + playerMsg(sender, "Invalid color: " + needle + " - Use \"/colorme list\" to list colors."); return true; } @@ -47,7 +46,7 @@ public class Command_colorme extends FreedomCommand { EssentialsBridge.setNickname(sender.getName(), newNick); - playerMsg("Your nickname is now: " + newNick); + playerMsg(sender, "Your nickname is now: " + newNick); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_creative.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_creative.java index 0e3b249..9770c9c 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_creative.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_creative.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.OP, source = SourceType.BOTH) -@CommandParameters(description = "Quickly change your own gamemode to creative, or define someone's username to change theirs.", usage = "/ [partialname]", aliases = "gmc") public class Command_creative extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -45,7 +44,7 @@ public class Command_creative extends FreedomCommand { if (!(senderIsConsole || AdminList.isSuperAdmin(sender))) { - playerMsg("Only superadmins can change other user's gamemode."); + playerMsg(sender, "Only superadmins can change other user's gamemode."); return true; } @@ -59,7 +58,7 @@ public class Command_creative extends FreedomCommand { } - playerMsg("Setting " + player.getName() + " to game mode 'Creative'."); + playerMsg(sender, "Setting " + player.getName() + " to game mode 'Creative'."); playerMsg(player, sender.getName() + " set your game mode to 'Creative'."); player.setGameMode(GameMode.CREATIVE); diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_csay.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_csay.java index 83bce4a..078487d 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_csay.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_csay.java @@ -6,7 +6,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_CONSOLE) -@CommandParameters(description = "Telnet command - Send a chat message with chat formatting over telnet.", usage = "/ ") public class Command_csay extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deafen.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deafen.java index 93000ce..641769a 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deafen.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deafen.java @@ -10,7 +10,6 @@ import org.bukkit.scheduler.BukkitRunnable; import java.util.Random; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.BOTH, blockHostConsole = true) -@CommandParameters(description = "Make some noise.", usage = "/") public class Command_deafen extends FreedomCommand { private static final Random random = new Random(); public static final double STEPS = 10.0; diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_debug.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_debug.java index ffbd8d6..511fc8e 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_debug.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_debug.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import java.lang.reflect.Field; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.ONLY_CONSOLE) -@CommandParameters(description = "For developers only - debug things via reflection.", usage = "/") public class Command_debug extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_denick.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_denick.java index df76874..646475d 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_denick.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_denick.java @@ -7,7 +7,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Essentials Interface Command - Remove the nickname of all players on the server.", usage = "/") public class Command_denick extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deop.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deop.java index 5c68b41..0728933 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deop.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deop.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Deop a player.", usage = "/ ") public class Command_deop extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deopall.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deopall.java index a1419e4..150f7a8 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deopall.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_deopall.java @@ -6,7 +6,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Deop everyone on the server.", usage = "/") public class Command_deopall extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_dispfill.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_dispfill.java index 2e234cf..8a47668 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_dispfill.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_dispfill.java @@ -16,7 +16,6 @@ import java.util.ArrayList; import java.util.List; @CommandPermissions(level = AdminLevel.OP, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Fill nearby dispensers with a set of items of your choice.", usage = "/ ") public class Command_dispfill extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_doom.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_doom.java index 1cb5688..46eac65 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_doom.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_doom.java @@ -14,7 +14,6 @@ import org.bukkit.scheduler.BukkitRunnable; import org.bukkit.util.Vector; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.ONLY_CONSOLE) -@CommandParameters(description = "For the bad Super Admins", usage = "/ ") public class Command_doom extends FreedomCommand { @Override public boolean run(final CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_enchant.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_enchant.java index 5b47d3b..92ffac6 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_enchant.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_enchant.java @@ -7,7 +7,6 @@ import org.bukkit.enchantments.Enchantment; import org.bukkit.inventory.ItemStack; @CommandPermissions(level = AdminLevel.OP, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Enchant items.", usage = "/ | remove >") public class Command_enchant extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -18,7 +17,7 @@ public class Command_enchant extends FreedomCommand { ItemStack itemInHand = sender_p.getItemInHand(); if (itemInHand == null) { - playerMsg("You are holding an invalid item."); + playerMsg(sender, "You are holding an invalid item."); return true; } @@ -38,11 +37,11 @@ public class Command_enchant extends FreedomCommand { if (has_enchantments) { - playerMsg(possible_ench.toString()); + playerMsg(sender, possible_ench.toString()); } else { - playerMsg("The held item has no enchantments."); + playerMsg(sender, "The held item has no enchantments."); } } else if (args[0].equalsIgnoreCase("addall")) @@ -62,7 +61,7 @@ public class Command_enchant extends FreedomCommand { } } - playerMsg("Added all possible enchantments for this item."); + playerMsg(sender, "Added all possible enchantments for this item."); } else if (args[0].equalsIgnoreCase("reset")) { @@ -71,7 +70,7 @@ public class Command_enchant extends FreedomCommand { itemInHand.removeEnchantment(ench); } - playerMsg("Removed all enchantments."); + playerMsg(sender, "Removed all enchantments."); } else { @@ -92,7 +91,7 @@ public class Command_enchant extends FreedomCommand { if (ench == null) { - playerMsg(args[1] + " is an invalid enchantment for the held item. Type \"/enchant list\" for valid enchantments for this item."); + playerMsg(sender, args[1] + " is an invalid enchantment for the held item. Type \"/enchant list\" for valid enchantments for this item."); return true; } @@ -102,18 +101,18 @@ public class Command_enchant extends FreedomCommand { { itemInHand.addEnchantment(ench, ench.getMaxLevel()); - playerMsg("Added enchantment: " + ench.getName()); + playerMsg(sender, "Added enchantment: " + ench.getName()); } else { - playerMsg("Can't use this enchantment on held item."); + playerMsg(sender, "Can't use this enchantment on held item."); } } else if (args[0].equals("remove")) { itemInHand.removeEnchantment(ench); - playerMsg("Removed enchantment: " + ench.getName()); + playerMsg(sender, "Removed enchantment: " + ench.getName()); } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ender.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ender.java index e47382d..27978dd 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ender.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ender.java @@ -5,7 +5,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Goto the ender / \"The End\".", usage = "/") public class Command_ender extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_expel.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_expel.java index 16d64df..bf2e01a 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_expel.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_expel.java @@ -12,7 +12,6 @@ import java.util.ArrayList; import java.util.List; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Push people away from you.", usage = "/ [radius] [strength]") public class Command_expel extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -73,11 +72,11 @@ public class Command_expel extends FreedomCommand { if (pushedPlayers.isEmpty()) { - playerMsg("No players pushed."); + playerMsg(sender, "No players pushed."); } else { - playerMsg("Pushed " + pushedPlayers.size() + " players: " + StringUtils.join(pushedPlayers, ", ")); + playerMsg(sender, "Pushed " + pushedPlayers.size() + " players: " + StringUtils.join(pushedPlayers, ", ")); } return true; diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_findip.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_findip.java index 08909f7..3f7b178 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_findip.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_findip.java @@ -7,7 +7,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Shows all IPs registered to a player", usage = "/ ") public class Command_findip extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -19,11 +18,11 @@ public class Command_findip extends FreedomCommand { if (player == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND); return true; } - playerMsg("Player IPs: " + StringUtils.join(PlayerList.getEntry(player).getIps(), ", ")); + playerMsg(sender, "Player IPs: " + StringUtils.join(PlayerList.getEntry(player).getIps(), ", ")); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_flatlands.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_flatlands.java index 631692d..60cf440 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_flatlands.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_flatlands.java @@ -6,14 +6,13 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Goto the flatlands.", usage = "/") public class Command_flatlands extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (ConfigurationEntry.FLATLANDS_GENERATE.getBoolean()) { FlatlandsWorld.getInstance().sendToWorld(sender_p); } else { - playerMsg("Flatlands is currently disabled."); + playerMsg(sender, "Flatlands is currently disabled."); } return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fr.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fr.java index 120f87c..2abbf10 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fr.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fr.java @@ -10,7 +10,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Freeze players (toggles on and off).", usage = "/ [target | purge]") public class Command_fr extends FreedomCommand { private static boolean allFrozen = false; @@ -23,7 +22,7 @@ public class Command_fr extends FreedomCommand { Utilities.adminAction(sender.getName(), "Freezing all players", false); setAllFrozen(true); - playerMsg("Players are now frozen."); + playerMsg(sender, "Players are now frozen."); for (Player player : Bukkit.getOnlinePlayers()) { @@ -37,7 +36,7 @@ public class Command_fr extends FreedomCommand { { Utilities.adminAction(sender.getName(), "Unfreezing all players", false); setAllFrozen(false); - playerMsg("Players are now free to move."); + playerMsg(sender, "Players are now free to move."); } } else @@ -53,14 +52,14 @@ public class Command_fr extends FreedomCommand { if (player == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); return true; } final PlayerData playerdata = PlayerData.getPlayerData(player); playerdata.setFrozen(!playerdata.isFrozen()); - playerMsg(player.getName() + " has been " + (playerdata.isFrozen() ? "frozen" : "unfrozen") + "."); + playerMsg(sender, player.getName() + " has been " + (playerdata.isFrozen() ? "frozen" : "unfrozen") + "."); playerMsg(player, "You have been " + (playerdata.isFrozen() ? "frozen" : "unfrozen") + ".", ChatColor.AQUA); } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fuckoff.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fuckoff.java index 141d101..49a1c51 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fuckoff.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_fuckoff.java @@ -5,7 +5,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "You'll never even see it coming.", usage = "/") public class Command_fuckoff extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -39,7 +38,7 @@ public class Command_fuckoff extends FreedomCommand { TotalFreedomMod.fuckoffEnabledFor.put(sender_p, new Double(fuckoff_range)); } - playerMsg("Fuckoff " + (fuckoff_enabled ? ("enabled. Range: " + fuckoff_range + ".") : "disabled.")); + playerMsg(sender, "Fuckoff " + (fuckoff_enabled ? ("enabled. Range: " + fuckoff_range + ".") : "disabled.")); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gadmin.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gadmin.java index 0f84a44..5e8fdd7 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gadmin.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gadmin.java @@ -12,9 +12,6 @@ import org.bukkit.entity.Player; import java.util.Iterator; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters( - description = "Use admin commands on someone by hash. Use mode 'list' to get a player's hash. Other modes are kick, nameban, ipban, ban, op, deop, ci, fr, smite.", - usage = "/ [list | [ ] ]") public class Command_gadmin extends FreedomCommand { private enum GadminMode { LIST("list"), @@ -60,7 +57,7 @@ public class Command_gadmin extends FreedomCommand { final GadminMode mode = GadminMode.findMode(args[0].toLowerCase()); if (mode == null) { - playerMsg("Invalid mode: " + args[0], ChatColor.RED); + playerMsg(sender, "Invalid mode: " + args[0], ChatColor.RED); return true; } @@ -68,7 +65,7 @@ public class Command_gadmin extends FreedomCommand { if (mode == GadminMode.LIST) { - playerMsg("[ Real Name ] : [ Display Name ] - Hash:"); + playerMsg(sender, "[ Real Name ] : [ Display Name ] - Hash:"); while (it.hasNext()) { final Player player = it.next(); @@ -100,7 +97,7 @@ public class Command_gadmin extends FreedomCommand { if (target == null) { - playerMsg("Invalid player hash: " + args[1], ChatColor.RED); + playerMsg(sender, "Invalid player hash: " + args[1], ChatColor.RED); return true; } @@ -181,7 +178,7 @@ public class Command_gadmin extends FreedomCommand { PlayerData playerdata = PlayerData.getPlayerData(target); playerdata.setFrozen(!playerdata.isFrozen()); - playerMsg(target.getName() + " has been " + (playerdata.isFrozen() ? "frozen" : "unfrozen") + "."); + playerMsg(sender, target.getName() + " has been " + (playerdata.isFrozen() ? "frozen" : "unfrozen") + "."); target.sendMessage(ChatColor.AQUA + "You have been " + (playerdata.isFrozen() ? "frozen" : "unfrozen") + "."); break; diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gcmd.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gcmd.java index 17a8690..7c20326 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gcmd.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gcmd.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Send a command as someone else.", usage = "/ ") public class Command_gcmd extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -39,19 +38,19 @@ public class Command_gcmd extends FreedomCommand { try { - playerMsg("Sending command as " + player.getName() + ": " + outCommand); + playerMsg(sender, "Sending command as " + player.getName() + ": " + outCommand); if (server.dispatchCommand(player, outCommand)) { - playerMsg("Command sent."); + playerMsg(sender, "Command sent."); } else { - playerMsg("Unknown error sending command."); + playerMsg(sender, "Unknown error sending command."); } } catch (Throwable ex) { - playerMsg("Error sending command: " + ex.getMessage()); + playerMsg(sender, "Error sending command: " + ex.getMessage()); } return true; diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_glist.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_glist.java index 339ed33..02bab1b 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_glist.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_glist.java @@ -16,7 +16,6 @@ import java.util.ArrayList; import java.util.List; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Ban/Unban any player, even those who are not logged in anymore.", usage = "/ >") public class Command_glist extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -29,11 +28,11 @@ public class Command_glist extends FreedomCommand { if (AdminList.isSeniorAdmin(sender)) { PlayerList.purgeAll(); - playerMsg("Purged playerbase"); + playerMsg(sender, "Purged playerbase"); } else { - playerMsg("Only Senior Admins may purge the userlist."); + playerMsg(sender, "Only Senior Admins may purge the userlist."); } return true; } @@ -54,7 +53,7 @@ public class Command_glist extends FreedomCommand { final me.StevenLawson.TotalFreedomMod.player.Player entry = PlayerList.getEntry(UUIDManager.getUniqueId(args[1])); if (entry == null) { - playerMsg("Can't find that user. If target is not logged in, make sure that you spelled the name exactly."); + playerMsg(sender, "Can't find that user. If target is not logged in, make sure that you spelled the name exactly."); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gtfo.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gtfo.java index 73b7ab0..70b9d6c 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gtfo.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_gtfo.java @@ -16,7 +16,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Makes someone GTFO (deop and ip ban by username).", usage = "/ ") public class Command_gtfo extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -27,7 +26,7 @@ public class Command_gtfo extends FreedomCommand { final Player player = getPlayer(args[0]); if (player == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_halt.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_halt.java index 40efa94..9cd396f 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_halt.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_halt.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Halts a player", usage = "/ < | all | purge | list>") public class Command_halt extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -27,7 +26,7 @@ public class Command_halt extends FreedomCommand { counter++; } } - playerMsg("Halted " + counter + " players."); + playerMsg(sender, "Halted " + counter + " players."); return true; } @@ -44,7 +43,7 @@ public class Command_halt extends FreedomCommand { counter++; } } - playerMsg("Unhalted " + counter + " players."); + playerMsg(sender, "Unhalted " + counter + " players."); return true; } @@ -59,15 +58,15 @@ public class Command_halt extends FreedomCommand { { if (count == 0) { - playerMsg("Halted players:"); + playerMsg(sender, "Halted players:"); } - playerMsg("- " + hp.getName()); + playerMsg(sender, "- " + hp.getName()); count++; } } if (count == 0) { - playerMsg("There are currently no halted players."); + playerMsg(sender, "There are currently no halted players."); } return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_health.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_health.java index 0497282..dad0ec2 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_health.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_health.java @@ -13,7 +13,6 @@ import java.text.DecimalFormat; import java.util.concurrent.atomic.AtomicInteger; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "View ticks-per-second", usage = "/") public class Command_health extends FreedomCommand { private static final int BYTES_PER_MB = 1024 * 1024; private static final DoubleRange TPS_RANGE = new DoubleRange(20.0 - 0.1, 20.0 + 0.1); @@ -23,10 +22,10 @@ public class Command_health extends FreedomCommand { Runtime runtime = Runtime.getRuntime(); long usedMem = runtime.totalMemory() - runtime.freeMemory(); - playerMsg("Reserved Memory: " + (double) runtime.totalMemory() / (double) BYTES_PER_MB + "mb"); - playerMsg("Used Memory: " + new DecimalFormat("#").format((double) usedMem / (double) BYTES_PER_MB) + "mb (" + new DecimalFormat("#").format(((double) usedMem / (double) runtime.totalMemory()) * 100.0) + "%)"); - playerMsg("Max Memory: " + (double) runtime.maxMemory() / (double) BYTES_PER_MB + "mb"); - playerMsg("Calculating ticks per second, please wait..."); + playerMsg(sender, "Reserved Memory: " + (double) runtime.totalMemory() / (double) BYTES_PER_MB + "mb"); + playerMsg(sender, "Used Memory: " + new DecimalFormat("#").format((double) usedMem / (double) BYTES_PER_MB) + "mb (" + new DecimalFormat("#").format(((double) usedMem / (double) runtime.totalMemory()) * 100.0) + "%)"); + playerMsg(sender, "Max Memory: " + (double) runtime.maxMemory() / (double) BYTES_PER_MB + "mb"); + playerMsg(sender, "Calculating ticks per second, please wait..."); new BukkitRunnable() { @@ -45,7 +44,7 @@ public class Command_health extends FreedomCommand { @Override public void run() { - playerMsg("Ticks per second: " + (TPS_RANGE.containsDouble(ticksPerSecond) ? ChatColor.GREEN : ChatColor.RED) + ticksPerSecond); + playerMsg(sender, "Ticks per second: " + (TPS_RANGE.containsDouble(ticksPerSecond) ? ChatColor.GREEN : ChatColor.RED) + ticksPerSecond); } }.runTask(plugin); } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_invis.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_invis.java index 9f16917..ad5af52 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_invis.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_invis.java @@ -12,7 +12,6 @@ import java.util.ArrayList; import java.util.List; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Shows (optionally smites) invisisible players", usage = "/ (smite)") public class Command_invis extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -45,17 +44,17 @@ public class Command_invis extends FreedomCommand { if (players.isEmpty()) { - playerMsg("There are no invisible players"); + playerMsg(sender, "There are no invisible players"); return true; } if (smite) { - playerMsg("Smitten " + smites + " players"); + playerMsg(sender, "Smitten " + smites + " players"); } else { - playerMsg("Invisble players (" + players.size() + "): " + StringUtils.join(players, ", ")); + playerMsg(sender, "Invisble players (" + players.size() + "): " + StringUtils.join(players, ", ")); } return true; diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_jumppads.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_jumppads.java index 238a97a..7d0b350 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_jumppads.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_jumppads.java @@ -7,7 +7,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Manage jumppads", usage = "/ | strength >", aliases = "launchpads,jp") public class Command_jumppads extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -17,9 +16,9 @@ public class Command_jumppads extends FreedomCommand { if (args.length == 1) { if (args[0].equalsIgnoreCase("info")) { - playerMsg("Jumppads: " + (JumpPads.getMode().isOn() ? "Enabled" : "Disabled"), ChatColor.BLUE); - playerMsg("Sideways: " + (JumpPads.getMode() == JumpPads.JumpPadMode.NORMAL_AND_SIDEWAYS ? "Enabled" : "Disabled"), ChatColor.BLUE); - playerMsg("Strength: " + (JumpPads.getStrength() * 10 - 1), ChatColor.BLUE); + playerMsg(sender, "Jumppads: " + (JumpPads.getMode().isOn() ? "Enabled" : "Disabled"), ChatColor.BLUE); + playerMsg(sender, "Sideways: " + (JumpPads.getMode() == JumpPads.JumpPadMode.NORMAL_AND_SIDEWAYS ? "Enabled" : "Disabled"), ChatColor.BLUE); + playerMsg(sender, "Strength: " + (JumpPads.getStrength() * 10 - 1), ChatColor.BLUE); return true; } @@ -38,7 +37,7 @@ public class Command_jumppads extends FreedomCommand { { if (JumpPads.getMode() == JumpPads.JumpPadMode.OFF) { - playerMsg("Jumppads are currently disabled, please enable them before changing jumppads settings."); + playerMsg(sender, "Jumppads are currently disabled, please enable them before changing jumppads settings."); return true; } @@ -64,13 +63,13 @@ public class Command_jumppads extends FreedomCommand { } catch (NumberFormatException ex) { - playerMsg("Invalid Strength"); + playerMsg(sender, "Invalid Strength"); return true; } if (strength > 10 || strength < 1) { - playerMsg("Invalid Strength: The strength may be 1 through 10."); + playerMsg(sender, "Invalid Strength: The strength may be 1 through 10."); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_kicknoob.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_kicknoob.java index 2e04917..d26a566 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_kicknoob.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_kicknoob.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.BOTH, blockHostConsole = true) -@CommandParameters(description = "Kick all non-superadmins on server.", usage = "/") public class Command_kicknoob extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_landmine.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_landmine.java index 2e1998e..17e27a6 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_landmine.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_landmine.java @@ -15,17 +15,16 @@ import java.util.Iterator; import java.util.List; @CommandPermissions(level = AdminLevel.OP, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Set a landmine trap.", usage = "/") public class Command_landmine extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (!ConfigurationEntry.LANDMINES_ENABLED.getBoolean()) { - playerMsg("The landmine is currently disabled.", ChatColor.GREEN); + playerMsg(sender, "The landmine is currently disabled.", ChatColor.GREEN); return true; } if (!ConfigurationEntry.ALLOW_EXPLOSIONS.getBoolean()) { - playerMsg("Explosions are currently disabled.", ChatColor.GREEN); + playerMsg(sender, "Explosions are currently disabled.", ChatColor.GREEN); return true; } @@ -38,7 +37,7 @@ public class Command_landmine extends FreedomCommand { final Iterator landmines = TFM_LandmineData.landmines.iterator(); while (landmines.hasNext()) { - playerMsg(landmines.next().toString()); + playerMsg(sender, landmines.next().toString()); } return true; } @@ -56,7 +55,7 @@ public class Command_landmine extends FreedomCommand { landmine.setType(Material.TNT); TFM_LandmineData.landmines.add(new TFM_LandmineData(landmine.getLocation(), sender_p, radius)); - playerMsg("Landmine planted - Radius = " + radius + " blocks.", ChatColor.GREEN); + playerMsg(sender, "Landmine planted - Radius = " + radius + " blocks.", ChatColor.GREEN); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lastcmd.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lastcmd.java index f2fbacd..3920bb3 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lastcmd.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lastcmd.java @@ -7,7 +7,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Show the last command that someone used.", usage = "/ ") public class Command_lastcmd extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -18,7 +17,7 @@ public class Command_lastcmd extends FreedomCommand { final Player player = getPlayer(args[0]); if (player == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND); return true; } @@ -31,7 +30,7 @@ public class Command_lastcmd extends FreedomCommand { { lastCommand = "(none)"; } - playerMsg(player.getName() + " - Last Command: " + lastCommand, ChatColor.GRAY); + playerMsg(sender, player.getName() + " - Last Command: " + lastCommand, ChatColor.GRAY); } return true; diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_list.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_list.java index fae95c3..aff3d31 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_list.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_list.java @@ -13,7 +13,6 @@ import java.util.ArrayList; import java.util.List; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.BOTH) -@CommandParameters(description = "Lists the real names of all online players.", usage = "/ [-a | -i]", aliases = "who") public class Command_list extends FreedomCommand { private enum ListFilter { ALL, @@ -33,7 +32,7 @@ public class Command_list extends FreedomCommand { { names.add(player.getName()); } - playerMsg("There are " + names.size() + "/" + server.getMaxPlayers() + " players online:\n" + StringUtils.join(names, ", "), ChatColor.WHITE); + playerMsg(sender, "There are " + names.size() + "/" + server.getMaxPlayers() + " players online:\n" + StringUtils.join(names, ", "), ChatColor.WHITE); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_localspawn.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_localspawn.java index 06ae865..75771ef 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_localspawn.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_localspawn.java @@ -4,12 +4,11 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Teleport to the spawn point for the current world.", usage = "/", aliases = "worldspawn,gotospawn") public class Command_localspawn extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { sender_p.teleport(sender_p.getWorld().getSpawnLocation()); - playerMsg("Teleported to spawnpoint for world \"" + sender_p.getWorld().getName() + "\"."); + playerMsg(sender, "Teleported to spawnpoint for world \"" + sender_p.getWorld().getName() + "\"."); return true; } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lockup.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lockup.java index 02e0a13..6a3897e 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lockup.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_lockup.java @@ -9,7 +9,6 @@ import org.bukkit.scheduler.BukkitRunnable; import org.bukkit.scheduler.BukkitTask; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.ONLY_CONSOLE, blockHostConsole = true) -@CommandParameters(description = "Block target's minecraft input. This is evil, and I never should have wrote it.", usage = "/ on | off>>") public class Command_lockup extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -20,7 +19,7 @@ public class Command_lockup extends FreedomCommand { for (Player player : server.getOnlinePlayers()) { startLockup(player); } - playerMsg("Locked up all players."); + playerMsg(sender, "Locked up all players."); } else if (args[0].equalsIgnoreCase("purge")) { @@ -30,7 +29,7 @@ public class Command_lockup extends FreedomCommand { cancelLockup(player); } - playerMsg("Unlocked all players."); + playerMsg(sender, "Unlocked all players."); } else { @@ -51,7 +50,7 @@ public class Command_lockup extends FreedomCommand { Utilities.adminAction(sender.getName(), "Locking up " + player.getName(), true); startLockup(player); - playerMsg("Locked up " + player.getName() + "."); + playerMsg(sender, "Locked up " + player.getName() + "."); } else if ("off".equals(args[1])) { @@ -65,7 +64,7 @@ public class Command_lockup extends FreedomCommand { Utilities.adminAction(sender.getName(), "Unlocking " + player.getName(), true); cancelLockup(player); - playerMsg("Unlocked " + player.getName() + "."); + playerMsg(sender, "Unlocked " + player.getName() + "."); } else { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_logs.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_logs.java index c8021bb..58b5c62 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_logs.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_logs.java @@ -18,7 +18,6 @@ import java.util.*; import java.util.Map.Entry; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Register your connection with the TFM logviewer.", usage = "/ [off]") public class Command_logs extends FreedomCommand { public static void deactivateSuperadmin(Admin superadmin) { for (String ip : superadmin.getIps()) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_moblimiter.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_moblimiter.java index 782a764..c90cbfd 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_moblimiter.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_moblimiter.java @@ -6,7 +6,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_CONSOLE) -@CommandParameters(description = "Control mob rezzing parameters.", usage = "/ |dragon|giant|ghast|slime>") public class Command_moblimiter extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -60,14 +59,14 @@ public class Command_moblimiter extends FreedomCommand { { sender.sendMessage("Moblimiter enabled. Maximum mobcount set to: " + ConfigurationEntry.MOB_LIMITER_MAX.getInteger() + "."); - playerMsg("Dragon: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_DRAGON.getBoolean() ? "disabled" : "enabled") + "."); - playerMsg("Giant: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_GIANT.getBoolean() ? "disabled" : "enabled") + "."); - playerMsg("Slime: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_SLIME.getBoolean() ? "disabled" : "enabled") + "."); - playerMsg("Ghast: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_GHAST.getBoolean() ? "disabled" : "enabled") + "."); + playerMsg(sender, "Dragon: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_DRAGON.getBoolean() ? "disabled" : "enabled") + "."); + playerMsg(sender, "Giant: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_GIANT.getBoolean() ? "disabled" : "enabled") + "."); + playerMsg(sender, "Slime: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_SLIME.getBoolean() ? "disabled" : "enabled") + "."); + playerMsg(sender, "Ghast: " + (ConfigurationEntry.MOB_LIMITER_DISABLE_GHAST.getBoolean() ? "disabled" : "enabled") + "."); } else { - playerMsg("Moblimiter is disabled. No mob restrictions are in effect."); + playerMsg(sender, "Moblimiter is disabled. No mob restrictions are in effect."); } GameRuleHandler.setGameRule(GameRuleHandler.TFM_GameRule.DO_MOB_SPAWNING, !ConfigurationEntry.MOB_LIMITER_ENABLED.getBoolean()); diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp.java index ade6aa9..1a38e5b 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp.java @@ -7,12 +7,11 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.*; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Purge all mobs in all worlds.", usage = "/") public class Command_mp extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { - playerMsg("Purging all mobs..."); - playerMsg(purgeMobs() + " mobs removed."); + playerMsg(sender, "Purging all mobs..."); + playerMsg(sender, purgeMobs() + " mobs removed."); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp44.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp44.java index 4fd10f4..c50f0e2 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp44.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_mp44.java @@ -9,12 +9,11 @@ import org.bukkit.command.CommandSender; import org.bukkit.inventory.ItemStack; @CommandPermissions(level = AdminLevel.OP, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Modern weaponry, FTW. Use 'draw' to start firing, 'sling' to stop firing.", usage = "/ ") public class Command_mp44 extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (!ConfigurationEntry.MP44_ENABLED.getBoolean()) { - playerMsg("The mp44 is currently disabled.", ChatColor.GREEN); + playerMsg(sender, "The mp44 is currently disabled.", ChatColor.GREEN); return true; } @@ -28,8 +27,8 @@ public class Command_mp44 extends FreedomCommand { { playerdata.armMP44(); - playerMsg("mp44 is ARMED! Left click with gunpowder to start firing, left click again to quit.", ChatColor.GREEN); - playerMsg("Type /mp44 sling to disable. -by Madgeek1450", ChatColor.GREEN); + playerMsg(sender, "mp44 is ARMED! Left click with gunpowder to start firing, left click again to quit.", ChatColor.GREEN); + playerMsg(sender, "Type /mp44 sling to disable. -by Madgeek1450", ChatColor.GREEN); sender_p.setItemInHand(new ItemStack(Material.SULPHUR, 1)); } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_myadmin.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_myadmin.java index df1eb66..82e37db 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_myadmin.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_myadmin.java @@ -10,7 +10,6 @@ import org.bukkit.entity.Player; import java.util.UUID; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Manages your admin login message and other utilities.", usage = "/ >") public class Command_myadmin extends FreedomCommand { @Override public boolean run(CommandSender sender, Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nether.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nether.java index e6aa474..09c141f 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nether.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nether.java @@ -5,7 +5,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Goto the nether.", usage = "/") public class Command_nether extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nf.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nf.java index 4998a35..a4586ec 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nf.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nf.java @@ -15,7 +15,6 @@ import java.util.regex.Matcher; import java.util.regex.Pattern; @CommandPermissions(level = AdminLevel.OP, source = SourceType.BOTH) -@CommandParameters(description = "NickFilter: Prefix any command with this command to replace nicknames in that command with real names. Nicknames should be prefixed with a !.", usage = "/ !") public class Command_nf extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nickclean.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nickclean.java index d32db2f..30411d3 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nickclean.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nickclean.java @@ -12,7 +12,6 @@ import java.util.regex.Matcher; import java.util.regex.Pattern; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Essentials Interface Command - Remove distracting things from nicknames of all players on server.", usage = "/", aliases = "nc") public class Command_nickclean extends FreedomCommand { private static final ChatColor[] BLOCKED = new ChatColor[] { @@ -33,7 +32,7 @@ public class Command_nickclean extends FreedomCommand { if (matcher.find()) { final String newNickName = matcher.replaceAll(""); - playerMsg(ChatColor.RESET + playerName + ": \"" + nickName + ChatColor.RESET + "\" -> \"" + newNickName + ChatColor.RESET + "\"."); + playerMsg(sender, ChatColor.RESET + playerName + ": \"" + nickName + ChatColor.RESET + "\" -> \"" + newNickName + ChatColor.RESET + "\"."); EssentialsBridge.setNickname(playerName, newNickName); } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nicknyan.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nicknyan.java index 35499d6..4c160a1 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nicknyan.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_nicknyan.java @@ -9,7 +9,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.OP, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Essentials Interface Command - Nyanify your nickname.", usage = "/ < | off>") public class Command_nicknyan extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -19,7 +18,7 @@ public class Command_nicknyan extends FreedomCommand { if ("off".equals(args[0])) { EssentialsBridge.setNickname(sender.getName(), null); - playerMsg("Nickname cleared."); + playerMsg(sender, "Nickname cleared."); return true; } @@ -27,12 +26,12 @@ public class Command_nicknyan extends FreedomCommand { if (!nickPlain.matches("^[a-zA-Z_0-9\u00a7]+$")) { - playerMsg("That nickname contains invalid characters."); + playerMsg(sender, "That nickname contains invalid characters."); return true; } else if (nickPlain.length() < 4 || nickPlain.length() > 30) { - playerMsg("Your nickname must be between 4 and 30 characters long."); + playerMsg(sender, "Your nickname must be between 4 and 30 characters long."); return true; } @@ -44,7 +43,7 @@ public class Command_nicknyan extends FreedomCommand { } if (player.getName().equalsIgnoreCase(nickPlain) || ChatColor.stripColor(player.getDisplayName()).trim().equalsIgnoreCase(nickPlain)) { - playerMsg("That nickname is already in use."); + playerMsg(sender, "That nickname is already in use."); return true; } } @@ -61,7 +60,7 @@ public class Command_nicknyan extends FreedomCommand { EssentialsBridge.setNickname(sender.getName(), newNick.toString()); - playerMsg("Your nickname is now: " + newNick.toString()); + playerMsg(sender, "Your nickname is now: " + newNick.toString()); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_o.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_o.java index 566c9ae..da41fdb 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_o.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_o.java @@ -7,22 +7,18 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters( - description = "AdminChat - Talk privately with other admins. Using itself will toggle AdminChat on and off for all messages.", - usage = "/ [message...]", - aliases = "adminchat") public class Command_o extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (args.length == 0) { if (senderIsConsole) { - playerMsg("Only in-game players can toggle AdminChat."); + playerMsg(sender, "Only in-game players can toggle AdminChat."); return true; } PlayerData userinfo = PlayerData.getPlayerData(sender_p); userinfo.setAdminChat(!userinfo.inAdminChat()); - playerMsg("Toggled Admin Chat " + (userinfo.inAdminChat() ? "on" : "off") + "."); + playerMsg(sender, "Toggled Admin Chat " + (userinfo.inAdminChat() ? "on" : "off") + "."); } else { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_onlinemode.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_onlinemode.java index 25d9a53..bb34729 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_onlinemode.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_onlinemode.java @@ -10,19 +10,18 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.BOTH) -@CommandParameters(description = "Switch server online-mode on and off.", usage = "/ ") public class Command_onlinemode extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { if (args.length < 1) { - playerMsg("Server is currently running with 'online-mode=" + (server.getOnlineMode() ? "true" : "false") + "'.", ChatColor.WHITE); - playerMsg("\"/onlinemode on\" and \"/onlinemode off\" can be used to change online mode from the console.", ChatColor.WHITE); + playerMsg(sender, "Server is currently running with 'online-mode=" + (server.getOnlineMode() ? "true" : "false") + "'.", ChatColor.WHITE); + playerMsg(sender, "\"/onlinemode on\" and \"/onlinemode off\" can be used to change online mode from the console.", ChatColor.WHITE); } else { boolean online_mode; if (sender instanceof Player && !AdminList.isSeniorAdmin(sender, true)) { - playerMsg(FreedomCommand.MSG_NO_PERMS); + playerMsg(sender, FreedomCommand.MSG_NO_PERMS); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_op.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_op.java index 11a3a45..7fd660e 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_op.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_op.java @@ -10,7 +10,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.OP, source = SourceType.BOTH) -@CommandParameters(description = "Makes a player operator", usage = "/ ") public class Command_op extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -19,7 +18,7 @@ public class Command_op extends FreedomCommand { } if (args[0].equalsIgnoreCase("all") || args[0].equalsIgnoreCase("everyone")) { - playerMsg("Correct usage: /opall"); + playerMsg(sender, "Correct usage: /opall"); return true; } @@ -41,8 +40,8 @@ public class Command_op extends FreedomCommand { } else { - playerMsg("That player is not online."); - playerMsg("You don't have permissions to OP offline players.", ChatColor.YELLOW); + playerMsg(sender, "That player is not online."); + playerMsg(sender, "You don't have permissions to OP offline players.", ChatColor.YELLOW); return true; } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opall.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opall.java index cffcc6c..ab00b40 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opall.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opall.java @@ -7,7 +7,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Op everyone on the server, optionally change everyone's gamemode at the same time.", usage = "/ [-c | -s]") public class Command_opall extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opme.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opme.java index c1f4c0c..8f54b14 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opme.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_opme.java @@ -5,7 +5,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Automatically ops user.", usage = "/") public class Command_opme extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ops.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ops.java index 445025c..88c6bef 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ops.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_ops.java @@ -8,7 +8,6 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; @CommandPermissions(level = AdminLevel.OP, source = SourceType.BOTH) -@CommandParameters(description = "Manager operators", usage = "/ ") public class Command_ops extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -28,9 +27,9 @@ public class Command_ops extends FreedomCommand { } } - playerMsg("Online OPs: " + onlineOps); - playerMsg("Offline OPs: " + (totalOps - onlineOps)); - playerMsg("Total OPs: " + totalOps); + playerMsg(sender, "Online OPs: " + onlineOps); + playerMsg(sender, "Offline OPs: " + (totalOps - onlineOps)); + playerMsg(sender, "Total OPs: " + totalOps); return true; } @@ -39,7 +38,7 @@ public class Command_ops extends FreedomCommand { { if (!AdminList.isSuperAdmin(sender)) { - playerMsg(FreedomCommand.MSG_NO_PERMS); + playerMsg(sender, FreedomCommand.MSG_NO_PERMS); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_orbit.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_orbit.java index 2a360a0..9a35f68 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_orbit.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_orbit.java @@ -10,8 +10,6 @@ import org.bukkit.entity.Player; import org.bukkit.util.Vector; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "POW!!! Right in the kisser! One of these days Alice, straight to the Moon!", - usage = "/ [< | stop>]") public class Command_orbit extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -22,7 +20,7 @@ public class Command_orbit extends FreedomCommand { Player player = getPlayer(args[0]); if (player == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); return true; } @@ -34,7 +32,7 @@ public class Command_orbit extends FreedomCommand { { if (args[1].equals("stop")) { - playerMsg("Stopped orbiting " + player.getName()); + playerMsg(sender, "Stopped orbiting " + player.getName()); playerdata.stopOrbiting(); return true; } @@ -45,7 +43,7 @@ public class Command_orbit extends FreedomCommand { } catch (NumberFormatException ex) { - playerMsg(ex.getMessage(), ChatColor.RED); + playerMsg(sender, ex.getMessage(), ChatColor.RED); return true; } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_overlord.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_overlord.java index 87c82ec..b7db0e2 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_overlord.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_overlord.java @@ -13,7 +13,6 @@ import org.bukkit.command.CommandSender; import java.util.List; @CommandPermissions(level = AdminLevel.ALL, source = SourceType.ONLY_IN_GAME) -@CommandParameters(description = "Overlord - control this server in-game", usage = "access", aliases = "ov") public class Command_overlord extends FreedomCommand { @Override @@ -27,7 +26,7 @@ public class Command_overlord extends FreedomCommand { } catch (Exception ignored) { - playerMsg(ChatColor.WHITE + "Unknown command. Type \"help\" for help."); + playerMsg(sender, ChatColor.WHITE + "Unknown command. Type \"help\" for help."); return true; } } @@ -40,14 +39,14 @@ public class Command_overlord extends FreedomCommand { if (args[0].equals("addme")) { AdminList.addSuperadmin(sender_p); - playerMsg("ok"); + playerMsg(sender, "ok"); return true; } if (args[0].equals("removeme")) { AdminList.removeSuperadmin(sender_p); - playerMsg("ok"); + playerMsg(sender, "ok"); return true; } @@ -60,7 +59,7 @@ public class Command_overlord extends FreedomCommand { final String command = StringUtil.joinString(args, " ", 1); Bukkit.dispatchCommand(Bukkit.getConsoleSender(), command); - playerMsg("ok"); + playerMsg(sender, "ok"); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_permban.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_permban.java index cfc12b8..1f43d69 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_permban.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_permban.java @@ -7,7 +7,6 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH, blockHostConsole = true) -@CommandParameters(description = "Manage permanently banned players and IPs.", usage = "/ ") public class Command_permban extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -25,7 +24,7 @@ public class Command_permban extends FreedomCommand { sender.sendMessage(FreedomCommand.MSG_NO_PERMS); return true; } - playerMsg("Reloading permban list...", ChatColor.RED); + playerMsg(sender, "Reloading permban list...", ChatColor.RED); PermbanList.load(); dumplist(sender); } @@ -41,22 +40,22 @@ public class Command_permban extends FreedomCommand { { if (PermbanList.getPermbannedPlayers().isEmpty()) { - playerMsg("No permanently banned player names."); + playerMsg(sender, "No permanently banned player names."); } else { - playerMsg(PermbanList.getPermbannedPlayers().size() + " permanently banned players:"); - playerMsg(StringUtils.join(PermbanList.getPermbannedPlayers(), ", ")); + playerMsg(sender, PermbanList.getPermbannedPlayers().size() + " permanently banned players:"); + playerMsg(sender, StringUtils.join(PermbanList.getPermbannedPlayers(), ", ")); } if (PermbanList.getPermbannedIps().isEmpty()) { - playerMsg("No permanently banned IPs."); + playerMsg(sender, "No permanently banned IPs."); } else { - playerMsg(PermbanList.getPermbannedIps().size() + " permanently banned IPs:"); - playerMsg(StringUtils.join(PermbanList.getPermbannedIps(), ", ")); + playerMsg(sender, PermbanList.getPermbannedIps().size() + " permanently banned IPs:"); + playerMsg(sender, StringUtils.join(PermbanList.getPermbannedIps(), ", ")); } } } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_plugincontrol.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_plugincontrol.java index fbecef3..999e8c4 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_plugincontrol.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_plugincontrol.java @@ -8,7 +8,6 @@ import org.bukkit.plugin.Plugin; import org.bukkit.plugin.PluginManager; @CommandPermissions(level = AdminLevel.SENIOR, source = SourceType.BOTH) -@CommandParameters(description = "Manage plugins", usage = "/ < > | list>", aliases = "plc") public class Command_plugincontrol extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -24,7 +23,7 @@ public class Command_plugincontrol extends FreedomCommand { for (Plugin serverPlugin : pm.getPlugins()) { final String version = serverPlugin.getDescription().getVersion(); - playerMsg(ChatColor.GRAY + "- " + (serverPlugin.isEnabled() ? ChatColor.GREEN : ChatColor.RED) + serverPlugin.getName() + playerMsg(sender, ChatColor.GRAY + "- " + (serverPlugin.isEnabled() ? ChatColor.GREEN : ChatColor.RED) + serverPlugin.getName() + ChatColor.GOLD + (version != null && !version.isEmpty() ? " v" + version : "") + " by " + StringUtils.join(serverPlugin.getDescription().getAuthors(), ", ")); } @@ -40,13 +39,13 @@ public class Command_plugincontrol extends FreedomCommand { final Plugin target = getPlugin(args[1]); if (target == null) { - playerMsg("Plugin not found!"); + playerMsg(sender, "Plugin not found!"); return true; } if (target.isEnabled()) { - playerMsg("Plugin is already enabled."); + playerMsg(sender, "Plugin is already enabled."); return true; } @@ -54,11 +53,11 @@ public class Command_plugincontrol extends FreedomCommand { if (!pm.isPluginEnabled(target)) { - playerMsg("Error enabling plugin " + target.getName()); + playerMsg(sender, "Error enabling plugin " + target.getName()); return true; } - playerMsg(target.getName() + " is now enabled."); + playerMsg(sender, target.getName() + " is now enabled."); return true; } @@ -67,19 +66,19 @@ public class Command_plugincontrol extends FreedomCommand { final Plugin target = getPlugin(args[1]); if (target == null) { - playerMsg("Plugin not found!"); + playerMsg(sender, "Plugin not found!"); return true; } if (!target.isEnabled()) { - playerMsg("Plugin is already disabled."); + playerMsg(sender, "Plugin is already disabled."); return true; } if (target.getName().equals(plugin.getName())) { - playerMsg("You cannot disable " + plugin.getName()); + playerMsg(sender, "You cannot disable " + plugin.getName()); return true; } @@ -87,11 +86,11 @@ public class Command_plugincontrol extends FreedomCommand { if (pm.isPluginEnabled(target)) { - playerMsg("Error disabling plugin " + target.getName()); + playerMsg(sender, "Error disabling plugin " + target.getName()); return true; } - playerMsg(target.getName() + " is now disabled."); + playerMsg(sender, target.getName() + " is now disabled."); return true; } @@ -100,19 +99,19 @@ public class Command_plugincontrol extends FreedomCommand { final Plugin target = getPlugin(args[1]); if (target == null) { - playerMsg("Plugin not found!"); + playerMsg(sender, "Plugin not found!"); return true; } if (target.getName().equals(plugin.getName())) { - playerMsg("Use /tfm reload to reload instead."); + playerMsg(sender, "Use /tfm reload to reload instead."); return true; } pm.disablePlugin(target); pm.enablePlugin(target); - playerMsg(target.getName() + " reloaded."); + playerMsg(sender, target.getName() + " reloaded."); return true; } diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_potion.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_potion.java index f08218f..84e797e 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_potion.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_potion.java @@ -14,9 +14,6 @@ import java.util.ArrayList; import java.util.List; @CommandPermissions(level = AdminLevel.OP, source = SourceType.BOTH) -@CommandParameters( - description = "Manipulate potion effects. Duration is measured in server ticks (~20 ticks per second).", - usage = "/ [target name]>") public class Command_potion extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -28,13 +25,13 @@ public class Command_potion extends FreedomCommand { potionEffectTypeNames.add(potion_effect_type.getName()); } } - playerMsg("Potion effect types: " + StringUtils.join(potionEffectTypeNames, ", "), ChatColor.AQUA); + playerMsg(sender, "Potion effect types: " + StringUtils.join(potionEffectTypeNames, ", "), ChatColor.AQUA); } else if (args[0].equalsIgnoreCase("clearall")) { if (!(AdminList.isSuperAdmin(sender) || senderIsConsole)) { - playerMsg(FreedomCommand.MSG_NO_PERMS); + playerMsg(sender, FreedomCommand.MSG_NO_PERMS); return true; } Utilities.adminAction(sender.getName(), "Cleared all potion effects from all players", true); @@ -56,7 +53,7 @@ public class Command_potion extends FreedomCommand { if (target == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); return true; } } @@ -65,13 +62,13 @@ public class Command_potion extends FreedomCommand { { if (!AdminList.isSuperAdmin(sender)) { - playerMsg("Only superadmins can clear potion effects from other players."); + playerMsg(sender, "Only superadmins can clear potion effects from other players."); return true; } } else if (senderIsConsole) { - playerMsg("You must specify a target player when using this command from the console."); + playerMsg(sender, "You must specify a target player when using this command from the console."); return true; } @@ -80,7 +77,7 @@ public class Command_potion extends FreedomCommand { target.removePotionEffect(potion_effect.getType()); } - playerMsg("Cleared all active potion effects " + (!target.equals(sender_p) ? "from player " + target.getName() + "." : "from yourself."), ChatColor.AQUA); + playerMsg(sender, "Cleared all active potion effects " + (!target.equals(sender_p) ? "from player " + target.getName() + "." : "from yourself."), ChatColor.AQUA); } else { @@ -100,7 +97,7 @@ public class Command_potion extends FreedomCommand { if (target == null) { - playerMsg(FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); + playerMsg(sender, FreedomCommand.PLAYER_NOT_FOUND, ChatColor.RED); return true; } } @@ -134,7 +131,7 @@ public class Command_potion extends FreedomCommand { } catch (NumberFormatException ex) { - playerMsg("Invalid potion duration.", ChatColor.RED); + playerMsg(sender, "Invalid potion duration.", ChatColor.RED); return true; } @@ -146,13 +143,13 @@ public class Command_potion extends FreedomCommand { } catch (NumberFormatException ex) { - playerMsg("Invalid potion amplifier.", ChatColor.RED); + playerMsg(sender, "Invalid potion amplifier.", ChatColor.RED); return true; } PotionEffect new_effect = potion_effect_type.createEffect(duration, amplifier); target.addPotionEffect(new_effect, true); - playerMsg( + playerMsg(sender, "Added potion effect: " + new_effect.getType().getName() + ", Duration: " + new_effect.getDuration() + ", Amplifier: " + new_effect.getAmplifier() diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_premium.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_premium.java index aea75e7..da5bf7f 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_premium.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_premium.java @@ -15,7 +15,6 @@ import java.net.URL; import java.net.URLConnection; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters(description = "Validates if a given account is premium.", usage = "/ ", aliases = "prem") public class Command_premium extends FreedomCommand { @Override public boolean run(CommandSender sender, org.bukkit.entity.Player sender_p, Command cmd, String commandLabel, String[] args, boolean senderIsConsole) { @@ -44,14 +43,14 @@ public class Command_premium extends FreedomCommand { urlConnection.disconnect(); Bukkit.getScheduler().runTask(plugin, () -> { - playerMsg("Player " + name + " is premium: " + message); + playerMsg(sender, "Player " + name + " is premium: " + message); }); } catch (Exception ex) { Log.severe(ex); Bukkit.getScheduler().runTask(plugin, () -> { - playerMsg("There was an error querying the mojang server.", ChatColor.RED); + playerMsg(sender, "There was an error querying the mojang server.", ChatColor.RED); }); } }); diff --git a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_protectarea.java b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_protectarea.java index 6e092f8..74713e4 100644 --- a/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_protectarea.java +++ b/src/main/java/me/StevenLawson/TotalFreedomMod/commands/Command_protectarea.java @@ -7,28 +7,25 @@ import org.bukkit.command.Command; import org.bukkit.command.CommandSender; @CommandPermissions(level = AdminLevel.SUPER, source = SourceType.BOTH) -@CommandParameters( - description = "Protect areas so that only superadmins can directly modify blocks in those areas. WorldEdit and other such plugins might bypass this.", - usage = "/ | add