search

Malorolam / LootBags

Other
21 stars 31 forks source link

Fix multiple duplication bugs and NPE #209

Open jackodsteel opened 4 years ago

jackodsteel commented 4 years ago

Fix #185 where it removes 2 but only gives one lootbag when shift clicking

Fix #188 fix #178 where using the number keys gives a stack when it should only give 1.

Pretty sure it fixes #191, although their description seems the opposite of the bug I'm seeing.

Also fixed two bugs I found myself. One where using pipes (tested with EnderIO), it will extract a larger number of bags than it removes, thus duping bags. Second, a null pointer when interacting with the bags GUI sometimes.

jackodsteel commented 4 years ago

Hey @Malorolam, any chance of you taking a look at this PR? Happy to help with general maintainance too if you'd like, for example replying to/closing issues.