Scissors/patches/unapplied/server/0008-Fixes-Knowledge-Books-causing-log-spam-when-invalid-.patch

From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Video <videogamesm12@gmail.com>
Date: Sun, 13 Mar 2022 18:42:07 -0600
Subject: [PATCH] Fixes Knowledge Books causing log spam when invalid data is
 provided


diff --git a/src/main/java/net/minecraft/world/item/KnowledgeBookItem.java b/src/main/java/net/minecraft/world/item/KnowledgeBookItem.java
index 37f37be56bab171df442b980ff46ff325daae283..deade19d16a7d6870171b9a60806a8cadb437db4 100644
--- a/src/main/java/net/minecraft/world/item/KnowledgeBookItem.java
+++ b/src/main/java/net/minecraft/world/item/KnowledgeBookItem.java
@@ -40,9 +40,9 @@ public class KnowledgeBookItem extends Item {
 
                 for(int i = 0; i < listTag.size(); ++i) {
                     String string = listTag.getString(i);
-                    Optional<RecipeHolder<?>> optional = recipeManager.byKey(new ResourceLocation(string));
+                    Optional<RecipeHolder<?>> optional = recipeManager.byKey(ResourceLocation.tryParse(string)); // Scissors - Validate resource locations
                     if (!optional.isPresent()) {
-                        LOGGER.error("Invalid recipe: {}", (Object)string);
+                        // Scissors - Don't log errors caused by invalid recipes being provided
                         return InteractionResultHolder.fail(itemStack);
                     }
 
@@ -55,7 +55,7 @@ public class KnowledgeBookItem extends Item {
 
             return InteractionResultHolder.sidedSuccess(itemStack, world.isClientSide());
         } else {
-            LOGGER.error("Tag not valid: {}", (Object)compoundTag);
+            // Scissors - Don't throw errors into the logs if an NBT compound isn't present or is missing the Recipes tag.
             return InteractionResultHolder.fail(itemStack);
         }
     }
diff --git a/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaKnowledgeBook.java b/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaKnowledgeBook.java
index 50fdb086ffec84edc5138737c95f08ed4757a6f3..da312b4670fc0ac07e4ab798d4793025e362783e 100644
--- a/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaKnowledgeBook.java
+++ b/src/main/java/org/bukkit/craftbukkit/inventory/CraftMetaKnowledgeBook.java
@@ -41,7 +41,11 @@ public class CraftMetaKnowledgeBook extends CraftMetaItem implements KnowledgeBo
             for (int i = 0; i < pages.size(); i++) {
                 String recipe = pages.getString(i);
 
-                this.addRecipe(CraftNamespacedKey.fromString(recipe));
+                // Scissors start - Don't add recipes with invalid namespaces
+                try {
+                    this.addRecipe(CraftNamespacedKey.fromString(recipe));
+                } catch (Exception ignored) {}
+                // Scissors end
             }
         }
     }