Fixes that children get replaced twice if colors are included in repl…

…acements
Phoenix616 · Jan 16, 2022 · 110f941 · 110f941
1 parent 3605778
commit 110f941
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/main/java/de/themoep/minedown/adventure/Replacer.java b/src/main/java/de/themoep/minedown/adventure/Replacer.java
@@ -214,11 +214,13 @@ public Component replaceIn(Component component) {
                 component = ((TextComponent) component).content("");
                 List<Component> children = new ArrayList<>();
                 children.add(replacedComponent);
-                children.addAll(component.children());
+                children.addAll(replaceIn(component.children()));
                 component = component.children(children);
             } else {
-                component = ((TextComponent) component).content(replaced);
+                component = ((TextComponent) component).content(replaced).children(replaceIn(component.children()));
             }
+        } else if (!component.children().isEmpty()) {
+            component = component.children(replaceIn(component.children()));
         }
         if (component instanceof TranslatableComponent) {
             component = ((TranslatableComponent) component).key(replaceIn(((TranslatableComponent) component).key()));
@@ -259,8 +261,6 @@ public Component replaceIn(Component component) {
             }
         }
 
-        component = component.children(replaceIn(component.children()));
-
         // Component replacements
         List<Component> replacedComponents = new ArrayList<>();
         replacedComponents.add(component);