Hephaestus tools don't fire Fabric API PlayerBlockBreakEvents
unilock opened this issue · 3 comments
Minecraft Version
1.20.1
Fabric Version
0.14.23
Fabric API Version
0.90.4+1.20.1
Hephaestus Version
1.20.1-3.6.3.236
Describe your issue
Tools created with Hephaestus do not fire Fabric API's PlayerBlockBreakEvents, breaking compatibility with many mods, including (for example) FallingTree and VeinDigging.
To diagnose this issue, I created a mod that sends a chat message to the player whenever any of Fabric API's PlayerBlockBreakEvents.BEFORE / AFTER events, Porting Lib's BlockEvents.BreakEvent.BLOCK_BREAK event, or Architectury API's BlockEvent.BREAK event is fired. (code: https://github.com/unilock/BreakEventTest)
Below are snippets of screenshots of the results of using a pickaxe from Hephaestus to break stone, and a shovel from vanilla Minecraft to break dirt, with said mod.
Hephaestus pickaxe:
Vanilla shovel:
(Yes, Porting Lib's event did fire twice with the pickaxe; not sure what's going on there 🤷♀️)
I'm aware of the existence of Tamago, a mod that fixed this issue with Architectury API's event (specifically) on 1.18.2.
Now, with Tamago being All Rights Reserved and closed source, it would be unethical for me to attempt to port it to 1.20.1. But I'm not one to let things like "the law" get in my way.
So I decompiled and re-created a version of it that adds support for Fabric API's events as well. It does work, but obviously it's not an ideal solution.
Crash Report
N/A
Other mods
- Fabric API
- Hephaestus
Tried reproducing with just Tinkers?
Yes
Performance Enchancers
None of the above
Searched for known issues?
Checked pinned issues, Searched open issues, Searched closed issues
I can look into fixing this soon, I plan on releasing a update to tamago very soon for 1.20.1, anyway here is the source for tamago I was just too lazy to upload it https://github.com/AlphaMode/Tamago/ its MIT not ARR
That is definitely not the same Tamago I was looking at :P
https://modrinth.com/mod/tamago/version/1.0
This is my much less comprehensive rendition:
https://github.com/unilock/NotTamago