Implement ItemFishedEvent (#147)

* Implement ItemFishedEvent

* Make event cancelable

* Actually use the event's rod damage

* </p> nit

Co-authored-by: TheGlitch76 <[email protected]>

Author: cittyinthecloud

patchwork-events-entity/src/main/java/net/minecraftforge/event/entity/player/ItemFishedEvent.java

@@ -0,0 +1,92 @@
+/*
+ * Minecraft Forge, Patchwork Project
+ * Copyright (c) 2016-2020, 2019-2020
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation version 2.1
+ * of the License.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+package net.minecraftforge.event.entity.player;
+
+import java.util.List;
+
+import javax.annotation.Nonnegative;
+
+import com.google.common.base.Preconditions;
+
+import net.minecraft.entity.projectile.FishingBobberEntity;
+import net.minecraft.item.ItemStack;
+import net.minecraft.util.DefaultedList;
+
+/**
+ * This event is called when a player fishes an item.
+ *
+ * <p>This event is cancellable.
+ * Canceling the event will cause the player to receive no items at all.
+ * The hook will still take the damage specified</p>
+ */
+
+public class ItemFishedEvent extends PlayerEvent {
+	private final DefaultedList<ItemStack> stacks = DefaultedList.of();
+	private final FishingBobberEntity hook;
+	private int rodDamage;
+
+	public ItemFishedEvent(List<ItemStack> stacks, int rodDamage, FishingBobberEntity hook) {
+		super(hook.getOwner());
+		this.stacks.addAll(stacks);
+		this.rodDamage = rodDamage;
+		this.hook = hook;
+	}
+
+	/**
+	 * Get the damage the rod will take.
+	 *
+	 * @return The damage the rod will take
+	 */
+	public int getRodDamage() {
+		return rodDamage;
+	}
+
+	/**
+	 * Specifies the amount of damage that the fishing rod should take.
+	 * This is not added to the pre-existing damage to be taken.
+	 *
+	 * @param rodDamage The damage the rod will take. Must be nonnegative
+	 */
+	public void damageRodBy(@Nonnegative int rodDamage) {
+		Preconditions.checkArgument(rodDamage >= 0);
+		this.rodDamage = rodDamage;
+	}
+
+	/**
+	 * Use this to get the items the player will receive.
+	 * You cannot use this to modify the drops the player will get.
+	 * If you want to affect the loot, you should use LootTables.
+	 */
+	public DefaultedList<ItemStack> getDrops() {
+		return stacks;
+	}
+
+	/**
+	 * Use this to stuff related to the hook itself, like the position of the bobber.
+	 */
+	public FishingBobberEntity getHookEntity() {
+		return hook;
+	}
+
+	@Override
+	public boolean isCancelable() {
+		return true;
+	}
+}

patchwork-events-entity/src/main/java/net/patchworkmc/mixin/event/entity/MixinFishingBobberEntity.java

@@ -0,0 +1,72 @@
+/*
+ * Minecraft Forge, Patchwork Project
+ * Copyright (c) 2016-2020, 2019-2020
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation version 2.1
+ * of the License.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
+ */
+
+package net.patchworkmc.mixin.event.entity;
+
+import java.util.List;
+
+import net.minecraftforge.common.MinecraftForge;
+import net.minecraftforge.event.entity.player.ItemFishedEvent;
+import org.spongepowered.asm.mixin.Mixin;
+import org.spongepowered.asm.mixin.Shadow;
+import org.spongepowered.asm.mixin.Unique;
+import org.spongepowered.asm.mixin.injection.At;
+import org.spongepowered.asm.mixin.injection.Inject;
+import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
+import org.spongepowered.asm.mixin.injection.callback.LocalCapture;
+
+import net.minecraft.entity.projectile.FishingBobberEntity;
+import net.minecraft.item.ItemStack;
+import net.minecraft.loot.LootTable;
+import net.minecraft.loot.context.LootContext;
+
+@Mixin(FishingBobberEntity.class)
+public abstract class MixinFishingBobberEntity {
+	@Shadow
+	private boolean field_7176; // is stuck to block
+
+	@Shadow
+	public abstract void remove();
+
+	@Unique
+	private final ThreadLocal<Integer> rodDamage = ThreadLocal.withInitial(() -> -1);
+
+	@Inject(method = "method_6957", at = @At(value = "INVOKE", target = "Lnet/minecraft/advancement/criterion/FishingRodHookedCriterion;trigger(Lnet/minecraft/server/network/ServerPlayerEntity;Lnet/minecraft/item/ItemStack;Lnet/minecraft/entity/projectile/FishingBobberEntity;Ljava/util/Collection;)V", ordinal = 1), cancellable = true, locals = LocalCapture.CAPTURE_FAILHARD)
+	private void onReeledIn(ItemStack itemStack, CallbackInfoReturnable<Integer> cir, int i, LootContext.Builder builder, LootTable lootTable, List<ItemStack> list) {
+		ItemFishedEvent event = new ItemFishedEvent(list, this.field_7176 ? 2 : 1, (FishingBobberEntity) (Object) this);
+		MinecraftForge.EVENT_BUS.post(event);
+
+		if (event.isCanceled()) {
+			this.remove();
+			cir.setReturnValue(event.getRodDamage());
+		} else {
+			rodDamage.set(event.getRodDamage());
+		}
+	}
+
+	@Inject(method = "method_6957", at = @At("RETURN"), cancellable = true)
+	private void onRodDamage(ItemStack itemStack, CallbackInfoReturnable<Integer> cir) {
+		Integer damage = rodDamage.get();
+
+		if (damage != -1) {
+			cir.setReturnValue(damage);
+			rodDamage.set(-1);
+		}
+	}
+}

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

@@ -8,6 +8,7 @@
     "MixinEntityTrackerEntry",
     "MixinEntityType",
     "MixinExperienceOrbEntity",
+    "MixinFishingBobberEntity",
     "MixinFurnaceOutputSlot",
     "MixinItemEntity",
     "MixinLivingEntity",