[1.20.1] NullPointerException when opening DisenchanterGui
AgentK20 opened this issue ยท 1 comments
Base information
- Minecraft version: 1.20.1
- Mod version: 3.1.2.583
- Minecraft Forge version: 47.3.5
- Mod Pack: Heavily modified version of Worldly Galaxies 4.18
Crash report
Description / steps to reproduce
Repro steps on my pack: Put an iron sword (unsure what enchants were on it) in the Disenchanter. Game crashes.
Pretty sure this crash isn't explicitly DE's fault (more likely a bad mod interaction with something changing how enchantments work), but looking through the source I can see that the DisenchanterGui is unsafely handling a potential null within DisenchanterGui.populateList. getEnchantmentFromTag can return null, which is then immediately passed into getCostInLevels which blows up, unlike other handling with the same file.
I'm going to keep digging on my side to see which of the mods is causing a null Enchantment to be returned, but a potential fix on the DE side would be to validate the return of getEnchantmentFromTag as non-null before allowing an item to be placed in the Disenchanter.
Thanks!