Feature/block harvest (#174)

* Impl removedByPlayer and canHarvestBlock in IForgeBlock

* Apply kitlith's suggestions

* Fix ThreadLocal memory leakage

* Apply suggestions

* Remove unneeded dependency

* Disable an unnecessary 1.15 patch

* Move BlockEvent from events-world to extensions-block.

This removes a unnecessary dependency.

* Replace a redirect with injection

* Apply suggestions

* Merge remote-tracking branch 'origin/master' into feature/block-harvest

Conflicts:
	patchwork-events-world/src/main/java/net/patchworkmc/impl/event/world/WorldEvents.java

Co-authored-by: TheGlitch76 <glitchieproductionsofficial@gmail.com>

Author: rikka0w0

patchwork-events-world/src/main/java/net/patchworkmc/impl/event/world/WorldEvents.java

@@ -25,16 +25,10 @@
 import javax.annotation.Nullable;
 
 import net.minecraftforge.common.MinecraftForge;
-import net.minecraftforge.event.world.BlockEvent;
 import net.minecraftforge.event.world.ChunkEvent;
 import net.minecraftforge.event.world.ChunkWatchEvent;
 import net.minecraftforge.event.world.WorldEvent;
 
-import net.minecraft.block.BlockState;
-import net.minecraft.entity.player.PlayerEntity;
-import net.minecraft.item.ItemStack;
-import net.minecraft.util.DefaultedList;
-import net.minecraft.world.World;
 import net.minecraft.entity.EntityCategory;
 import net.minecraft.server.network.ServerPlayerEntity;
 import net.minecraft.server.world.ServerWorld;
@@ -78,13 +72,6 @@ public static void onWorldSave(IWorld world) {
 		MinecraftForge.EVENT_BUS.post(new WorldEvent.Save(world));
 	}
 
-	// TODO: Leaving this unfired is intentional. See: https://github.com/MinecraftForge/MinecraftForge/issues/5828
-	public static float fireBlockHarvesting(DefaultedList<ItemStack> drops, World world, BlockPos pos, BlockState state, int fortune, float dropChance, boolean silkTouch, PlayerEntity player) {
-		BlockEvent.HarvestDropsEvent event = new BlockEvent.HarvestDropsEvent(world, pos, state, fortune, dropChance, drops, player, silkTouch);
-		MinecraftForge.EVENT_BUS.post(event);
-		return event.getDropChance();
-	}
-
 	public static void fireChunkWatch(boolean watch, ServerPlayerEntity entity, ChunkPos chunkpos, ServerWorld world) {
 		if (watch) {
 			MinecraftForge.EVENT_BUS.post(new ChunkWatchEvent.Watch(entity, chunkpos, world));

patchwork-events-world/src/main/java/net/patchworkmc/mixin/event/world/MixinServerPlayerInteractionManager.java

@@ -5,10 +5,9 @@
   "mixins": [
     "MixinChunkGenerator",
     "MixinMinecraftServer",
-    "MixinServerPlayerInteractionManager",
     "MixinServerWorld",
     "MixinSpawnHelper",
-    "MixinThreadedAnvilChunkStorage",
+    "MixinThreadedAnvilChunkStorage"
   ],
   "client": [
     "MixinClientWorld",

patchwork-events-world/src/main/resources/patchwork-events-world.mixins.json

@@ -4,6 +4,7 @@ version = getSubprojectVersion(project, "0.3.0")
 dependencies {
 	implementation project(path: ':patchwork-api-base', configuration: 'dev')
 	implementation project(path: ':patchwork-enum-hacks', configuration: 'dev')
+	implementation project(path: ':patchwork-extensions-item', configuration: 'dev')
 	implementation project(path: ':patchwork-tooltype', configuration: 'dev')
 }
 

patchwork-extensions-block/build.gradle

@@ -27,6 +27,7 @@
 import javax.annotation.Nullable;
 
 import net.minecraftforge.common.IPlantable;
+import net.minecraftforge.common.ToolType;
 
 import net.minecraft.block.BedBlock;
 import net.minecraft.block.Block;
@@ -82,10 +83,12 @@
 import net.fabricmc.api.EnvType;
 import net.fabricmc.api.Environment;
 
+import net.patchworkmc.impl.extensions.block.BlockHarvestManager;
+import net.patchworkmc.impl.extensions.block.PatchworkBlock;
 import net.patchworkmc.mixin.extensions.block.FireBlockAccessor;
 import net.patchworkmc.mixin.extensions.block.PlantBlockAccessor;
 
-public interface IForgeBlock {
+public interface IForgeBlock extends PatchworkBlock {
 	default Block getBlock() {
 		return (Block) this;
 	}
@@ -204,18 +207,18 @@ default BlockEntity createTileEntity(BlockState state, BlockView world) {
 		return null;
 	}
 
-	/* TODO IForgeBlock#canHarvestBlock indirectly requires ToolType (via ForgeHooks#canHarvestBlock)
+	/* TODO IForgeBlock#canHarvestBlock indirectly requires ToolType (via ForgeHooks#canHarvestBlock) */
 	/**
 	 * Determines if the player can harvest this block, obtaining it's drops when the block is destroyed.
 	 *
 	 * @param world  The current world
 	 * @param pos    The block's current position
 	 * @param player The player damaging the block
 	 * @return True to spawn the drops
-	 *
+	 */
 	default boolean canHarvestBlock(BlockState state, BlockView world, BlockPos pos, PlayerEntity player) {
-		return ForgeHooks.canHarvestBlock(state, player, world, pos);
-	}*/
+		return BlockHarvestManager.canHarvestBlock(state, player, world, pos);
+	}
 
 	// TODO Call locations: Patches: ServerPlayerInteractionManager*
 	/**
@@ -240,7 +243,7 @@ default boolean canHarvestBlock(BlockState state, BlockView world, BlockPos pos,
 	 */
 	default boolean removedByPlayer(BlockState state, World world, BlockPos pos, PlayerEntity player, boolean willHarvest, FluidState fluid) {
 		getBlock().onBreak(world, pos, state, player);
-		return world.removeBlock(pos, false);
+		return world.setBlockState(pos, fluid.getBlockState(), world.isClient ? 11 : 3);
 	}
 
 	// TODO Call locations: Patches: LivingEntity*, PlayerEntity*, Forge classes: ForgeEventFactory (called from LivingEntity patch)
@@ -645,6 +648,7 @@ default boolean isBeaconBase(BlockState state, CollisionView world, BlockPos pos
 	// TODO Call locations: Forge classes: BreakEvent*
 	/**
 	 * Gathers how much experience this block drops when broken.
+	 * TODO: there's no equivalent callback in Fabric API, so for now Fabric mods should always return 0 here.
 	 *
 	 * @param state     The current state
 	 * @param world     The world
@@ -778,12 +782,12 @@ default boolean getWeakChanges(BlockState state, CollisionView world, BlockPos p
 		return false;
 	}
 
-	/* TODO IForgeBlock#getHarvestTool needs ToolType
+	/* TODO IForgeBlock#getHarvestTool needs ToolType */
 	/**
 	 * Queries the class of tool required to harvest this block, if null is returned
 	 * we assume that anything can harvest this block.
-	 *
-	ToolType getHarvestTool(BlockState state);*/
+	 */
+	ToolType getHarvestTool(BlockState state);
 
 	// TODO Call locations: Patches: PickaxeItem*, Forge classes: ForgeHooks*
 	/**
@@ -793,18 +797,18 @@ default boolean getWeakChanges(BlockState state, CollisionView world, BlockPos p
 	 */
 	int getHarvestLevel(BlockState state);
 
-	/* TODO IForgeBlock#isToolEffective needs ToolType
+	/* TODO IForgeBlock#isToolEffective needs ToolType */
 	/**
 	 * Checks if the specified tool type is efficient on this block,
 	 * meaning that it digs at full speed.
-	 *
+	 */
 	default boolean isToolEffective(BlockState state, ToolType tool) {
 		if (tool == ToolType.PICKAXE && (this.getBlock() == Blocks.REDSTONE_ORE || this.getBlock() == Blocks.REDSTONE_LAMP || this.getBlock() == Blocks.OBSIDIAN)) {
 			return false;
 		}
 
 		return tool == getHarvestTool(state);
-	}*/
+	}
 
 	// TODO Call locations: Forge classes: ForgeHooksClient
 	/**

patchwork-extensions-block/src/main/java/net/minecraftforge/common/extensions/IForgeBlock.java

@@ -25,6 +25,7 @@
 import javax.annotation.Nullable;
 
 import net.minecraftforge.common.IPlantable;
+import net.minecraftforge.common.ToolType;
 
 import net.minecraft.block.Block;
 import net.minecraft.block.BlockEntityProvider;
@@ -149,18 +150,18 @@ default BlockEntity createTileEntity(BlockView world) {
 		return patchwork$getForgeBlock().createTileEntity(getBlockState(), world);
 	}
 
-	/* TODO IForgeBlockState#canHarvestBlock indirectly requires ToolType
+	/* TODO IForgeBlockState#canHarvestBlock indirectly requires ToolType */
 	/**
 	 * Determines if the player can harvest this block, obtaining it's drops when the block is destroyed.
 	 *
 	 * @param world  The current world
 	 * @param pos    The block's current position
 	 * @param player The player damaging the block
 	 * @return True to spawn the drops
-	 *
+	 */
 	default boolean canHarvestBlock(BlockView world, BlockPos pos, PlayerEntity player) {
 		return patchwork$getForgeBlock().canHarvestBlock(getBlockState(), world, pos, player);
-	}*/
+	}
 
 	/**
 	 * Called when a player removes a block.  This is responsible for
@@ -582,14 +583,14 @@ default boolean getWeakChanges(CollisionView world, BlockPos pos) {
 		return patchwork$getForgeBlock().getWeakChanges(getBlockState(), world, pos);
 	}
 
-	/* TODO IForgeBlockState#getHarvestTool needs ToolType
+	/* TODO IForgeBlockState#getHarvestTool needs ToolType */
 	/**
 	 * Queries the class of tool required to harvest this block, if null is returned
 	 * we assume that anything can harvest this block.
-	 *
+	 */
 	default ToolType getHarvestTool() {
 		return patchwork$getForgeBlock().getHarvestTool(getBlockState());
-	}*/
+	}
 
 	default int getHarvestLevel() {
 		return patchwork$getForgeBlock().getHarvestLevel(getBlockState());
@@ -605,13 +606,15 @@ default boolean isToolEffective(ToolType tool) {
 	}*/
 
 	/**
+	 * TODO: do not bother implementing hooks, deprecated since 1.13
 	 * Can return IExtendedBlockState.
 	 */
 	default BlockState getExtendedState(BlockView world, BlockPos pos) {
 		return patchwork$getForgeBlock().getExtendedState(getBlockState(), world, pos);
 	}
 
 	/**
+	 * TODO: do not bother implementing hooks, deprecated since 1.15
 	 * Queries if this block should render in a given layer.
 	 * A custom {@link net.minecraft.client.render.model.BakedModel} can use {@link net.minecraftforge.client.MinecraftForgeClient#getRenderLayer()} to alter the model based on layer.
 	 */

patchwork-extensions-block/src/main/java/net/minecraftforge/common/extensions/IForgeBlockState.java

@@ -21,16 +21,21 @@
 
 import java.util.List;
 
+import net.minecraftforge.common.extensions.IForgeBlockState;
 import net.minecraftforge.eventbus.api.Event;
 
 import net.minecraft.item.ItemStack;
 import net.minecraft.util.DefaultedList;
 import net.minecraft.block.BlockState;
+import net.minecraft.enchantment.EnchantmentHelper;
+import net.minecraft.enchantment.Enchantments;
 import net.minecraft.entity.player.PlayerEntity;
 import net.minecraft.util.math.BlockPos;
 import net.minecraft.world.IWorld;
 import net.minecraft.world.World;
 
+import net.patchworkmc.impl.extensions.block.BlockHarvestManager;
+
 public class BlockEvent extends Event {
 	private final IWorld world;
 	private final BlockPos pos;
@@ -73,17 +78,14 @@ public BreakEvent(World world, BlockPos pos, BlockState state, PlayerEntity play
 			this.player = player;
 			this.exp = 0;
 
-			// TODO: BlockState#getExpDrop
-
-			/*
 			// Handle empty block or player unable to break block scenario
-			if (state == null || !ForgeHooks.canHarvestBlock(state, player, world, pos)) {
+			if (state == null || !BlockHarvestManager.canHarvestBlock(state, player, world, pos)) {
 				this.exp = 0;
 			} else {
-				int bonusLevel = EnchantmentHelper.getLevel(Enchantments.FORTUNE, player.getHeldItemMainhand());
-				int silklevel = EnchantmentHelper.getLevel(Enchantments.SILK_TOUCH, player.getHeldItemMainhand());
-				this.exp = state.getExpDrop(world, pos, bonusLevel, silklevel);
-			}*/
+				int bonusLevel = EnchantmentHelper.getLevel(Enchantments.FORTUNE, player.getMainHandStack());
+				int silklevel = EnchantmentHelper.getLevel(Enchantments.SILK_TOUCH, player.getMainHandStack());
+				this.exp = ((IForgeBlockState) state).getExpDrop(world, pos, bonusLevel, silklevel);
+			}
 		}
 
 		public PlayerEntity getPlayer() {
@@ -123,6 +125,7 @@ public boolean isCancelable() {
 	 *
 	 * <p>{@link #isSilkTouching} is set if this is considered a silk touch harvesting operation, vs a normal harvesting operation. Act accordingly.</p>
 	 */
+	@Deprecated
 	public static class HarvestDropsEvent extends BlockEvent {
 		private final int fortuneLevel;
 		private final DefaultedList<ItemStack> drops;