improve disallowed character check to also check compatibility with the

db collation

Author: zefir-git

src/main/java/pro/cloudnode/smp/bankaccounts/BankConfig.java

@@ -24,6 +24,7 @@
 import java.util.List;
 import java.util.Objects;
 import java.util.Optional;
+import java.util.Set;
 import java.util.TimeZone;
 
 public final class BankConfig {
@@ -624,10 +625,10 @@ public int invoicePerPage() {
     }
 
     // messages.errors.disallowed-characters
-    public @NotNull Component messagesErrorsDisallowedCharacters(final @NotNull String characters) {
+    public @NotNull Component messagesErrorsDisallowedCharacters(final @NotNull Set<@NotNull String> characters) {
         return MiniMessage.miniMessage().deserialize(
                 Objects.requireNonNull(config.getString("messages.errors.disallowed-characters")),
-                Placeholder.unparsed("characters", characters)
+                Placeholder.unparsed("characters", String.join("", characters))
         );
     }
 

src/main/java/pro/cloudnode/smp/bankaccounts/Command.java

@@ -12,6 +12,8 @@
 
 import java.util.List;
 import java.util.Optional;
+import java.util.Set;
+import java.util.stream.Collectors;
 
 public abstract class Command implements CommandExecutor, TabCompleter {
     /**
@@ -81,4 +83,18 @@ public final boolean onCommand(final @NotNull CommandSender sender, final @NotNu
         final @Nullable List<@NotNull String> suggestions = tab(sender, args);
         return Optional.ofNullable(suggestions).map(s -> s.stream().filter(suggestion -> suggestion.toLowerCase().startsWith(args[args.length - 1].toLowerCase())).toList()).orElse(null);
     }
+
+    protected static @NotNull Set<@NotNull String> getDisallowedCharacters(final @Nullable String input) {
+        if (input == null) return Set.of();
+        final @NotNull Set<@NotNull String> chars = input
+                .codePoints()
+                .filter(codePoint -> codePoint > 0xFFFF)
+                .mapToObj(codePoint -> new String(Character.toChars(codePoint)))
+                .collect(Collectors.toSet());
+        if (input.contains("<"))
+            chars.add("<");
+        if (input.contains(">"))
+            chars.add(">");
+        return chars;
+    }
 }

src/main/java/pro/cloudnode/smp/bankaccounts/commands/BankCommand.java

@@ -20,6 +20,7 @@
 import java.util.Arrays;
 import java.util.Objects;
 import java.util.Optional;
+import java.util.Set;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
@@ -382,8 +383,9 @@ public static boolean setName(final @NotNull CommandSender sender, final @NotNul
             name = name.length() > 32 ? name.substring(0, 32) : name;
             name = name.isEmpty() ? null : name;
 
-            if (name != null && (name.contains("<") || name.contains(">")))
-                return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters("<>"));
+            final @NotNull Set<@NotNull String> disallowedChars = getDisallowedCharacters(name);
+            if (!disallowedChars.isEmpty())
+                return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters(disallowedChars));
 
             account.get().name = name;
             account.get().update();
@@ -501,8 +503,9 @@ public static boolean transfer(final @NotNull CommandSender sender, final @NotNu
                 .join(" ", Arrays.copyOfRange(argsCopy, 3, argsCopy.length)).trim() : null;
         if (description != null && description.length() > 64) description = description.substring(0, 64);
 
-        if (description != null && (description.contains("<") || description.contains(">")))
-            return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters("<>"));
+        final @NotNull Set<@NotNull String> disallowedChars = getDisallowedCharacters(description);
+        if (!disallowedChars.isEmpty())
+            return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters(disallowedChars));
 
         if (!confirm && BankAccounts.getInstance().config().transferConfirmationEnabled()) {
             final @NotNull BigDecimal minAmount = BankAccounts.getInstance().config().transferConfirmationMinAmount();

src/main/java/pro/cloudnode/smp/bankaccounts/commands/InvoiceCommand.java

@@ -17,6 +17,7 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.Optional;
+import java.util.Set;
 import java.util.stream.Stream;
 
 public final class InvoiceCommand extends Command {
@@ -177,6 +178,9 @@ public static boolean create(final @NotNull CommandSender sender, @NotNull Strin
             return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsFrozen(account.get()));
 
         final @Nullable String description = argsCopy.length < 3 ? null : String.join(" ", Arrays.copyOfRange(argsCopy, 2, argsCopy.length));
+        final @NotNull Set<@NotNull String> disallowedChars = getDisallowedCharacters(description);
+        if (!disallowedChars.isEmpty())
+            return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters(disallowedChars));
 
         final @NotNull Invoice invoice = new Invoice(account.get(), amount, description, target);
         invoice.insert();

src/main/java/pro/cloudnode/smp/bankaccounts/commands/POSCommand.java

@@ -20,6 +20,7 @@
 import java.util.Arrays;
 import java.util.Date;
 import java.util.Optional;
+import java.util.Set;
 
 /**
  * Create a POS at the location the player is looking at.
@@ -78,8 +79,9 @@ public boolean execute(final @NotNull CommandSender sender, final @NotNull Strin
         @Nullable String description = args.length > 2 ? String.join(" ", Arrays.copyOfRange(args, 2, args.length)) : null;
         if (description != null && description.length() > 64) description = description.substring(0, 64);
 
-        if (description != null && (description.contains("<") || description.contains(">")))
-            return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters("<>"));
+        final @NotNull Set<@NotNull String> disallowedChars = getDisallowedCharacters(description);
+        if (!disallowedChars.isEmpty())
+            return sendMessage(sender, BankAccounts.getInstance().config().messagesErrorsDisallowedCharacters(disallowedChars));
 
         final @NotNull POS pos = new POS(target.getLocation(), price, description, account.get(), new Date());
         pos.save();