Ensure placeholder replacers process in sequence

Author: WiIIiam278

common/src/main/java/net/william278/huskchat/HuskChat.java

@@ -84,12 +84,11 @@ default void loadConfig() {
     List<PlaceholderReplacer> getPlaceholderReplacers();
 
     default CompletableFuture<String> replacePlaceholders(@NotNull Player player, @NotNull String message) {
-        return getPlaceholderReplacers().stream()
-                .map(replacer -> replacer.formatPlaceholders(message, player))
-                .reduce(
-                        CompletableFuture.completedFuture(message),
-                        (a, b) -> a.thenCombine(b, (x, y) -> y)
-                );
+        CompletableFuture<String> future = CompletableFuture.completedFuture(message);
+        for (PlaceholderReplacer replacer : getPlaceholderReplacers()) {
+            future = future.thenComposeAsync(toFormat -> replacer.formatPlaceholders(toFormat, player));
+        }
+        return future;
     }
 
     @NotNull

common/src/main/java/net/william278/huskchat/placeholders/DefaultReplacer.java

@@ -26,6 +26,7 @@
 import java.text.SimpleDateFormat;
 import java.util.Date;
 import java.util.HashSet;
+import java.util.Locale;
 import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 import java.util.function.BiFunction;
@@ -131,7 +132,7 @@ public enum Placeholder {
 
         Placeholder(@NotNull BiFunction<HuskChat, Player, String> replacer, @NotNull String... aliases) {
             this.replacer = replacer;
-            this.aliases.add(this.name());
+            this.aliases.add(this.name().toLowerCase(Locale.ENGLISH));
             this.aliases.addAll(Set.of(aliases));
         }