Remove unused imports

src/main/java/de/ellpeck/actuallyadditions/data/ItemRecipeGenerator.java

@@ -32,7 +32,6 @@ import net.minecraft.world.item.crafting.ShapelessRecipe;
 import net.minecraft.world.item.enchantment.Enchantment;
 import net.minecraft.world.item.enchantment.Enchantments;
 import net.minecraft.world.level.ItemLike;
-import net.neoforged.neoforge.common.CommonHooks;
 import net.neoforged.neoforge.common.Tags;
 import net.neoforged.neoforge.common.crafting.DataComponentIngredient;
 import net.neoforged.neoforge.registries.DeferredItem;

src/main/java/de/ellpeck/actuallyadditions/mod/blocks/BlockColoredLamp.java

@@ -11,7 +11,6 @@
 package de.ellpeck.actuallyadditions.mod.blocks;
 
 import de.ellpeck.actuallyadditions.mod.blocks.base.BlockBase;
-import de.ellpeck.actuallyadditions.mod.util.StackUtil;
 import net.minecraft.core.BlockPos;
 import net.minecraft.world.InteractionHand;
 import net.minecraft.world.ItemInteractionResult;

src/main/java/de/ellpeck/actuallyadditions/mod/blocks/base/BlockPlant.java

@@ -18,7 +18,6 @@ import net.minecraft.world.ItemInteractionResult;
 import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.item.Item;
 import net.minecraft.world.item.ItemStack;
-import net.minecraft.world.level.BlockGetter;
 import net.minecraft.world.level.ItemLike;
 import net.minecraft.world.level.Level;
 import net.minecraft.world.level.block.Block;

src/main/java/de/ellpeck/actuallyadditions/mod/event/CommonEvents.java

@@ -19,7 +19,6 @@ import de.ellpeck.actuallyadditions.mod.data.PlayerData;
 import de.ellpeck.actuallyadditions.mod.data.WorldData;
 import de.ellpeck.actuallyadditions.mod.items.ActuallyItems;
 import de.ellpeck.actuallyadditions.mod.items.DrillItem;
-import de.ellpeck.actuallyadditions.mod.items.ItemDrillUpgrade;
 import de.ellpeck.actuallyadditions.mod.items.ItemTag;
 import de.ellpeck.actuallyadditions.mod.items.Sack;
 import de.ellpeck.actuallyadditions.mod.network.PacketHandlerHelper;
@@ -28,7 +27,6 @@ import de.ellpeck.actuallyadditions.mod.tile.FilterSettings;
 import de.ellpeck.actuallyadditions.mod.util.ItemStackHandlerAA;
 import de.ellpeck.actuallyadditions.mod.util.ItemUtil;
 import de.ellpeck.actuallyadditions.mod.util.StackUtil;
-import de.ellpeck.actuallyadditions.mod.util.Util;
 import net.minecraft.core.BlockPos;
 import net.minecraft.core.registries.BuiltInRegistries;
 import net.minecraft.core.registries.Registries;
@@ -41,11 +39,8 @@ import net.minecraft.world.entity.monster.Spider;
 import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.item.Item;
 import net.minecraft.world.item.ItemStack;
-import net.minecraft.world.item.enchantment.Enchantments;
 import net.minecraft.world.level.LevelAccessor;
 import net.minecraft.world.level.block.Blocks;
-import net.minecraft.world.phys.BlockHitResult;
-import net.minecraft.world.phys.HitResult;
 import net.neoforged.bus.api.SubscribeEvent;
 import net.neoforged.neoforge.common.util.TriState;
 import net.neoforged.neoforge.event.AnvilUpdateEvent;
@@ -77,40 +72,9 @@ public class CommonEvents {
         final BlockPos pos = event.getPos();
         ItemStack stack = player.getMainHandItem();
         if (stack.getItem() instanceof DrillItem drillItem) {
-            boolean toReturn = false;
+            boolean toReturn = drillItem.onBreakBlock(stack, pos, player);
-            int use = drillItem.getEnergyUsePerBlock(stack);
+            if (toReturn) {
-            if (drillItem.getEnergyStored(stack) >= use) {
+                event.setCanceled(true);
-                //Enchants the Drill depending on the Upgrades it has
-                if (drillItem.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.SILK_TOUCH)) {
-                    ItemUtil.addEnchantment(stack, level.holderOrThrow(Enchantments.SILK_TOUCH), 1, level.registryAccess());
-                }
-                else {
-                    if (drillItem.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FORTUNE)) {
-                        ItemUtil.addEnchantment(stack, level.holderOrThrow(Enchantments.FORTUNE),
-                                drillItem.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FORTUNE_II) ? 3 : 1,
-                                level.registryAccess());
-                    }
-                }
-                //Block hit
-                HitResult ray = player.pick(Util.getReachDistance(player), 1f, false);
-                if (ray instanceof BlockHitResult trace) {
-                    //Breaks the Blocks
-                    if (!player.isShiftKeyDown() && drillItem.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.THREE_BY_THREE)) {
-                        if (drillItem.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FIVE_BY_FIVE)) {
-                            toReturn = drillItem.breakBlocks(stack, 2, player.level(), pos, trace.getDirection(), player);
-                        }
-                        else {
-                            toReturn = drillItem.breakBlocks(stack, 1, player.level(), pos, trace.getDirection(), player);
-                        }
-                    }
-                    else {
-                        toReturn = drillItem.breakBlocks(stack, 0, player.level(), pos, trace.getDirection(), player);
-                    }
-
-                    //Removes Enchantments added above
-                    ItemUtil.removeEnchantment(stack, level.holderOrThrow(Enchantments.SILK_TOUCH), level.registryAccess());
-                    ItemUtil.removeEnchantment(stack, level.holderOrThrow(Enchantments.FORTUNE), level.registryAccess());
-                }
             }
         }
     }

src/main/java/de/ellpeck/actuallyadditions/mod/gen/ActuallyConfiguredFeatures.java

@@ -6,7 +6,6 @@ import net.minecraft.core.registries.Registries;
 import net.minecraft.data.worldgen.BootstrapContext;
 import net.minecraft.data.worldgen.features.FeatureUtils;
 import net.minecraft.resources.ResourceKey;
-import net.minecraft.resources.ResourceLocation;
 import net.minecraft.tags.BlockTags;
 import net.minecraft.world.level.levelgen.feature.ConfiguredFeature;
 import net.minecraft.world.level.levelgen.feature.Feature;

src/main/java/de/ellpeck/actuallyadditions/mod/inventory/SackContainer.java

@@ -27,7 +27,6 @@ import net.minecraft.world.inventory.Slot;
 import net.minecraft.world.item.ItemStack;
 
 import javax.annotation.Nonnull;
-import java.util.UUID;
 
 public class SackContainer extends AbstractContainerMenu implements IButtonReactor {
 

src/main/java/de/ellpeck/actuallyadditions/mod/items/DrillItem.java

@@ -17,6 +17,8 @@ import de.ellpeck.actuallyadditions.mod.config.CommonConfig;
 import de.ellpeck.actuallyadditions.mod.inventory.ContainerDrill;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemEnergy;
 import de.ellpeck.actuallyadditions.mod.util.ItemStackHandlerAA;
+import de.ellpeck.actuallyadditions.mod.util.ItemUtil;
+import de.ellpeck.actuallyadditions.mod.util.Util;
 import de.ellpeck.actuallyadditions.mod.util.WorldUtil;
 import net.minecraft.core.BlockPos;
 import net.minecraft.core.Direction;
@@ -38,10 +40,12 @@ import net.minecraft.world.item.Tiers;
 import net.minecraft.world.item.component.ItemAttributeModifiers;
 import net.minecraft.world.item.component.ItemContainerContents;
 import net.minecraft.world.item.context.UseOnContext;
+import net.minecraft.world.item.enchantment.Enchantments;
 import net.minecraft.world.level.Level;
 import net.minecraft.world.level.block.Block;
 import net.minecraft.world.level.block.state.BlockState;
 import net.minecraft.world.phys.BlockHitResult;
+import net.minecraft.world.phys.HitResult;
 import net.neoforged.neoforge.common.ItemAbilities;
 import net.neoforged.neoforge.common.ItemAbility;
 import net.neoforged.neoforge.items.IItemHandler;
@@ -49,7 +53,10 @@ import net.neoforged.neoforge.items.IItemHandlerModifiable;
 
 import javax.annotation.Nonnull;
 import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
+import java.util.UUID;
 
 public class DrillItem extends ItemEnergy {
     public static final int HARVEST_LEVEL = 4;
@@ -177,7 +184,6 @@ public class DrillItem extends ItemEnergy {
     public InteractionResultHolder<ItemStack> use(Level world, @Nonnull Player player, @Nonnull InteractionHand hand) {
         if (!world.isClientSide && player.isShiftKeyDown() && hand == InteractionHand.MAIN_HAND) {
             player.openMenu(new SimpleMenuProvider((id, inv, p) -> new ContainerDrill(id, inv), Component.translatable("container.actuallyadditions.drill")));
-//            player.openGui(ActuallyAdditions.INSTANCE, GuiHandler.GuiTypes.DRILL.ordinal(), world, (int) player.posX, (int) player.posY, (int) player.posZ);
         }
         return new InteractionResultHolder<>(InteractionResult.PASS, player.getItemInHand(hand));
     }
@@ -188,7 +194,7 @@ public class DrillItem extends ItemEnergy {
         int use = this.getEnergyUsePerBlock(stack);
         if (!(entityHit instanceof Player) || !((Player) entityHit).isCreative()) {
             if (this.getEnergyStored(stack) >= use) {
-                this.extractEnergyInternal(stack, use, false);
+                this.extractEnergy(stack, use, false);
             }
         }
         return InteractionResult.SUCCESS;
@@ -212,42 +218,49 @@ public class DrillItem extends ItemEnergy {
                 : 0.1F;
     }
 
-    //    @Override TODO: Check if CommonEvents.onBlockBreaking works
+    private Set<UUID> breakers = new HashSet<>();
-//    public boolean onBlockStartBreak(@Nonnull ItemStack stack, @Nonnull BlockPos pos, @Nonnull Player player) {
+
-//        boolean toReturn = false;
+    public boolean onBreakBlock(@Nonnull ItemStack stack, @Nonnull BlockPos pos, @Nonnull Player player) {
-//        int use = this.getEnergyUsePerBlock(stack);
+        if (!breakers.add(player.getUUID())) return false; // Prevent multiple break operations from cascading, and don't execute when sneaking. (Borrowed from Apotheosis)
-//        if (this.getEnergyStored(stack) >= use) {
+        Level level = player.level();
-//            //Enchants the Drill depending on the Upgrades it has
+        boolean toReturn = false;
-//            if (this.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.SILK_TOUCH)) {
+        int use = this.getEnergyUsePerBlock(stack);
-//                ItemUtil.addEnchantment(stack, Enchantments.SILK_TOUCH, 1);
+        if (getEnergyStored(stack) >= use) {
-//            } else {
+            //Enchants the Drill depending on the Upgrades it has
-//                if (this.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FORTUNE)) {
+            if (getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.SILK_TOUCH)) {
-//                    ItemUtil.addEnchantment(stack, Enchantments.BLOCK_FORTUNE, this.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FORTUNE_II)
+                ItemUtil.addEnchantment(stack, level.holderOrThrow(Enchantments.SILK_TOUCH), 1, level.registryAccess());
-//                            ? 3
+            }
-//                            : 1);
+            else {
-//                }
+                if (getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FORTUNE)) {
-//            }
+                    ItemUtil.addEnchantment(stack, level.holderOrThrow(Enchantments.FORTUNE),
-//            //Block hit
+                            getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FORTUNE_II) ? 3 : 1,
-//            HitResult ray = player.pick(Util.getReachDistance(player), 1f, false);
+                            level.registryAccess());
-//            if (ray instanceof BlockHitResult trace) {
+                }
-//                //Breaks the Blocks
+            }
-//                if (!player.isShiftKeyDown() && this.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.THREE_BY_THREE)) {
+            //Block hit
-//                    if (this.getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FIVE_BY_FIVE)) {
+            HitResult ray = player.pick(Util.getReachDistance(player), 1f, false);
-//                        toReturn = this.breakBlocks(stack, 2, player.level(), pos, trace.getDirection(), player);
+            if (ray instanceof BlockHitResult trace) {
-//                    } else {
+                //Breaks the Blocks
-//                        toReturn = this.breakBlocks(stack, 1, player.level(), pos, trace.getDirection(), player);
+                if (!player.isShiftKeyDown() && getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.THREE_BY_THREE)) {
-//                    }
+                    if (getHasUpgrade(stack, ItemDrillUpgrade.UpgradeType.FIVE_BY_FIVE)) {
-//                } else {
+                        toReturn = breakBlocks(stack, 2, player.level(), pos, trace.getDirection(), player);
-//                    toReturn = this.breakBlocks(stack, 0, player.level(), pos, trace.getDirection(), player);
+                    }
-//                }
+                    else {
-//
+                        toReturn = breakBlocks(stack, 1, player.level(), pos, trace.getDirection(), player);
-//                //Removes Enchantments added above
+                    }
-//                ItemUtil.removeEnchantment(stack, Enchantments.SILK_TOUCH);
+                }
-//                ItemUtil.removeEnchantment(stack, Enchantments.BLOCK_FORTUNE);
+                else {
-//            }
+                    toReturn = breakBlocks(stack, 0, player.level(), pos, trace.getDirection(), player);
-//        }
+                }
-//        return toReturn;
+
-//    }
+                //Removes Enchantments added above
+                ItemUtil.removeEnchantment(stack, level.holderOrThrow(Enchantments.SILK_TOUCH), level.registryAccess());
+                ItemUtil.removeEnchantment(stack, level.holderOrThrow(Enchantments.FORTUNE), level.registryAccess());
+            }
+        }
+        breakers.remove(player.getUUID());
+        return toReturn;
+    }
 
 /*    @Override //TODO old one
     public boolean isCorrectToolForDrops(ItemStack stack, BlockState state) {
@@ -528,24 +541,23 @@ public class DrillItem extends ItemEnergy {
      * Breaks the Block, drops Particles etc.
      * Has to be called on both Server and Client
      *
-     * @param world   The World
+     * @param level   The Level
      * @param isExtra If the Block is the Block that was looked at when breaking or an additional Block
      * @param stack   The Drill
      * @param player  The Player breaking the Blocks
      * @param use     The Energy that should be extracted per Block
      */
-    private boolean tryHarvestBlock(Level world, BlockPos pos, boolean isExtra, ItemStack stack, Player player, int use) {
+    private boolean tryHarvestBlock(Level level, BlockPos pos, boolean isExtra, ItemStack stack, Player player, int use) {
-        BlockState state = world.getBlockState(pos);
+        BlockState state = level.getBlockState(pos);
-        Block block = state.getBlock();
+        float hardness = state.getDestroySpeed(level, pos);
-        float hardness = state.getDestroySpeed(world, pos);
 
-        boolean canHarvest = (player.hasCorrectToolForDrops(state) || this.isCorrectToolForDrops(stack, state)) && (!isExtra || this.getDestroySpeed(stack, world.getBlockState(pos)) > 1.0F);
+        boolean canHarvest = (player.hasCorrectToolForDrops(state) || this.isCorrectToolForDrops(stack, state)) && (!isExtra || this.getDestroySpeed(stack, level.getBlockState(pos)) > 1.0F);
         if (hardness >= 0.0F && (!isExtra || canHarvest && !state.hasBlockEntity())) {
             if (!player.isCreative()) {
-                this.extractEnergyInternal(stack, use, false);
+                this.extractEnergy(stack, use, false);
             }
             //Break the Block
-            return WorldUtil.breakExtraBlock(stack, world, player, pos);
+            return WorldUtil.breakExtraBlock(stack, level, player, pos);
         }
         return false;
     }

src/main/java/de/ellpeck/actuallyadditions/mod/items/ItemDrillUpgrade.java

@@ -12,7 +12,6 @@ package de.ellpeck.actuallyadditions.mod.items;
 
 import de.ellpeck.actuallyadditions.mod.components.ActuallyComponents;
 import de.ellpeck.actuallyadditions.mod.items.base.ItemBase;
-import net.minecraft.nbt.CompoundTag;
 import net.minecraft.network.chat.Component;
 import net.minecraft.world.InteractionHand;
 import net.minecraft.world.InteractionResult;

src/main/java/de/ellpeck/actuallyadditions/mod/items/base/ItemEnergy.java

@@ -103,7 +103,7 @@ public abstract class ItemEnergy extends ItemBase {
             .orElse(0);
     }
 
-    public int extractEnergyInternal(ItemStack stack, int maxExtract, boolean simulate) {
+    public int extractEnergyInternal(ItemStack stack, int maxExtract, boolean simulate) { //TODO: Check why this method doesn't work
         return Optional.ofNullable(stack.getCapability(Capabilities.EnergyStorage.ITEM))
             .map(cap -> cap instanceof EnergyStorage
                 ? cap.extractEnergy(maxExtract, simulate)
@@ -140,7 +140,17 @@ public abstract class ItemEnergy extends ItemBase {
 		return stack.getOrDefault(ActuallyComponents.ENERGY_STORAGE, 0);
 	}
 
-//    @Override TODO: Register Energy cap/attachment
+    @Override
+    public int getMaxDamage(ItemStack stack) {
+        return getEnergyStorage(stack);
+    }
+
+    @Override
+    public boolean isDamageable(ItemStack stack) {
+        return false;
+    }
+
+    //    @Override TODO: Register Energy cap/attachment
 //    public ICapabilityProvider initCapabilities(ItemStack stack, CompoundTag nbt) {
 //        return new EnergyCapabilityProvider(stack, this);
 //    }

src/main/java/de/ellpeck/actuallyadditions/mod/material/ArmorMaterials.java

@@ -1,13 +1,7 @@
 package de.ellpeck.actuallyadditions.mod.material;
 
 import de.ellpeck.actuallyadditions.mod.ActuallyAdditions;
-import net.minecraft.Util;
-import net.minecraft.core.Holder;
-import net.minecraft.core.Registry;
-import net.minecraft.core.registries.BuiltInRegistries;
 import net.minecraft.core.registries.Registries;
-import net.minecraft.resources.ResourceLocation;
-import net.minecraft.sounds.SoundEvent;
 import net.minecraft.sounds.SoundEvents;
 import net.minecraft.world.item.ArmorItem;
 import net.minecraft.world.item.ArmorMaterial;
@@ -16,10 +10,8 @@ import net.neoforged.bus.api.IEventBus;
 import net.neoforged.neoforge.registries.DeferredHolder;
 import net.neoforged.neoforge.registries.DeferredRegister;
 
-import java.util.EnumMap;
 import java.util.List;
 import java.util.Map;
-import java.util.function.Supplier;
 
 /**
  * Complete copy paste from {@link net.minecraft.world.item.ArmorMaterial}

src/main/java/de/ellpeck/actuallyadditions/mod/misc/apiimpl/farmer/DefaultFarmerBehavior.java

@@ -13,7 +13,6 @@ package de.ellpeck.actuallyadditions.mod.misc.apiimpl.farmer;
 import de.ellpeck.actuallyadditions.api.farmer.FarmerResult;
 import de.ellpeck.actuallyadditions.api.farmer.IFarmerBehavior;
 import de.ellpeck.actuallyadditions.api.internal.IFarmer;
-import de.ellpeck.actuallyadditions.mod.util.StackUtil;
 import net.minecraft.core.BlockPos;
 import net.minecraft.core.Direction;
 import net.minecraft.server.level.ServerLevel;
@@ -29,7 +28,6 @@ import net.minecraft.world.item.Items;
 import net.minecraft.world.level.Level;
 import net.minecraft.world.level.block.Block;
 import net.minecraft.world.level.block.Blocks;
-import net.minecraft.world.level.block.BonemealableBlock;
 import net.minecraft.world.level.block.CropBlock;
 import net.minecraft.world.level.block.StemBlock;
 import net.minecraft.world.level.block.state.BlockState;

src/main/java/de/ellpeck/actuallyadditions/mod/util/Util.java

@@ -15,7 +15,6 @@ import net.minecraft.world.entity.ai.attributes.AttributeInstance;
 import net.minecraft.world.entity.ai.attributes.Attributes;
 import net.minecraft.world.entity.player.Player;
 import net.neoforged.fml.loading.FMLLoader;
-import net.neoforged.neoforge.common.NeoForgeMod;
 
 public final class Util {
 

src/main/java/de/ellpeck/actuallyadditions/mod/util/WorldUtil.java

@@ -328,8 +328,6 @@ public final class WorldUtil {
             if (block.onDestroyedByPlayer(state, level, pos, player, true, state.getFluidState())) {
                 block.destroy(level, pos, state);
             }
-            // callback to the tool
-            stack.mineBlock(level, state, pos, player);
 
             // send an update to the server, so we get an update back