search

Globox1997 / tiered

https://modrinth.com/mod/tieredz
https://www.curseforge.com/minecraft/mc-mods/tieredz
MIT License
1 stars 15 forks source link

Modern industrialization incompatibility #46

Closed RatDonkeyGoat closed 1 year ago

RatDonkeyGoat commented 1 year ago

Basically, you'd have to change this https://github.com/Globox1997/tiered/blob/1.19/src/main/java/draylar/tiered/reforge/ReforgeScreen.java#L82 to something like else if (itemStack.getItem() instanceof ArmorItem armorItem) { if (armorItem.getMaterial().getRepairIngredient() != null) { baseItems.addAll(Arrays.asList(armorItem.getMaterial().getRepairIngredient().getMatchingStacks())); } } else { ...

(Didn't test it for errors and such, but that should avoid the NPE)

latest.log

Globox1997 commented 1 year ago

mh the repair ingredient should not be null but are there other cases like tools which have no ingredient either?

Globox1997 commented 1 year ago

Fixed with https://github.com/Globox1997/tiered/commit/08d3446f769395b076af459f35238b7496666486