Add support for modded tridents to trigger the thrown trident advancement #2009
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Eventually Mojang might end up making an entity type tag for tridents similar to the one they have for arrows, but until then there isn't a way for mods to have custom tridents trigger the thrown_trident advancement. This PR has us override the advancement to check for any subclass of ThrownTrident. Given I advancement criteria are only checked server side, if it is preferred we could instead add a entity type tag for tridents, and then check that instead of a custom sub predicate that does an instance check.
I also slightly refactored the code I wrote previously for replacing entity predicates (for worn equipment checks) to make it easier to extend for replacing different parts of an
EntityPredicate