From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 From: Jake Potrebic Date: Sat, 21 Aug 2021 17:25:38 -0700 Subject: [PATCH] API for updating recipes on clients diff --git a/src/main/java/net/minecraft/server/players/PlayerList.java b/src/main/java/net/minecraft/server/players/PlayerList.java index 80919d247a9204ebf8d0e55df7fc6a2443ed91aa..3a70b7e1319c3ecab9eb720f8a1a34c0efe21a4b 100644 --- a/src/main/java/net/minecraft/server/players/PlayerList.java +++ b/src/main/java/net/minecraft/server/players/PlayerList.java @@ -1539,6 +1539,13 @@ public abstract class PlayerList { } public void reloadResources() { + // Paper start - split this method up into separate methods + this.reloadAdvancementData(); + this.reloadTagData(); + this.reloadRecipeData(); + } + public void reloadAdvancementData() { + // Paper end // CraftBukkit start /*Iterator iterator = this.advancements.values().iterator(); @@ -1554,7 +1561,15 @@ public abstract class PlayerList { } // CraftBukkit end + // Paper start + } + public void reloadTagData() { + // Paper end this.broadcastAll(new ClientboundUpdateTagsPacket(TagNetworkSerialization.serializeTagsToNetwork(this.registries))); + // Paper start + } + public void reloadRecipeData() { + // Paper end ClientboundUpdateRecipesPacket packetplayoutrecipeupdate = new ClientboundUpdateRecipesPacket(this.server.getRecipeManager().getRecipes()); Iterator iterator1 = this.players.iterator(); diff --git a/src/main/java/org/bukkit/craftbukkit/CraftServer.java b/src/main/java/org/bukkit/craftbukkit/CraftServer.java index 16906e8ba7d05275561d465a08b792137d284c4e..6a8f954f38007f47139da348a611e4eb2bc09289 100644 --- a/src/main/java/org/bukkit/craftbukkit/CraftServer.java +++ b/src/main/java/org/bukkit/craftbukkit/CraftServer.java @@ -1133,6 +1133,18 @@ public final class CraftServer implements Server { ReloadCommand.reload(this.console); } + // Paper start + @Override + public void updateResources() { + this.playerList.reloadResources(); + } + + @Override + public void updateRecipes() { + this.playerList.reloadRecipeData(); + } + // Paper end + private void loadIcon() { this.icon = new CraftIconCache(null); try { @@ -1476,6 +1488,13 @@ public final class CraftServer implements Server { @Override public boolean addRecipe(Recipe recipe) { + // Paper start + return this.addRecipe(recipe, false); + } + + @Override + public boolean addRecipe(Recipe recipe, boolean resendRecipes) { + // Paper end CraftRecipe toAdd; if (recipe instanceof CraftRecipe) { toAdd = (CraftRecipe) recipe; @@ -1505,6 +1524,11 @@ public final class CraftServer implements Server { } } toAdd.addToCraftingManager(); + // Paper start + if (resendRecipes) { + this.playerList.reloadRecipeData(); + } + // Paper end return true; } @@ -1685,10 +1709,23 @@ public final class CraftServer implements Server { @Override public boolean removeRecipe(NamespacedKey recipeKey) { + // Paper start + return this.removeRecipe(recipeKey, false); + } + + @Override + public boolean removeRecipe(NamespacedKey recipeKey, boolean resendRecipes) { + // Paper end Preconditions.checkArgument(recipeKey != null, "recipeKey == null"); ResourceLocation mcKey = CraftNamespacedKey.toMinecraft(recipeKey); - return this.getServer().getRecipeManager().removeRecipe(mcKey); + // Paper start - resend recipes on successful removal + boolean removed = this.getServer().getRecipeManager().removeRecipe(mcKey); + if (removed && resendRecipes) { + this.playerList.reloadRecipeData(); + } + return removed; + // Paper end } @Override