TogAr2

> And when you switch item slots when having cast a fishing rod the dobber does not disappear and stays even when not holding a fishing rod. Is this issue...

Stale

Doesn’t this do effectively nothing? Since putIfAbsent won’t change the maps contents if the key is already present, the cached value will be the same

As a temporary workaround it would also be possible to cancel `CrystalPlaceEvent` instead of rewriting the feature. Will look at a fix soon.

I originally planned to implement `DataPredicate`s as well, but they are already available at https://github.com/GoldenStack/trove, along with more loot table stuff. I imagine we would eventually want Minestom to support...

I don't know why the build failed but I don't think it is an issue with the PR

Do you mean just the damage animation? It is also present in vanilla. I'm not sure what you are using viaversion for but older versions have a different animation so...

Using the latest commit, you should be able to fix your issue by using `FinalDamageEvent.setAnimationType(AnimationType.LEGACY)`

I see, this is likely an issue with Minestom. It always sends the damage animation even if you disable it. https://github.com/Minestom/Minestom/blob/96bf14500af6e7cd2fad928eec992db53c6bf9ec/src/main/java/net/minestom/server/entity/LivingEntity.java#L344 This line should probably move up inside the `if...

It doesn’t deal the damage, it sends the locational damage event. There are two of those. Line 361 deals the damage.