Ignore errors in book conversion

Author: kennytv

common/src/main/java/com/viaversion/viaversion/protocol/ProtocolPipelineImpl.java

@@ -107,21 +107,22 @@ public void transform(Direction direction, State state, PacketWrapper packetWrap
         int originalID = packetWrapper.getId();
 
         DebugHandler debugHandler = Via.getManager().debugHandler();
-        if (debugHandler.enabled() && !debugHandler.logPostPacketTransform() && debugHandler.shouldLog(packetWrapper, direction)) {
+        boolean debug = debugHandler.enabled();
+        if (debug && !debugHandler.logPostPacketTransform() && debugHandler.shouldLog(packetWrapper, direction)) {
             logPacket(direction, state, packetWrapper, originalID);
         }
 
         // Apply protocols
         packetWrapper.apply(direction, state, 0, protocolListFor(direction));
         super.transform(direction, state, packetWrapper);
 
-        if (debugHandler.enabled() && debugHandler.logPostPacketTransform() && debugHandler.shouldLog(packetWrapper, direction)) {
+        if (debug && debugHandler.logPostPacketTransform() && debugHandler.shouldLog(packetWrapper, direction)) {
             logPacket(direction, state, packetWrapper, originalID);
         }
     }
 
     private List<Protocol> protocolListFor(final Direction direction) {
-        return Collections.unmodifiableList(direction == Direction.SERVERBOUND ? protocolList : reversedProtocolList);
+        return direction == Direction.SERVERBOUND ? protocolList : reversedProtocolList;
     }
 
     private void logPacket(Direction direction, State state, PacketWrapper packetWrapper, int originalID) {

common/src/main/java/com/viaversion/viaversion/protocols/protocol1_20_3to1_20_2/rewriter/BlockItemPacketRewriter1_20_3.java

@@ -22,6 +22,7 @@
 import com.github.steveice10.opennbt.tag.builtin.StringTag;
 import com.github.steveice10.opennbt.tag.builtin.Tag;
 import com.google.gson.JsonElement;
+import com.viaversion.viaversion.api.Via;
 import com.viaversion.viaversion.api.data.ParticleMappings;
 import com.viaversion.viaversion.api.minecraft.Particle;
 import com.viaversion.viaversion.api.minecraft.item.Item;
@@ -160,8 +161,14 @@ private void updatePages(final CompoundTag tag, final String key) {
             }
 
             final StringTag stringTag = (StringTag) pageTag;
-            final JsonElement updatedComponent = ComponentUtil.convertJson(stringTag.getValue(), ComponentUtil.SerializerVersion.V1_19_4, ComponentUtil.SerializerVersion.V1_20_3);
-            stringTag.setValue(updatedComponent.toString());
+            try {
+                final JsonElement updatedComponent = ComponentUtil.convertJson(stringTag.getValue(), ComponentUtil.SerializerVersion.V1_19_4, ComponentUtil.SerializerVersion.V1_20_3);
+                stringTag.setValue(updatedComponent.toString());
+            } catch (final Exception e) {
+                if (Via.getManager().isDebug()) {
+                    e.printStackTrace();
+                }
+            }
         }
     }
 }

common/src/main/java/com/viaversion/viaversion/rewriter/EntityRewriter.java

@@ -82,7 +82,7 @@ protected EntityRewriter(T protocol, boolean trackMappedType) {
     /**
      * Returns a meta filter builder.
      * <p>
-     * Calling {@link MetaFilter.Builder#register()} will automatically register the filter on this rewriter.
+     * Calling {@link com.viaversion.viaversion.rewriter.meta.MetaFilter.Builder#register()} will automatically register the filter on this rewriter.
      *
      * @return meta filter builder
      */
@@ -92,7 +92,7 @@ public MetaFilter.Builder filter() {
 
     /**
      * Registers a metadata filter.
-     * Note that {@link MetaFilter.Builder#register()} already calls this method.
+     * Note that {@link com.viaversion.viaversion.rewriter.meta.MetaFilter.Builder#register()} already calls this method.
      *
      * @param filter filter to register
      * @throws IllegalArgumentException if the filter is already registered