Forces Inventorio to pull out the apropriate Mining tool all the time
Enderteck opened this issue ยท 8 comments
Mod loader
NeoForge
Minecraft version
1.20.1
Mod version
11.7.1
Modloader version
Neoforge 47.1.85
Modpack info
Custom
If bug:
- Can you reproduce this issue with relevant mods only?
If bug: The latest.log file
No response
Issue description
If Jade is enabled, the tool belt from Inventorio is always putting the appropriate tool to mine the block looked at:
This doesn't happen if you disable the "Display Tooltip" is enabled:
Without it, there is no problem, and the tool only comes out when starting to break a block but Jade's tooltip is disabled:
This is very headache inducing as when running through fields or generally places with different block types. Your tool hand changes every second and its very annoying:
Screen-Recording-2024-01-15.16-31.video-converter.com.webm
Since it only happens because of this mod, others on the Inventorio Github page said it's a problem on jade's part, so that's why I'm submitting this bug report. This could also be fixed in Jade Addons with an integration.
Will the mod update to fix this be published soon (for neforge1.20.1) ? Just wondering.
Better workaround:
As long as the displayTool
is set to a non-empty stack it is treated as being in the player's main hand, even on the server. This is required for e.g. the durability to be used up correctly
Hi! Inventorio maintainer here. I'm pretty sure this is caused by the following line:
Jade/src/main/java/snownee/jade/addon/harvest/HarvestToolProvider.java
Lines 108 to 109 in f7a2ddf
the state.getDestroyProgress
call internally calls a method which Inventorio injects into to auto-select the "best" tool. In vanilla, that method is only called when actually "attacking" a block. Now I'm not sure what the best way to resolve this is. I guess I could inspect the stack trace in Inventorio and not select the tool if the call comes from within Jade, but that seems very hacky and is also not very future proof with supporting other mods where the same thing might happen. But I also don't think Jade can do much about this. So I welcome any suggestions.
Thanks in advance!
I should also mention: A temporary "fix" is to disable the "Harvest Tool" setting in Jade, but that will obviously completely disable that feature.
Based on what I understand, your mod only switches the tool visually. If that's the case, maybe you could change the display tool only when the player is mining block.
Ok, because of how easy the stacktrace inspection is, I just did that for now to at least have it working with Jade specifically. I'm still open to other suggestions though.