Remove check for /say command

There are more commands in addition to /say that allow target selectors, and the exploit only targets Minecraft 1.12.2 and older. As those versions are over 3 years old, there isn't much point in supporting them.

src/main/java/pw/kaboom/extras/helpers/MessageInterceptingCommandRunner.java

@@ -1,167 +0,0 @@
-package pw.kaboom.extras.helpers;
-
-import java.util.Set;
-
-import javax.annotation.Nullable;
-
-import org.bukkit.Server;
-import org.bukkit.command.CommandSender;
-import org.bukkit.command.ConsoleCommandSender;
-import org.bukkit.conversations.Conversation;
-import org.bukkit.conversations.ConversationAbandonedEvent;
-import org.bukkit.permissions.Permission;
-import org.bukkit.permissions.PermissionAttachment;
-import org.bukkit.permissions.PermissionAttachmentInfo;
-import org.bukkit.plugin.Plugin;
-
-public class MessageInterceptingCommandRunner implements ConsoleCommandSender {
-    private final ConsoleCommandSender wrappedSender;
-    private final Spigot spigotWrapper;
-
-    private class Spigot extends CommandSender.Spigot {
-        /**
-         * Sends this sender a chat component.
-         *
-         * @param component the components to send
-         */
-        @Override
-		public final void sendMessage(final @NotNull net.md_5.bungee.api.chat.BaseComponent component) {
-            wrappedSender.spigot().sendMessage();
-        }
-
-        /**
-         * Sends an array of components as a single message to the sender.
-         *
-         * @param components the components to send
-         */
-        @Override
-		public final void sendMessage(final @NotNull net.md_5.bungee.api.chat.BaseComponent... components) {
-            wrappedSender.spigot().sendMessage(components);
-        }
-    }
-
-    public MessageInterceptingCommandRunner(final ConsoleCommandSender wrappedSenderIn) {
-        this.wrappedSender = wrappedSenderIn;
-        spigotWrapper = new Spigot();
-    }
-
-    @Override
-    public final void sendMessage(final @NotNull String message) {
-        wrappedSender.sendMessage(message.substring(0, Math.min(message.length(), 256)));
-    }
-
-    @Override
-    public final void sendMessage(final @NotNull String[] messages) {
-        wrappedSender.sendMessage(messages);
-    }
-
-    @Override
-    public final @NotNull Server getServer() {
-        return wrappedSender.getServer();
-    }
-
-    @Override
-    public final @NotNull String getName() {
-        return "OrderFulfiller";
-    }
-
-    @Override
-    public final @NotNull CommandSender.Spigot spigot() {
-        return spigotWrapper;
-    }
-
-    @Override
-    public final boolean isConversing() {
-        return wrappedSender.isConversing();
-    }
-
-    @Override
-    public final void acceptConversationInput(final @NotNull String input) {
-        wrappedSender.acceptConversationInput(input);
-    }
-
-    @Override
-    public final boolean beginConversation(final @NotNull Conversation conversation) {
-        return wrappedSender.beginConversation(conversation);
-    }
-
-    @Override
-    public final void abandonConversation(final @NotNull Conversation conversation) {
-        wrappedSender.abandonConversation(conversation);
-    }
-
-    @Override
-    public final void abandonConversation(final @NotNull Conversation conversation, final @NotNull ConversationAbandonedEvent details) {
-        wrappedSender.abandonConversation(conversation, details);
-    }
-
-    @Override
-    public final void sendRawMessage(final @NotNull String message) {
-        wrappedSender.sendRawMessage(message.substring(0, Math.min(message.length(), 256)));
-    }
-
-    @Override
-    public final boolean isPermissionSet(final @NotNull String name) {
-        return wrappedSender.isPermissionSet(name);
-    }
-
-    @Override
-    public final boolean isPermissionSet(final @NotNull Permission perm) {
-        return wrappedSender.isPermissionSet(perm);
-    }
-
-    @Override
-    public final boolean hasPermission(final @NotNull String name) {
-        return wrappedSender.hasPermission(name);
-    }
-
-    @Override
-    public final boolean hasPermission(final @NotNull Permission perm) {
-        return wrappedSender.hasPermission(perm);
-    }
-
-    @Override
-    public final @NotNull PermissionAttachment addAttachment(final @NotNull Plugin plugin, final @NotNull String name, final boolean value) {
-        return wrappedSender.addAttachment(plugin, name, value);
-    }
-
-    @Override
-    public final @NotNull PermissionAttachment addAttachment(final @NotNull Plugin plugin) {
-        return wrappedSender.addAttachment(plugin);
-    }
-
-    @Override
-    public final @Nullable PermissionAttachment addAttachment(final @NotNull Plugin plugin, final @NotNull String name, final boolean value, final int ticks) {
-        return wrappedSender.addAttachment(plugin, name, value, ticks);
-    }
-
-    @Override
-    public final @Nullable PermissionAttachment addAttachment(final @NotNull Plugin plugin, final int ticks) {
-        return wrappedSender.addAttachment(plugin, ticks);
-    }
-
-    @Override
-    public final void removeAttachment(final @NotNull PermissionAttachment attachment) {
-        wrappedSender.removeAttachment(attachment);
-    }
-
-    @Override
-    public final void recalculatePermissions() {
-        wrappedSender.recalculatePermissions();
-    }
-
-    @Override
-    public final @NotNull Set<PermissionAttachmentInfo> getEffectivePermissions() {
-        return wrappedSender.getEffectivePermissions();
-    }
-
-    @Override
-    public final boolean isOp() {
-        return wrappedSender.isOp();
-    }
-
-    @Override
-    public final void setOp(final boolean value) {
-        wrappedSender.setOp(value);
-    }
-}

src/main/java/pw/kaboom/extras/helpers/NotNull.java

@@ -1,5 +0,0 @@
-package pw.kaboom.extras.helpers;
-
-public @interface NotNull {
-
-}

src/main/java/pw/kaboom/extras/modules/server/ServerCommand.java

@@ -34,7 +34,8 @@ public final class ServerCommand implements Listener {
 			|| "tellraw".equalsIgnoreCase(cmd)
 			|| "tm".equalsIgnoreCase(cmd)
 			|| "tp".equalsIgnoreCase(cmd)
-											|| "w".equalsIgnoreCase(cmd));
+			|| "w".equalsIgnoreCase(cmd)
+		);
 	}
 	public static String checkCommand(final CommandSender sender, final String command, final boolean isConsoleCommand) {
 		final String[] arr = command.split(" ");
@@ -126,6 +127,12 @@ public final class ServerCommand implements Listener {
 						return String.join(" ", arr);
 					}
 					break;
+				case "/minecraft:title":
+				case "/title":
+					if (command.contains("selector")) {
+						return "cancel";
+					}
+					break;
 				case "/viaversion:viaver":
 				case "/viaversion:viaversion":
 				case "/viaversion:vvbukkit":
@@ -137,14 +144,6 @@ public final class ServerCommand implements Listener {
 						return "cancel";
 					}
 					break;
-				case "/minecraft:say":
-				case "/say":
-					for (int i = 0; i < arr.length; i++) {
-						if (arr[i].toLowerCase().contains("@")) {
-							return "cancel";
-						}
-					}
-					break;
 				default:
 					break;
 			}