From afe5fc14c846e97d1adbf1d01955fd25624f0f83 Mon Sep 17 00:00:00 2001 From: Tamion <70228790+notTamion@users.noreply.github.com> Date: Sat, 28 Sep 2024 21:14:20 +0200 Subject: [PATCH] Add startingBrewTime (#11406) * add startingBrewTime * casting * requests * comments and fix setBrewTime * a * b * move to own class * rename * missing Paper comment * missing experimentals * rebased --------- Co-authored-by: Jake Potrebic <jake.m.potrebic@gmail.com> --- patches/api/Add-recipeBrewTime.patch | 158 +++++++++++++++++++++ patches/server/Add-recipeBrewTime.patch | 181 ++++++++++++++++++++++++ 2 files changed, 339 insertions(+) create mode 100644 patches/api/Add-recipeBrewTime.patch create mode 100644 patches/server/Add-recipeBrewTime.patch diff --git a/patches/api/Add-recipeBrewTime.patch b/patches/api/Add-recipeBrewTime.patch new file mode 100644 index 0000000000..5cffda2afc --- /dev/null +++ b/patches/api/Add-recipeBrewTime.patch @@ -0,0 +1,158 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Tamion <70228790+notTamion@users.noreply.github.com> +Date: Fri, 20 Sep 2024 17:39:22 +0200 +Subject: [PATCH] Add recipeBrewTime + + +diff --git a/src/main/java/org/bukkit/block/BrewingStand.java b/src/main/java/org/bukkit/block/BrewingStand.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/org/bukkit/block/BrewingStand.java ++++ b/src/main/java/org/bukkit/block/BrewingStand.java +@@ -0,0 +0,0 @@ public interface BrewingStand extends Container { + */ + void setBrewingTime(int brewTime); + ++ // Paper start - Add recipeBrewTime ++ /** ++ * Sets the recipe time for the brewing process which is ++ * used to compute the progress of the brewing process with ++ * {@link #getBrewingTime()}. ++ * ++ * @param recipeBrewTime recipe brew time (in ticks) ++ * @throws IllegalArgumentException if the recipe brew time is non-positive ++ */ ++ @org.jetbrains.annotations.ApiStatus.Experimental ++ void setRecipeBrewTime(@org.jetbrains.annotations.Range(from = 1, to = Integer.MAX_VALUE) int recipeBrewTime); ++ ++ /** ++ * Gets the recipe time for the brewing process which is ++ * used to compute the progress of the brewing process with ++ * {@link #getBrewingTime()}. ++ * ++ * @return recipe brew time (in ticks) ++ */ ++ @org.jetbrains.annotations.ApiStatus.Experimental ++ @org.jetbrains.annotations.Range(from = 1, to = Integer.MAX_VALUE) int getRecipeBrewTime(); ++ // Paper end - Add recipeBrewTime ++ + /** + * Get the level of current fuel for brewing. + * +diff --git a/src/main/java/org/bukkit/event/block/BrewingStartEvent.java b/src/main/java/org/bukkit/event/block/BrewingStartEvent.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/org/bukkit/event/block/BrewingStartEvent.java ++++ b/src/main/java/org/bukkit/event/block/BrewingStartEvent.java +@@ -0,0 +0,0 @@ public class BrewingStartEvent extends InventoryBlockStartEvent { + + // Paper - remove HandlerList + private int brewingTime; ++ private int recipeBrewTime = 400; // Paper - Add recipeBrewTime + ++ @org.jetbrains.annotations.ApiStatus.Internal // Paper + public BrewingStartEvent(@NotNull final Block furnace, @NotNull ItemStack source, int brewingTime) { + super(furnace, source); + this.brewingTime = brewingTime; +@@ -0,0 +0,0 @@ public class BrewingStartEvent extends InventoryBlockStartEvent { + * Gets the total brew time associated with this event. + * + * @return the total brew time ++ * @deprecated use {@link #getBrewingTime()} instead + */ ++ @Deprecated(since = "1.21", forRemoval = true) // Paper + public int getTotalBrewTime() { + return brewingTime; + } +@@ -0,0 +0,0 @@ public class BrewingStartEvent extends InventoryBlockStartEvent { + * Sets the total brew time for this event. + * + * @param brewTime the new total brew time ++ * @deprecated use {@link #setBrewingTime(int)} instead + */ ++ @Deprecated(since = "1.21", forRemoval = true) // Paper + public void setTotalBrewTime(int brewTime) { +- this.brewingTime = brewTime; ++ this.setBrewingTime(brewTime); // Paper - delegate to new method + } + + // Paper - remove HandlerList ++ ++ // Paper start - add recipeBrewTime ++ /** ++ * Gets the recipe time for the brewing process which is ++ * used to compute the progress of the brewing process with ++ * {@link #getBrewingTime()}. ++ * ++ * @return recipe brew time (in ticks) ++ */ ++ @org.jetbrains.annotations.ApiStatus.Experimental ++ public @org.jetbrains.annotations.Range(from = 1, to = Integer.MAX_VALUE) int getRecipeBrewTime() { ++ return this.recipeBrewTime; ++ } ++ ++ /** ++ * Sets the recipe time for the brewing process which is ++ * used to compute the progress of the brewing process with ++ * {@link #getBrewingTime()}. ++ * ++ * @param recipeBrewTime recipe brew time (in ticks) ++ * @throws IllegalArgumentException if the recipe brew time is non-positive ++ */ ++ @org.jetbrains.annotations.ApiStatus.Experimental ++ public void setRecipeBrewTime(@org.jetbrains.annotations.Range(from = 1, to = Integer.MAX_VALUE) int recipeBrewTime) { ++ com.google.common.base.Preconditions.checkArgument(recipeBrewTime > 0, "recipeBrewTime must be positive"); ++ this.recipeBrewTime = recipeBrewTime; ++ } ++ ++ /** ++ * Gets the amount of brewing ticks left. ++ * ++ * @return The amount of ticks left for the brewing task ++ */ ++ public @org.jetbrains.annotations.Range(from = 0, to = Integer.MAX_VALUE) int getBrewingTime() { ++ return this.brewingTime; ++ } ++ ++ /** ++ * Sets the brewing ticks left. ++ * ++ * @param brewTime the ticks left, which is no less than 0 ++ * @throws IllegalArgumentException if the ticks are less than 0 ++ */ ++ public void setBrewingTime(@org.jetbrains.annotations.Range(from = 0, to = Integer.MAX_VALUE) int brewTime) { ++ com.google.common.base.Preconditions.checkArgument(brewTime >= 0, "brewTime must be non-negative"); ++ this.brewingTime = brewTime; ++ } ++ // Paper end - add recipeBrewTime + } +diff --git a/src/main/java/org/bukkit/inventory/view/BrewingStandView.java b/src/main/java/org/bukkit/inventory/view/BrewingStandView.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/org/bukkit/inventory/view/BrewingStandView.java ++++ b/src/main/java/org/bukkit/inventory/view/BrewingStandView.java +@@ -0,0 +0,0 @@ public interface BrewingStandView extends InventoryView { + * @throws IllegalArgumentException if the ticks are less than 0 + */ + void setBrewingTicks(final int ticks) throws IllegalArgumentException; ++ ++ // Paper start - Add recipeBrewTime ++ /** ++ * Sets the recipe time for the brewing process which is ++ * used to compute the progress of the brewing process with ++ * {@link #getBrewingTicks()}. ++ * ++ * @param recipeBrewTime recipe brew time (in ticks) ++ * @throws IllegalArgumentException if the recipe brew time is non-positive ++ */ ++ @org.jetbrains.annotations.ApiStatus.Experimental ++ void setRecipeBrewTime(@org.jetbrains.annotations.Range(from = 1, to = Integer.MAX_VALUE) int recipeBrewTime); ++ ++ /** ++ * Gets the recipe time for the brewing process which is ++ * used to compute the progress of the brewing process with ++ * {@link #getBrewingTicks()}. ++ * ++ * @return recipe brew time (in ticks) ++ */ ++ @org.jetbrains.annotations.ApiStatus.Experimental ++ @org.jetbrains.annotations.Range(from = 1, to = Integer.MAX_VALUE) int getRecipeBrewTime(); ++ // Paper end - Add recipeBrewTime + } diff --git a/patches/server/Add-recipeBrewTime.patch b/patches/server/Add-recipeBrewTime.patch new file mode 100644 index 0000000000..ef5b87e03f --- /dev/null +++ b/patches/server/Add-recipeBrewTime.patch @@ -0,0 +1,181 @@ +From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 +From: Tamion <70228790+notTamion@users.noreply.github.com> +Date: Sun, 15 Sep 2024 19:17:12 +0200 +Subject: [PATCH] Add recipeBrewTime + +== AT == +public net.minecraft.world.inventory.BrewingStandMenu brewingStandData + +diff --git a/src/main/java/io/papermc/paper/inventory/BrewingSimpleContainerData.java b/src/main/java/io/papermc/paper/inventory/BrewingSimpleContainerData.java +new file mode 100644 +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 +--- /dev/null ++++ b/src/main/java/io/papermc/paper/inventory/BrewingSimpleContainerData.java +@@ -0,0 +0,0 @@ ++package io.papermc.paper.inventory; ++ ++import net.minecraft.world.inventory.SimpleContainerData; ++ ++public class BrewingSimpleContainerData extends SimpleContainerData { ++ ++ public BrewingSimpleContainerData() { ++ super(3); ++ this.set(2, 400); ++ } ++} +diff --git a/src/main/java/net/minecraft/world/inventory/BrewingStandMenu.java b/src/main/java/net/minecraft/world/inventory/BrewingStandMenu.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/net/minecraft/world/inventory/BrewingStandMenu.java ++++ b/src/main/java/net/minecraft/world/inventory/BrewingStandMenu.java +@@ -0,0 +0,0 @@ public class BrewingStandMenu extends AbstractContainerMenu { + // CraftBukkit end + + public BrewingStandMenu(int syncId, Inventory playerInventory) { +- this(syncId, playerInventory, new SimpleContainer(5), new SimpleContainerData(2)); ++ this(syncId, playerInventory, new SimpleContainer(5), new io.papermc.paper.inventory.BrewingSimpleContainerData()); // Paper - Add totalBrewTime + } + + public BrewingStandMenu(int syncId, Inventory playerInventory, Container inventory, ContainerData propertyDelegate) { + super(MenuType.BREWING_STAND, syncId); + this.player = playerInventory; // CraftBukkit + checkContainerSize(inventory, 5); +- checkContainerDataCount(propertyDelegate, 2); ++ checkContainerDataCount(propertyDelegate, 3); // Paper - Add recipeBrewTime + this.brewingStand = inventory; + this.brewingStandData = propertyDelegate; + PotionBrewing potionbrewer = playerInventory.player.level().potionBrewing(); +@@ -0,0 +0,0 @@ public class BrewingStandMenu extends AbstractContainerMenu { + // Paper end - custom potion mixes + this.ingredientSlot = this.addSlot(new BrewingStandMenu.IngredientsSlot(potionbrewer, inventory, 3, 79, 17)); + this.addSlot(new BrewingStandMenu.FuelSlot(inventory, 4, 17, 17)); +- this.addDataSlots(propertyDelegate); ++ // Paper start - Add recipeBrewTime ++ this.addDataSlots(new SimpleContainerData(2) { ++ @Override ++ public int get(final int index) { ++ if (index == 0) return 400 * propertyDelegate.get(index) / propertyDelegate.get(2); ++ return propertyDelegate.get(index); ++ } ++ ++ @Override ++ public void set(final int index, final int value) { ++ propertyDelegate.set(index, value); ++ } ++ }); ++ // Paper end - Add recipeBrewTime + + int j; + +diff --git a/src/main/java/net/minecraft/world/level/block/entity/BrewingStandBlockEntity.java b/src/main/java/net/minecraft/world/level/block/entity/BrewingStandBlockEntity.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/net/minecraft/world/level/block/entity/BrewingStandBlockEntity.java ++++ b/src/main/java/net/minecraft/world/level/block/entity/BrewingStandBlockEntity.java +@@ -0,0 +0,0 @@ public class BrewingStandBlockEntity extends BaseContainerBlockEntity implements + public static final int NUM_DATA_VALUES = 2; + private NonNullList<ItemStack> items; + public int brewTime; ++ public int recipeBrewTime = 400; // Paper - Add recipeBrewTime + private boolean[] lastPotionCount; + private Item ingredient; + public int fuel; +@@ -0,0 +0,0 @@ public class BrewingStandBlockEntity extends BaseContainerBlockEntity implements + case 1: + j = BrewingStandBlockEntity.this.fuel; + break; ++ // Paper start - Add recipeBrewTime ++ case 2: ++ j = BrewingStandBlockEntity.this.recipeBrewTime; ++ break; ++ // Paper end - Add recipeBrewTime + default: + j = 0; + } +@@ -0,0 +0,0 @@ public class BrewingStandBlockEntity extends BaseContainerBlockEntity implements + break; + case 1: + BrewingStandBlockEntity.this.fuel = value; ++ // Paper start - Add recipeBrewTime ++ case 2: ++ BrewingStandBlockEntity.this.recipeBrewTime = value; ++ break; ++ // Paper end - Add recipeBrewTime + } + + } + + @Override + public int getCount() { +- return 2; ++ return 3; // Paper - Add recipeBrewTime + } + }; + } +@@ -0,0 +0,0 @@ public class BrewingStandBlockEntity extends BaseContainerBlockEntity implements + // CraftBukkit start + BrewingStartEvent event = new BrewingStartEvent(CraftBlock.at(world, pos), CraftItemStack.asCraftMirror(itemstack1), 400); + world.getCraftServer().getPluginManager().callEvent(event); +- blockEntity.brewTime = event.getTotalBrewTime(); // 400 -> event.getTotalBrewTime() ++ blockEntity.recipeBrewTime = event.getRecipeBrewTime(); // Paper - use recipe brew time from event ++ blockEntity.brewTime = event.getBrewingTime(); // 400 -> event.getTotalBrewTime() // Paper - use brewing time from event + // CraftBukkit end + blockEntity.ingredient = itemstack1.getItem(); + setChanged(world, pos, state); +diff --git a/src/main/java/org/bukkit/craftbukkit/block/CraftBrewingStand.java b/src/main/java/org/bukkit/craftbukkit/block/CraftBrewingStand.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/org/bukkit/craftbukkit/block/CraftBrewingStand.java ++++ b/src/main/java/org/bukkit/craftbukkit/block/CraftBrewingStand.java +@@ -0,0 +0,0 @@ public class CraftBrewingStand extends CraftContainer<BrewingStandBlockEntity> i + this.getSnapshot().brewTime = brewTime; + } + ++ // Paper start - Add recipeBrewTime ++ @Override ++ public void setRecipeBrewTime(int recipeBrewTime) { ++ com.google.common.base.Preconditions.checkArgument(recipeBrewTime > 0, "recipeBrewTime must be positive"); ++ this.getSnapshot().recipeBrewTime = recipeBrewTime; ++ } ++ ++ @Override ++ public int getRecipeBrewTime() { ++ return this.getSnapshot().recipeBrewTime; ++ } ++ // Paper end - Add recipeBrewTime ++ + @Override + public int getFuelLevel() { + return this.getSnapshot().fuel; +diff --git a/src/main/java/org/bukkit/craftbukkit/inventory/CraftContainer.java b/src/main/java/org/bukkit/craftbukkit/inventory/CraftContainer.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/org/bukkit/craftbukkit/inventory/CraftContainer.java ++++ b/src/main/java/org/bukkit/craftbukkit/inventory/CraftContainer.java +@@ -0,0 +0,0 @@ public class CraftContainer extends AbstractContainerMenu { + this.delegate = new EnchantmentMenu(windowId, bottom); + break; + case BREWING: +- this.delegate = new BrewingStandMenu(windowId, bottom, top, new SimpleContainerData(2)); ++ this.delegate = new BrewingStandMenu(windowId, bottom, top, new io.papermc.paper.inventory.BrewingSimpleContainerData()); // Paper - Add recipeBrewTime + break; + case HOPPER: + this.delegate = new HopperMenu(windowId, bottom, top); +diff --git a/src/main/java/org/bukkit/craftbukkit/inventory/view/CraftBrewingStandView.java b/src/main/java/org/bukkit/craftbukkit/inventory/view/CraftBrewingStandView.java +index 0000000000000000000000000000000000000000..0000000000000000000000000000000000000000 100644 +--- a/src/main/java/org/bukkit/craftbukkit/inventory/view/CraftBrewingStandView.java ++++ b/src/main/java/org/bukkit/craftbukkit/inventory/view/CraftBrewingStandView.java +@@ -0,0 +0,0 @@ public class CraftBrewingStandView extends CraftInventoryView<BrewingStandMenu> + Preconditions.checkArgument(brewingTicks > 0, "The given brewing ticks must be greater than 0"); + this.container.setData(BrewingStandBlockEntity.DATA_BREW_TIME, brewingTicks); + } ++ ++ // Paper start - Add recipeBrewTime ++ @Override ++ public void setRecipeBrewTime(int recipeBrewTime) { ++ com.google.common.base.Preconditions.checkArgument(recipeBrewTime > 0, "recipeBrewTime must be positive"); ++ this.container.brewingStandData.set(2, recipeBrewTime); ++ } ++ ++ @Override ++ public int getRecipeBrewTime() { ++ return this.container.brewingStandData.get(2); ++ } ++ // Paper end - Add recipeBrewTime + }