Merge remote-tracking branch 'origin/master'

Author: Johni0702

api/src/main/java/com/viaversion/viaversion/api/configuration/ViaVersionConfig.java

@@ -285,6 +285,13 @@ public interface ViaVersionConfig extends Config {
      */
     boolean isSuppressConversionWarnings();
 
+    /**
+     * Should we hide errors that occur when trying to convert text components?
+     *
+     * @return true if enabled
+     */
+    boolean isSuppressTextComponentConversionWarnings();
+
     /**
      * Should we disable the 1.13 auto-complete feature to stop spam kicks? (for any server lower than 1.13)
      *

api/src/main/java/com/viaversion/viaversion/api/minecraft/BlockChangeRecord1_16_2.java

@@ -58,7 +58,7 @@ public byte getSectionZ() {
 
     @Override
     public short getY(int chunkSectionY) {
-        Preconditions.checkArgument(chunkSectionY >= 0, "Invalid chunkSectionY: " + chunkSectionY);
+        Preconditions.checkArgument(chunkSectionY >= 0, "Invalid chunkSectionY: %s", chunkSectionY);
         return (short) ((chunkSectionY << 4) + sectionY);
     }
 

api/src/main/java/com/viaversion/viaversion/api/minecraft/Holder.java

@@ -23,6 +23,7 @@
 package com.viaversion.viaversion.api.minecraft;
 
 import it.unimi.dsi.fastutil.ints.Int2IntFunction;
+import java.util.function.Function;
 
 public interface Holder<T> {
 
@@ -89,4 +90,12 @@ static <T> Holder<T> of(final T value) {
      * @return a new holder with the id rewritten, or self
      */
     Holder<T> updateId(final Int2IntFunction rewriteFunction);
+
+    /**
+     * Returns a new holder with the value rewritten using the given function, or self if this is an id holder or the value did not change.
+     *
+     * @param rewriteFunction the function to rewrite the value
+     * @return a new holder with the value rewritten, or self
+     */
+    Holder<T> updateValue(final Function<T, T> rewriteFunction);
 }

api/src/main/java/com/viaversion/viaversion/api/minecraft/IdHolder.java

@@ -24,6 +24,7 @@
 
 import com.google.common.base.Preconditions;
 import it.unimi.dsi.fastutil.ints.Int2IntFunction;
+import java.util.function.Function;
 
 record IdHolder<T>(int id) implements Holder<T> {
 
@@ -57,4 +58,9 @@ public Holder<T> updateId(final Int2IntFunction rewriteFunction) {
         }
         return Holder.of(rewrittenId);
     }
+
+    @Override
+    public Holder<T> updateValue(final Function<T, T> rewriteFunction) {
+        return this;
+    }
 }

api/src/main/java/com/viaversion/viaversion/api/minecraft/Particle.java

@@ -121,7 +121,7 @@ public void write(final PacketWrapper wrapper) {
 
         @Override
         public ParticleData<T> copy() {
-            return new ParticleData<>(type, copy(value));
+            return new ParticleData<>(type, Copyable.copy(value));
         }
 
         @Override

api/src/main/java/com/viaversion/viaversion/api/minecraft/ValueHolder.java

@@ -23,6 +23,7 @@
 package com.viaversion.viaversion.api.minecraft;
 
 import it.unimi.dsi.fastutil.ints.Int2IntFunction;
+import java.util.function.Function;
 
 record ValueHolder<T>(T value) implements Holder<T> {
 
@@ -45,4 +46,10 @@ public int id() {
     public Holder<T> updateId(final Int2IntFunction rewriteFunction) {
         return this;
     }
+
+    @Override
+    public Holder<T> updateValue(final Function<T, T> rewriteFunction) {
+        final T rewrittenValue = rewriteFunction.apply(value);
+        return rewrittenValue != value ? Holder.of(rewrittenValue) : this;
+    }
 }

api/src/main/java/com/viaversion/viaversion/api/minecraft/data/FilledStructuredData.java

@@ -23,6 +23,7 @@
 package com.viaversion.viaversion.api.minecraft.data;
 
 import com.google.common.base.Preconditions;
+import com.viaversion.viaversion.util.Copyable;
 import io.netty.buffer.ByteBuf;
 import java.util.Objects;
 
@@ -61,7 +62,7 @@ public StructuredDataKey<T> key() {
 
     @Override
     public StructuredData<T> copy() {
-        return new FilledStructuredData<>(this.key, this.copy(this.value), this.id);
+        return new FilledStructuredData<>(this.key, Copyable.copy(this.value), this.id);
     }
 
     @Override

api/src/main/java/com/viaversion/viaversion/api/minecraft/item/data/AdventureModePredicate.java

@@ -22,14 +22,17 @@
  */
 package com.viaversion.viaversion.api.minecraft.item.data;
 
+import com.viaversion.viaversion.api.connection.UserConnection;
 import com.viaversion.viaversion.api.minecraft.data.StructuredData;
+import com.viaversion.viaversion.api.protocol.Protocol;
 import com.viaversion.viaversion.api.type.Type;
 import com.viaversion.viaversion.api.type.types.ArrayType;
 import com.viaversion.viaversion.util.Copyable;
+import com.viaversion.viaversion.util.Rewritable;
 import io.netty.buffer.ByteBuf;
-import it.unimi.dsi.fastutil.ints.Int2IntFunction;
 
-public record AdventureModePredicate(BlockPredicate[] predicates, boolean showInTooltip) implements Copyable {
+public record AdventureModePredicate(BlockPredicate[] predicates,
+                                     boolean showInTooltip) implements Copyable, Rewritable {
 
     public AdventureModePredicate(final BlockPredicate[] predicates) {
         this(predicates, true);
@@ -70,16 +73,17 @@ public void write(final ByteBuf buffer, final AdventureModePredicate value) {
         }
     }
 
-    public AdventureModePredicate rewrite(final Int2IntFunction blockIdRewriter) {
+    @Override
+    public AdventureModePredicate rewrite(final UserConnection connection, final Protocol<?, ?, ?, ?> protocol, final boolean clientbound) {
         final BlockPredicate[] predicates = new BlockPredicate[this.predicates.length];
         for (int i = 0; i < predicates.length; i++) {
-            predicates[i] = this.predicates[i].rewrite(blockIdRewriter);
+            predicates[i] = this.predicates[i].rewrite(connection, protocol, clientbound);
         }
         return new AdventureModePredicate(predicates, showInTooltip);
     }
 
     @Override
     public AdventureModePredicate copy() {
-        return new AdventureModePredicate(copy(predicates), showInTooltip);
+        return new AdventureModePredicate(Copyable.copy(predicates), showInTooltip);
     }
 }

api/src/main/java/com/viaversion/viaversion/api/minecraft/item/data/ArmorTrim.java

@@ -22,12 +22,15 @@
  */
 package com.viaversion.viaversion.api.minecraft.item.data;
 
+import com.viaversion.viaversion.api.connection.UserConnection;
 import com.viaversion.viaversion.api.minecraft.Holder;
+import com.viaversion.viaversion.api.protocol.Protocol;
 import com.viaversion.viaversion.api.type.Type;
+import com.viaversion.viaversion.util.Rewritable;
 import io.netty.buffer.ByteBuf;
-import it.unimi.dsi.fastutil.ints.Int2IntFunction;
 
-public record ArmorTrim(Holder<ArmorTrimMaterial> material, Holder<ArmorTrimPattern> pattern, boolean showInTooltip) {
+public record ArmorTrim(Holder<ArmorTrimMaterial> material, Holder<ArmorTrimPattern> pattern,
+                        boolean showInTooltip) implements Rewritable {
 
     public ArmorTrim(Holder<ArmorTrimMaterial> material, Holder<ArmorTrimPattern> pattern) {
         this(material, pattern, true);
@@ -96,15 +99,16 @@ public void write(final ByteBuf buffer, final ArmorTrim value) {
         }
     };
 
-    public ArmorTrim rewrite(final Int2IntFunction idRewriteFunction) {
+    @Override
+    public ArmorTrim rewrite(final UserConnection connection, final Protocol<?, ?, ?, ?> protocol, final boolean clientbound) {
         Holder<ArmorTrimMaterial> material = this.material;
         if (material.isDirect()) {
-            material = Holder.of(material.value().rewrite(idRewriteFunction));
+            material = Holder.of(material.value().rewrite(connection, protocol, clientbound));
         }
 
         Holder<ArmorTrimPattern> pattern = this.pattern;
         if (pattern.isDirect()) {
-            pattern = Holder.of(pattern.value().rewrite(idRewriteFunction));
+            pattern = Holder.of(pattern.value().rewrite(connection, protocol, clientbound));
         }
         return new ArmorTrim(material, pattern, showInTooltip);
     }

api/src/main/java/com/viaversion/viaversion/api/minecraft/item/data/ArmorTrimMaterial.java

@@ -23,16 +23,19 @@
 package com.viaversion.viaversion.api.minecraft.item.data;
 
 import com.viaversion.nbt.tag.Tag;
+import com.viaversion.viaversion.api.connection.UserConnection;
+import com.viaversion.viaversion.api.protocol.Protocol;
 import com.viaversion.viaversion.api.type.Types;
 import com.viaversion.viaversion.api.type.types.misc.HolderType;
 import com.viaversion.viaversion.util.Copyable;
+import com.viaversion.viaversion.util.Rewritable;
 import io.netty.buffer.ByteBuf;
-import it.unimi.dsi.fastutil.ints.Int2IntFunction;
 import it.unimi.dsi.fastutil.objects.Object2ObjectArrayMap;
 import java.util.Map;
 
 public record ArmorTrimMaterial(String assetName, int itemId, float itemModelIndex,
-                                Map<String, String> overrideArmorMaterials, Tag description) implements Copyable {
+                                Map<String, String> overrideArmorMaterials,
+                                Tag description) implements Copyable, Rewritable {
 
     public ArmorTrimMaterial(final String assetName, final int itemId, final Map<String, String> overrideArmorMaterials, final Tag description) {
         this(assetName, itemId, 0F, overrideArmorMaterials, description);
@@ -175,8 +178,9 @@ public void writeDirect(final ByteBuf buffer, final ArmorTrimMaterial value) {
         }
     };
 
-    public ArmorTrimMaterial rewrite(final Int2IntFunction idRewriteFunction) {
-        return new ArmorTrimMaterial(assetName, idRewriteFunction.applyAsInt(itemId), itemModelIndex, overrideArmorMaterials, description);
+    @Override
+    public ArmorTrimMaterial rewrite(final UserConnection connection, final Protocol<?, ?, ?, ?> protocol, final boolean clientbound) {
+        return new ArmorTrimMaterial(assetName, Rewritable.rewriteItem(protocol, clientbound, itemId), itemModelIndex, overrideArmorMaterials, description);
     }
 
     @Override