
@Inject with unconditional cancel at "HEAD" should be replaced with @Overwrite
supersaiyansubtlety opened this issue ยท 4 comments
This has the same effect as an @Overwrite
except that when other mods mixin
- if their inject aren't required there's no log of it (which there should be)
- if their injects are required they don't fail (which they should)
The current inject just makes issues harder to track down.
That's not possible because of Fabric API injecting into it too:
https://github.com/FabricMC/fabric/blob/1.20.1/fabric-events-interaction-v0/src/client/java/net/fabricmc/fabric/mixin/event/interaction/client/MinecraftClientMixin.java#L67
It's a rather difficult situation, same with the issue with ICT.
Since PBP changes so much I have no clue how I could work around this issue properly.
Could PBP use the FAPI event?
If it did, I think that would actually fix #18
In the events branch I've added the 2 Fabric API events at pretty much the same locations.
ClientPickBlockGatherCallback: Right after getting getting the raytrace hit
ClientPickBlockApplyCallback: Right before the inventory manager does its job
I believe that should at least resolve this issue in the best way possible
Though I've got a feeling this won't be enough for #18, though I'll explain that over there