Fixed style not being copied

Lenni0451 · Sep 20, 2024 · 2ae02b7 · 2ae02b7
1 parent ab6890a
commit 2ae02b7
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 12 deletions.
diff --git a/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/KeybindComponent.java b/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/KeybindComponent.java
@@ -68,7 +68,7 @@ public ATextComponent copy() {
     public ATextComponent shallowCopy() {
         KeybindComponent copy = new KeybindComponent(this.keybind);
         copy.translator = this.translator;
-        return copy;
+        return copy.setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/ScoreComponent.java b/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/ScoreComponent.java
@@ -92,7 +92,7 @@ public ATextComponent copy() {
     public ATextComponent shallowCopy() {
         ScoreComponent copy = new ScoreComponent(this.name, this.objective);
         copy.value = this.value;
-        return copy;
+        return copy.setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/SelectorComponent.java b/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/SelectorComponent.java
@@ -70,8 +70,8 @@ public ATextComponent copy() {
 
     @Override
     public ATextComponent shallowCopy() {
-        if (this.separator == null) return new SelectorComponent(this.selector, null);
-        else return new SelectorComponent(this.selector, this.separator.copy());
+        if (this.separator == null) return new SelectorComponent(this.selector, null).setStyle(this.getStyle().copy());
+        else return new SelectorComponent(this.selector, this.separator.copy()).setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/StringComponent.java b/MCStructs-text/src/main/java/net/lenni0451/mcstructs/text/components/StringComponent.java
@@ -47,7 +47,7 @@ public ATextComponent copy() {
 
     @Override
     public ATextComponent shallowCopy() {
-        return new StringComponent(this.text);
+        return new StringComponent(this.text).setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/...ucts-text/src/main/java/net/lenni0451/mcstructs/text/components/TranslationComponent.java b/...ucts-text/src/main/java/net/lenni0451/mcstructs/text/components/TranslationComponent.java
@@ -154,7 +154,7 @@ public ATextComponent shallowCopy() {
         }
         TranslationComponent copy = new TranslationComponent(this.key, copyArgs);
         copy.translator = this.translator;
-        return copy;
+        return copy.setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/...cts-text/src/main/java/net/lenni0451/mcstructs/text/components/nbt/BlockNbtComponent.java b/...cts-text/src/main/java/net/lenni0451/mcstructs/text/components/nbt/BlockNbtComponent.java
@@ -45,8 +45,8 @@ public ATextComponent copy() {
 
     @Override
     public ATextComponent shallowCopy() {
-        if (this.getSeparator() == null) return new BlockNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator(), this.pos);
-        else return new BlockNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator().copy(), this.pos);
+        if (this.getSeparator() == null) return new BlockNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator(), this.pos).setStyle(this.getStyle().copy());
+        else return new BlockNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator().copy(), this.pos).setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/...ts-text/src/main/java/net/lenni0451/mcstructs/text/components/nbt/EntityNbtComponent.java b/...ts-text/src/main/java/net/lenni0451/mcstructs/text/components/nbt/EntityNbtComponent.java
@@ -45,8 +45,8 @@ public ATextComponent copy() {
 
     @Override
     public ATextComponent shallowCopy() {
-        if (this.getSeparator() == null) return new EntityNbtComponent(this.getComponent(), this.isResolve(), null, this.selector);
-        else return new EntityNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator(), this.selector);
+        if (this.getSeparator() == null) return new EntityNbtComponent(this.getComponent(), this.isResolve(), null, this.selector).setStyle(this.getStyle().copy());
+        else return new EntityNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator(), this.selector).setStyle(this.getStyle().copy());
     }
 
     @Override

diff --git a/...s-text/src/main/java/net/lenni0451/mcstructs/text/components/nbt/StorageNbtComponent.java b/...s-text/src/main/java/net/lenni0451/mcstructs/text/components/nbt/StorageNbtComponent.java
@@ -46,8 +46,8 @@ public ATextComponent copy() {
 
     @Override
     public ATextComponent shallowCopy() {
-        if (this.getSeparator() == null) return new StorageNbtComponent(this.getComponent(), this.isResolve(), null, this.id);
-        else return new StorageNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator(), this.id);
+        if (this.getSeparator() == null) return new StorageNbtComponent(this.getComponent(), this.isResolve(), null, this.id).setStyle(this.getStyle().copy());
+        else return new StorageNbtComponent(this.getComponent(), this.isResolve(), this.getSeparator(), this.id).setStyle(this.getStyle().copy());
     }
 
     @Override