AztechMC / Modern-Industrialization

Modern industrial mod for Minecraft.
https://www.curseforge.com/minecraft/mc-mods/modern-industrialization
MIT License
155 stars 86 forks source link

Return default value for hit index and hit side instead of throwing #849

Closed JustinHuPrime closed 3 months ago

JustinHuPrime commented 3 months ago

Fixes #847

Instead of throwing, we should return a reasonable default value - in this case, use the direction hit as the face hit and not deal with the overlay. (Additionally, we should consider only querying the overlay for wrench hits instead of all hits.)

JustinHuPrime commented 3 months ago

This fix should be okay to back-port if needed - the relevant code hasn't been changed recently

Technici4n commented 3 months ago

Thanks!