Missing Protections for Fish Bucket Placement

[cjburkey01]

Check for new spigot events to listen to

[Goldmensch]

I think this can be handled by the PlayerInteractEvent

[cjburkey01]

I think this can be handled by the PlayerInteractEvent

Do you know how to implement this? I added a handler for PlayerBucketEntityEvent in b7201d3 but if this is handled by the PlayerInteractEvent, shouldn't it be handled by this handler?

I'm going to have a better look at this mess of a function. I don't want to have to handle by making sure it's a fish bucket because I feel like that would lead to a bunch of compatibility problems if they ever change, especially across versions.