refactor(26.3)!: Remove setBurnTime in favor of Data Component

Author: BlayTheNinth

common/src/main/java/net/blay09/mods/balm/platform/BalmHooks.java

@@ -1,16 +1,13 @@
 package net.blay09.mods.balm.platform;
 
 import net.minecraft.core.BlockPos;
-import net.minecraft.core.Holder;
 import net.minecraft.nbt.CompoundTag;
-import net.minecraft.util.RandomSource;
 import net.minecraft.world.Container;
 import net.minecraft.world.InteractionHand;
 import net.minecraft.world.entity.Entity;
 import net.minecraft.world.entity.Pose;
 import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.item.DyeColor;
-import net.minecraft.world.item.Item;
 import net.minecraft.world.item.ItemStack;
 import net.minecraft.world.item.ItemStackTemplate;
 import net.minecraft.world.level.Level;
@@ -57,8 +54,6 @@ default CompoundTag getPersistentData(Player player) {
     @Nullable
     DyeColor getColor(ItemStack itemStack);
 
-    void setBurnTime(Item item, int burnTime);
-
     void firePlayerCraftingEvent(Player player, ItemStack crafted, Container craftMatrix);
 
     /**

fabric/src/main/java/net/blay09/mods/balm/fabric/platform/internal/FabricBalmHooks.java

@@ -6,14 +6,11 @@
 import net.blay09.mods.balm.world.entity.BalmForcedPoseHolder;
 import net.blay09.mods.balm.platform.capabilities.CommonCapabilities;
 import net.fabricmc.fabric.api.entity.FakePlayer;
-import net.fabricmc.fabric.api.registry.FuelValueEvents;
 import net.minecraft.core.BlockPos;
-import net.minecraft.core.Holder;
 import net.minecraft.core.component.DataComponents;
 import net.minecraft.core.registries.BuiltInRegistries;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.sounds.SoundEvents;
-import net.minecraft.util.RandomSource;
 import net.minecraft.world.Container;
 import net.minecraft.world.InteractionHand;
 import net.minecraft.world.entity.Entity;
@@ -72,11 +69,6 @@ public boolean isFakePlayer(Player player) {
         return itemStack.get(DataComponents.DYE);
     }
 
-    @Override
-    public void setBurnTime(Item item, int burnTime) {
-        FuelValueEvents.BUILD.register((builder, context) -> builder.add(item, burnTime));
-    }
-
     @Override
     public void firePlayerCraftingEvent(Player player, ItemStack crafted, Container craftMatrix) {
     }

forge/src/main/java/net/blay09/mods/balm/forge/platform/internal/ForgeBalmHooks.java

@@ -3,10 +3,8 @@
 import net.blay09.mods.balm.nbt.BalmDataHolder;
 import net.blay09.mods.balm.platform.BalmHooks;
 import net.minecraft.core.BlockPos;
-import net.minecraft.core.Holder;
 import net.minecraft.nbt.CompoundTag;
 import net.minecraft.server.level.ServerPlayer;
-import net.minecraft.util.RandomSource;
 import net.minecraft.world.Container;
 import net.minecraft.world.InteractionHand;
 import net.minecraft.world.entity.Entity;
@@ -18,7 +16,6 @@
 import net.minecraft.world.phys.BlockHitResult;
 import net.minecraftforge.common.util.Result;
 import net.minecraftforge.event.ForgeEventFactory;
-import net.minecraftforge.event.furnace.FurnaceFuelBurnTimeEvent;
 import net.minecraftforge.fluids.FluidUtil;
 import org.jspecify.annotations.Nullable;
 
@@ -27,19 +24,6 @@
 
 public class ForgeBalmHooks implements BalmHooks {
 
-    public final Map<Item, Integer> burnTimes = new ConcurrentHashMap<>();
-
-    public ForgeBalmHooks() {
-        FurnaceFuelBurnTimeEvent.BUS.addListener(this::furnaceFuelBurnTime);
-    }
-
-    private void furnaceFuelBurnTime(FurnaceFuelBurnTimeEvent event) {
-        final var found = burnTimes.get(event.getItemStack().getItem());
-        if (found != null) {
-            event.setBurnTime(found);
-        }
-    }
-
     @Override
     public boolean growCrop(ItemStack itemStack, Level level, BlockPos pos, @Nullable Player player) {
         if (player != null) {
@@ -89,11 +73,6 @@ public boolean isFakePlayer(Player player) {
         return DyeColor.getColor(itemStack);
     }
 
-    @Override
-    public void setBurnTime(Item item, int burnTime) {
-        burnTimes.put(item, burnTime);
-    }
-
     @Override
     public void firePlayerCraftingEvent(Player player, ItemStack crafted, Container craftMatrix) {
         ForgeEventFactory.firePlayerCraftingEvent(player, crafted, craftMatrix);

neoforge/src/main/java/net/blay09/mods/balm/neoforge/platform/internal/NeoForgeBalmHooks.java

@@ -3,9 +3,7 @@
 import net.blay09.mods.balm.nbt.BalmDataHolder;
 import net.blay09.mods.balm.platform.BalmHooks;
 import net.minecraft.core.BlockPos;
-import net.minecraft.core.Holder;
 import net.minecraft.nbt.CompoundTag;
-import net.minecraft.util.RandomSource;
 import net.minecraft.world.Container;
 import net.minecraft.world.InteractionHand;
 import net.minecraft.world.entity.Entity;
@@ -18,7 +16,6 @@
 import net.neoforged.neoforge.common.NeoForge;
 import net.neoforged.neoforge.common.util.FakePlayer;
 import net.neoforged.neoforge.event.EventHooks;
-import net.neoforged.neoforge.event.furnace.FurnaceFuelBurnTimeEvent;
 import net.neoforged.neoforge.transfer.fluid.FluidUtil;
 import org.jspecify.annotations.Nullable;
 
@@ -29,17 +26,6 @@ public class NeoForgeBalmHooks implements BalmHooks {
 
     public final Map<Item, Integer> burnTimes = new ConcurrentHashMap<>();
 
-    public NeoForgeBalmHooks() {
-        NeoForge.EVENT_BUS.addListener(this::furnaceFuelBurnTime);
-    }
-
-    private void furnaceFuelBurnTime(FurnaceFuelBurnTimeEvent event) {
-        final var found = burnTimes.get(event.getItemStack().getItem());
-        if (found != null) {
-            event.setBurnTime(found);
-        }
-    }
-
     @Override
     public boolean growCrop(ItemStack itemStack, Level level, BlockPos pos, @Nullable Player player) {
         if (player != null) {
@@ -89,11 +75,6 @@ public boolean isFakePlayer(Player player) {
         return DyeColor.getColor(itemStack);
     }
 
-    @Override
-    public void setBurnTime(Item item, int burnTime) {
-        burnTimes.put(item, burnTime);
-    }
-
     @Override
     public void firePlayerCraftingEvent(Player player, ItemStack crafted, Container craftMatrix) {
         EventHooks.firePlayerCraftingEvent(player, crafted, craftMatrix);