ProjectAres icon indicating copy to clipboard operation
ProjectAres copied to clipboard

Hooked items are converted to fish

Open Marcono1234 opened this issue 8 years ago • 0 comments

The bug

Based on the fix of MC-101001 you can now hook item entities and reel them in. The problem is that PGMListener.nerfFishing(PlayerFishEvent) converts any item entity even if it is hooked and not catched and therefore existed in the world before. This allows for example to "destroy" wool in the enemy's wool room.

The method should probably only convert the item if the state is PlayerFishEvent.State.CAUGHT_FISH.

However, to make sure a team cannot reel in the enemies wool maybe add a listener for a ProjectileHitEvent and then either prevent pulling items completely or prevent it for certain items like wool.

How to reproduce

This can be currently seen on play.stratus.network (but not on the mapdev server) since the treasure fishing loot is disabled there.

  1. Get a fishing rod (this is for example possible in "Persisto")
  2. Throw an item on the ground
  3. Cast the fishing rod at the item
  4. Try to reel in the item

Marcono1234 avatar Jun 17 '17 13:06 Marcono1234