From 4db6e035170954b9230fe96f70f0caf96658554f Mon Sep 17 00:00:00 2001 From: nossr50 Date: Sat, 23 Jul 2022 14:43:24 -0700 Subject: [PATCH] Hacky workaround for Spigot API bug not showing burntime to the client when burntime was too high --- Changelog.txt | 1 + .../java/com/gmail/nossr50/skills/smelting/SmeltingManager.java | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Changelog.txt b/Changelog.txt index bf402aa0a..b736bc608 100644 --- a/Changelog.txt +++ b/Changelog.txt @@ -1,4 +1,5 @@ Version 2.1.215 + Added hacky workaround for Spigot API bug that resulted in burnTimes not showing burning animation to the client Level up broadcasts from level milestones will now be visible to the player who achieved them Fixed a bug where hovering over skill descriptions did not display anything (thanks Greymagic27) Added Dripstone to experience.yml (thanks Greymagic27) diff --git a/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java b/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java index 83248c166..98d5a5a6f 100644 --- a/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java +++ b/src/main/java/com/gmail/nossr50/skills/smelting/SmeltingManager.java @@ -34,7 +34,7 @@ public class SmeltingManager extends SkillManager { * @param burnTime The initial burn time from the {@link FurnaceBurnEvent} */ public int fuelEfficiency(int burnTime) { - return Math.max(1, burnTime * getFuelEfficiencyMultiplier()); + return Math.min(Short.MAX_VALUE, Math.max(1, burnTime * getFuelEfficiencyMultiplier())); } public int getFuelEfficiencyMultiplier()