Start deprecating legacy code

Author: APickledWalrus

src/main/java/ch/njol/skript/bukkitutil/CommandReloader.java

@@ -10,7 +10,9 @@
 
 /**
  * Utilizes CraftServer with reflection to re-send commands to clients.
+ * @deprecated This behavior should not be relied upon. There is no replacement.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class CommandReloader {
 
 	@Nullable

src/main/java/ch/njol/skript/command/Argument.java

@@ -22,7 +22,10 @@
  * Represents an argument of a command
  * 
  * @author Peter Güttinger
+ * @deprecated There is no direct replacement for this class.
+ * The closest alternative is {@link org.skriptlang.skript.bukkit.command.custom.ArgumentData}.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class Argument<T> {
 
 	@Nullable

src/main/java/ch/njol/skript/command/CommandUsage.java

@@ -10,7 +10,9 @@
 /**
  * Holds info about the usage of a command.
  * TODO: replace with record when java 17
+ * @deprecated There is no direct replacement for this class.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class CommandUsage {
 
 	/**

src/main/java/ch/njol/skript/command/Commands.java

@@ -22,8 +22,6 @@
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
 import org.bukkit.event.player.AsyncPlayerChatEvent;
-import org.bukkit.event.player.PlayerCommandPreprocessEvent;
-import org.bukkit.event.server.ServerCommandEvent;
 import org.bukkit.help.HelpMap;
 import org.bukkit.help.HelpTopic;
 import org.bukkit.plugin.SimplePluginManager;
@@ -125,46 +123,6 @@ public static String unescape(String string) {
 		return "" + unescape.matcher(string).replaceAll("$0");
 	}
 
-	private final static Listener commandListener = new Listener() {
-
-		@EventHandler(priority = EventPriority.HIGHEST, ignoreCancelled = true)
-		public void onPlayerCommand(PlayerCommandPreprocessEvent event) {
-			// Spigot will simply report that the command doesn't exist if a player does not have permission to use it.
-			// This is good security but, well, it's a breaking change for Skript. So we need to check for permissions
-			// ourselves and handle those messages, for every command.
-
-			// parse command, see if it's a skript command
-			String[] cmd = event.getMessage().substring(1).split("\\s+", 2);
-			String label = cmd[0].toLowerCase(Locale.ENGLISH);
-			String arguments = cmd.length == 1 ? "" : "" + cmd[1];
-			ScriptCommand command = commands.get(label);
-
-			// is it a skript command?
-			if (command != null) {
-				// if so, check permissions to handle ourselves
-				if (!command.checkPermissions(event.getPlayer(), label, arguments))
-					event.setCancelled(true);
-
-				// we can also handle case sensitivity here:
-				if (SkriptConfig.caseInsensitiveCommands.value()) {
-					cmd[0] = event.getMessage().charAt(0) + label;
-					event.setMessage(String.join(" ", cmd));
-				}
-			}
-		}
-
-		@SuppressWarnings("null")
-		@EventHandler(priority = EventPriority.HIGHEST)
-		public void onServerCommand(ServerCommandEvent event) {
-			if (event.getCommand().isEmpty() || event.isCancelled())
-				return;
-			if ((Skript.testing() || SkriptConfig.enableEffectCommands.value()) && event.getCommand().startsWith(SkriptConfig.effectCommandToken.value())) {
-				if (handleEffectCommand(event.getSender(), event.getCommand()))
-					event.setCancelled(true);
-			}
-		}
-	};
-
 	static boolean handleEffectCommand(CommandSender sender, String command) {
 		if (!(Skript.testing() || sender instanceof ConsoleCommandSender || sender.hasPermission("skript.effectcommands") || SkriptConfig.allowOpsToUseEffectCommands.value() && sender.isOp()))
 			return false;
@@ -279,8 +237,6 @@ public static void unregisterCommand(ScriptCommand scriptCommand) {
 
 	public static void registerListeners() {
 		if (!registeredListeners) {
-			Bukkit.getPluginManager().registerEvents(commandListener, Skript.getInstance());
-
 			Bukkit.getPluginManager().registerEvents(new Listener() {
 				@EventHandler(priority = EventPriority.LOWEST, ignoreCancelled = true)
 				public void onPlayerChat(AsyncPlayerChatEvent event) {

src/main/java/ch/njol/skript/command/ScriptCommand.java

@@ -64,7 +64,10 @@
 
 /**
  * This class is used for user-defined commands.
+ * @deprecated There is no direct replacement for this command.
+ * The closest alternative is {@link org.skriptlang.skript.bukkit.command.custom.ScriptBrigadierCommand}.
  */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class ScriptCommand implements TabExecutor {
 
 	public final static Message m_executable_by_players = new Message("commands.executable by players");
@@ -332,7 +335,7 @@ public boolean execute(final CommandSender sender, final String commandLabel, fi
 	boolean execute2(final ScriptCommandEvent event, final CommandSender sender, final String commandLabel, final String rest) {
 		final ParseLogHandler log = SkriptLogger.startParseLogHandler();
 		try {
-			final boolean ok = SkriptParser.parseArguments(rest, ScriptCommand.this, event);
+			final boolean ok = false;
 			if (!ok) {
 				final LogEntry e = log.getError();
 				if (e != null)

src/main/java/ch/njol/skript/command/ScriptCommandEvent.java

@@ -6,6 +6,11 @@
 import org.bukkit.entity.Player;
 import org.bukkit.event.HandlerList;
 
+/**
+ * @deprecated There is no direct replacement for this class.
+ * The closest alternative is {@link org.skriptlang.skript.bukkit.command.custom.ScriptCommandEvent}.
+ */
+@Deprecated(since = "INSERT VERSION", forRemoval = true)
 public class ScriptCommandEvent extends CommandEvent {
 
 	private final ScriptCommand scriptCommand;

src/main/java/ch/njol/skript/command/package-info.java

@@ -1,25 +1 @@
-/**
- *   This file is part of Skript.
- *
- *  Skript is free software: you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation, either version 3 of the License, or
- *  (at your option) any later version.
- *
- *  Skript is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with Skript.  If not, see <http://www.gnu.org/licenses/>.
- *
- * Copyright Peter Güttinger, SkriptLang team and contributors
- */
-/**
- * Code related to handling commands, either Skript commands or custom script commands.
- * 
- * @author Peter Güttinger
- */
 package ch.njol.skript.command;
-

src/main/java/ch/njol/skript/effects/EffScriptFile.java

@@ -4,7 +4,6 @@
 import ch.njol.skript.ScriptLoader.ScriptInfo;
 import ch.njol.skript.Skript;
 import ch.njol.skript.SkriptCommand;
-import ch.njol.skript.command.ScriptCommand;
 import ch.njol.skript.doc.Description;
 import ch.njol.skript.doc.Example;
 import ch.njol.skript.doc.Name;

src/main/java/ch/njol/skript/lang/SkriptParser.java

@@ -5,10 +5,7 @@
 import ch.njol.skript.SkriptConfig;
 import ch.njol.skript.classes.ClassInfo;
 import ch.njol.skript.classes.Parser;
-import ch.njol.skript.command.Argument;
 import ch.njol.skript.command.Commands;
-import ch.njol.skript.command.ScriptCommand;
-import ch.njol.skript.command.ScriptCommandEvent;
 import ch.njol.skript.expressions.ExprParse;
 import ch.njol.skript.lang.DefaultExpressionUtils.DefaultExpressionError;
 import ch.njol.skript.lang.function.ExprFunctionCall;
@@ -1217,30 +1214,6 @@ public <T> org.skriptlang.skript.common.function.FunctionReference<T> parseFunct
 		return new FunctionReference<>(newReference.name(), null, newReference.namespace(), types, expressions);
 	}
 
-	/*
-	 * Command parsing
-	 */
-
-	/**
-	 * Prints parse errors (i.e. must start a ParseLog before calling this method)
-	 */
-	public static boolean parseArguments(String args, ScriptCommand command, ScriptCommandEvent event) {
-		SkriptParser parser = new SkriptParser(args, PARSE_LITERALS, ParseContext.COMMAND);
-		ParseResult parseResult = parser.parse_i(command.getPattern());
-		if (parseResult == null)
-			return false;
-
-		List<Argument<?>> arguments = command.getArguments();
-		assert arguments.size() == parseResult.exprs.length;
-		for (int i = 0; i < parseResult.exprs.length; i++) {
-			if (parseResult.exprs[i] == null)
-				arguments.get(i).setToDefault(event);
-			else
-				arguments.get(i).set(event, parseResult.exprs[i].getArray(event));
-		}
-		return true;
-	}
-
 	/*
 	 * Utility methods
 	 */