aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0263-Implement-furnace-cook-speed-multiplier-API.patch
diff options
context:
space:
mode:
Diffstat (limited to 'patches/server/0263-Implement-furnace-cook-speed-multiplier-API.patch')
-rw-r--r--patches/server/0263-Implement-furnace-cook-speed-multiplier-API.patch129
1 files changed, 129 insertions, 0 deletions
diff --git a/patches/server/0263-Implement-furnace-cook-speed-multiplier-API.patch b/patches/server/0263-Implement-furnace-cook-speed-multiplier-API.patch
new file mode 100644
index 0000000000..ef1f41e68e
--- /dev/null
+++ b/patches/server/0263-Implement-furnace-cook-speed-multiplier-API.patch
@@ -0,0 +1,129 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Tassu <[email protected]>
+Date: Thu, 13 Sep 2018 08:45:21 +0300
+Subject: [PATCH] Implement furnace cook speed multiplier API
+
+Signed-off-by: Tassu <[email protected]>
+
+Fixed an issue where a furnace's cook-speed multiplier rounds down
+to the nearest Integer when updating its current cook time.
+
+Modified by: Eric Su <[email protected]>
+
+diff --git a/src/main/java/net/minecraft/world/level/block/entity/AbstractFurnaceBlockEntity.java b/src/main/java/net/minecraft/world/level/block/entity/AbstractFurnaceBlockEntity.java
+index bbccb2f97ee88fe43af4eec30fbb2ecea1ebb1b4..d3a9b42fd4fa487e0acb1c95d57f78ccfbefbdff 100644
+--- a/src/main/java/net/minecraft/world/level/block/entity/AbstractFurnaceBlockEntity.java
++++ b/src/main/java/net/minecraft/world/level/block/entity/AbstractFurnaceBlockEntity.java
+@@ -76,11 +76,13 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ protected NonNullList<ItemStack> items;
+ public int litTime;
+ int litDuration;
++ public double cookSpeedMultiplier = 1.0; // Paper - cook speed multiplier API
+ public int cookingProgress;
+ public int cookingTotalTime;
+ protected final ContainerData dataAccess;
+ public final Object2IntOpenHashMap<ResourceLocation> recipesUsed;
+ private final RecipeManager.CachedCheck<Container, ? extends AbstractCookingRecipe> quickCheck;
++ public final RecipeType<? extends AbstractCookingRecipe> recipeType; // Paper
+
+ protected AbstractFurnaceBlockEntity(BlockEntityType<?> blockEntityType, BlockPos pos, BlockState state, RecipeType<? extends AbstractCookingRecipe> recipeType) {
+ super(blockEntityType, pos, state);
+@@ -127,6 +129,7 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ };
+ this.recipesUsed = new Object2IntOpenHashMap();
+ this.quickCheck = RecipeManager.createCheck((RecipeType<AbstractCookingRecipe>) recipeType); // CraftBukkit - decompile error // Eclipse fail
++ this.recipeType = recipeType; // Paper
+ }
+
+ public static Map<Item, Integer> getFuel() {
+@@ -285,6 +288,11 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ this.recipesUsed.put(new ResourceLocation(s), nbttagcompound1.getInt(s));
+ }
+
++ // Paper start - cook speed API
++ if (nbt.contains("Paper.CookSpeedMultiplier")) {
++ this.cookSpeedMultiplier = nbt.getDouble("Paper.CookSpeedMultiplier");
++ }
++ // Paper end
+ }
+
+ @Override
+@@ -293,6 +301,7 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ nbt.putShort("BurnTime", (short) this.litTime);
+ nbt.putShort("CookTime", (short) this.cookingProgress);
+ nbt.putShort("CookTimeTotal", (short) this.cookingTotalTime);
++ nbt.putDouble("Paper.CookSpeedMultiplier", this.cookSpeedMultiplier); // Paper - cook speed multiplier API
+ ContainerHelper.saveAllItems(nbt, this.items);
+ CompoundTag nbttagcompound1 = new CompoundTag();
+
+@@ -364,7 +373,7 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ CraftItemStack source = CraftItemStack.asCraftMirror(blockEntity.items.get(0));
+ CookingRecipe<?> recipe = (CookingRecipe<?>) irecipe.toBukkitRecipe();
+
+- FurnaceStartSmeltEvent event = new FurnaceStartSmeltEvent(CraftBlock.at(world, pos), source, recipe);
++ FurnaceStartSmeltEvent event = new FurnaceStartSmeltEvent(CraftBlock.at(world, pos), source, recipe, AbstractFurnaceBlockEntity.getTotalCookTime(world, blockEntity.recipeType, blockEntity, blockEntity.cookSpeedMultiplier)); // Paper - cook speed multiplier API
+ world.getCraftServer().getPluginManager().callEvent(event);
+
+ blockEntity.cookingTotalTime = event.getTotalCookTime();
+@@ -372,9 +381,9 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ // CraftBukkit end
+
+ ++blockEntity.cookingProgress;
+- if (blockEntity.cookingProgress == blockEntity.cookingTotalTime) {
++ if (blockEntity.cookingProgress >= blockEntity.cookingTotalTime) { // Paper - cook speed multiplier API
+ blockEntity.cookingProgress = 0;
+- blockEntity.cookingTotalTime = AbstractFurnaceBlockEntity.getTotalCookTime(world, blockEntity);
++ blockEntity.cookingTotalTime = AbstractFurnaceBlockEntity.getTotalCookTime(world, blockEntity.recipeType, blockEntity, blockEntity.cookSpeedMultiplier); // Paper
+ if (AbstractFurnaceBlockEntity.burn(blockEntity.level, blockEntity.worldPosition, irecipe, blockEntity.items, i)) { // CraftBukkit
+ blockEntity.setRecipeUsed(irecipe);
+ }
+@@ -474,9 +483,13 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ }
+ }
+
+- private static int getTotalCookTime(Level world, AbstractFurnaceBlockEntity furnace) {
+- return (world != null) ? (Integer) furnace.quickCheck.getRecipeFor(furnace, world).map(AbstractCookingRecipe::getCookingTime).orElse(200) : 200; // CraftBukkit - SPIGOT-4302
++ // Paper start
++ public static int getTotalCookTime(@Nullable Level world, RecipeType<? extends AbstractCookingRecipe> recipeType, AbstractFurnaceBlockEntity furnace, double cookSpeedMultiplier) {
++ /* Scale the recipe's cooking time to the current cookSpeedMultiplier */
++ int cookTime = world != null ? furnace.quickCheck.getRecipeFor(furnace, world).map(AbstractCookingRecipe::getCookingTime).orElse(200) : (net.minecraft.server.MinecraftServer.getServer().getRecipeManager().getRecipeFor(recipeType, furnace, world /* passing a null level here is safe. world is only used for map extending recipes which won't happen here */).map(AbstractCookingRecipe::getCookingTime).orElse(200));
++ return (int) Math.ceil (cookTime / cookSpeedMultiplier);
+ }
++ // Paper end
+
+ public static boolean isFuel(ItemStack stack) {
+ return AbstractFurnaceBlockEntity.getFuel().containsKey(stack.getItem());
+@@ -545,7 +558,7 @@ public abstract class AbstractFurnaceBlockEntity extends BaseContainerBlockEntit
+ }
+
+ if (slot == 0 && !flag) {
+- this.cookingTotalTime = AbstractFurnaceBlockEntity.getTotalCookTime(this.level, this);
++ this.cookingTotalTime = AbstractFurnaceBlockEntity.getTotalCookTime(this.level, this.recipeType, this, this.cookSpeedMultiplier); // Paper
+ this.cookingProgress = 0;
+ this.setChanged();
+ }
+diff --git a/src/main/java/org/bukkit/craftbukkit/block/CraftFurnace.java b/src/main/java/org/bukkit/craftbukkit/block/CraftFurnace.java
+index facf95a44b5d3a63fda156c6afc8cabe50b21d32..3da4616c904d47bbecae0d4cb6970496fbec9a8b 100644
+--- a/src/main/java/org/bukkit/craftbukkit/block/CraftFurnace.java
++++ b/src/main/java/org/bukkit/craftbukkit/block/CraftFurnace.java
+@@ -78,4 +78,20 @@ public abstract class CraftFurnace<T extends AbstractFurnaceBlockEntity> extends
+
+ return recipesUsed.build();
+ }
++
++ // Paper start - cook speed multiplier API
++ @Override
++ public double getCookSpeedMultiplier() {
++ return this.getSnapshot().cookSpeedMultiplier;
++ }
++
++ @Override
++ public void setCookSpeedMultiplier(double multiplier) {
++ com.google.common.base.Preconditions.checkArgument(multiplier >= 0, "Furnace speed multiplier cannot be negative");
++ com.google.common.base.Preconditions.checkArgument(multiplier <= 200, "Furnace speed multiplier cannot more than 200");
++ T snapshot = this.getSnapshot();
++ snapshot.cookSpeedMultiplier = multiplier;
++ snapshot.cookingTotalTime = AbstractFurnaceBlockEntity.getTotalCookTime(this.isPlaced() ? this.world.getHandle() : null, snapshot.recipeType, snapshot, snapshot.cookSpeedMultiplier); // Update the snapshot's current total cook time to scale with the newly set multiplier
++ }
++ // Paper end
+ }