Fix multiple username transformation (#683)

Author: barr-israel

src/main/java/codes/biscuit/skyblockaddons/listeners/PlayerListener.java

@@ -511,11 +511,13 @@ private void playerSymbolsDisplay(ClientChatReceivedEvent e, String unformattedT
                 if(!suffix.equals(" ")) {
                     String finalSuffix = suffix;
                     String finalUsername = username;
-                    TextUtils.recursiveTransformChatComponent(oldMessage, component -> {
+                    TextUtils.transformAnyChatComponent(oldMessage, component -> {
                                 if (component instanceof ChatComponentText & ((ChatComponentText)component).text.contains(finalUsername)) {
                                     ChatComponentText textComponent = (ChatComponentText) component;
                                     textComponent.text = textComponent.text.replace(finalUsername, finalUsername + finalSuffix);
+                                    return true;
                                 }
+                                return false;
                             }
                     );
                 }

src/main/java/codes/biscuit/skyblockaddons/utils/TextUtils.java

@@ -9,6 +9,7 @@
 import java.text.ParseException;
 import java.util.*;
 import java.util.function.Consumer;
+import java.util.function.Predicate;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
@@ -453,10 +454,28 @@ private static String mergeFormats(String firstFormat, String secondFormat) {
      * @param action action to be performed
      * @author Sychic
      */
-    public static void recursiveTransformChatComponent(IChatComponent chatComponent, Consumer<IChatComponent> action) {
+    public static void transformAllChatComponents(IChatComponent chatComponent, Consumer<IChatComponent> action) {
         action.accept(chatComponent);
         for (IChatComponent sibling : chatComponent.getSiblings()) {
-            recursiveTransformChatComponent(sibling, action);
+            transformAllChatComponents(sibling, action);
         }
     }
+
+    /**
+     * Recursively searches for a chat component to transform based on a given Predicate.
+     *
+     * Important to note that this function will stop on the first successful transformation, unlike {@link #transformAllChatComponents(IChatComponent, Consumer)}
+     * @param chatComponent root chat component
+     * @param action predicate that transforms a component and reports a successful transformation
+     * @return Whether any transformation occurred
+     */
+    public static boolean transformAnyChatComponent(IChatComponent chatComponent, Predicate<IChatComponent> action) {
+        if(action.test(chatComponent))
+            return true;
+        for (IChatComponent sibling : chatComponent.getSiblings()) {
+            if(transformAnyChatComponent(sibling, action))
+                return true;
+        }
+        return false;
+    }
 }