Malorolam / LootBags

Other
21 stars 31 forks source link

IllegalArgumentException in LootbagsUtil.generateStacks #159

Open mcsoapthgr8 opened 5 years ago

mcsoapthgr8 commented 5 years ago

Lootbags version: 1.12.2-2.5.7 Forge version: 14.23.5.2768

Using Stoneblock modpack: version 1.0.20

Crash file: https://gist.github.com/mcsoapthgr8/c6a5224119ea763e8fa5d24b37c2ae9b Config files lootbags.cfg: https://gist.github.com/mcsoapthgr8/3ff17627b5b9e428dd4f268f7bc6a4cc Lootbags_BagConfig.cfg: https://gist.github.com/mcsoapthgr8/a6dbbeeb7901aadec343d260fa2bd247

I have a Lootbag opener attached to a mob farm. I was not near the mob farm at the time, so I'm not sure what type of bag it was trying to open when this occurred.

Please let me know if you need further info from me for tracking this down.

Malorolam commented 5 years ago

It seems like there's an item who has a minimum count larger than the maximum, these are the ones I saw in the config: line 139: minecraft:gunpowder:0:4:2:20 line 146: minecraft:fish:2:3:2:20 The second number should be smaller or equal to the third number. Fixing those should prevent the crash. I'll add a catch to the code to stop similar problems in the future but it'll be a little while before I can make a proper release.

Rodderik commented 5 years ago

It seems like there's an item who has a minimum count larger than the maximum, these are the ones I saw in the config: line 139: minecraft:gunpowder:0:4:2:20 line 146: minecraft:fish:2:3:2:20 The second number should be smaller or equal to the third number. Fixing those should prevent the crash. I'll add a catch to the code to stop similar problems in the future but it'll be a little while before I can make a proper release.

Confirmed swapping the middle numbers on just those 2 lines in server config stops crashing. line 139: minecraft:gunpowder:0:2:4:20 line 146: minecraft:fish:2:2:3:20