search

Shurgent / TFCTech

TFCTech - Addon for TerraFirmaCraft. Custom integration and additional items for TFC + Tech-mods fusion. Base for the TFCTech Modpack (in future).
GNU General Public License v3.0
12 stars 18 forks source link

Fixes crash on hopper item insertion into side of Induction Smelter #21

Closed LudwigVonChesterfield closed 2 years ago

LudwigVonChesterfield commented 2 years ago

NullPointerException occured due to getAccesibleSlotsFromSide returning a null instead of an empty list of slots.

Additionally, this allows to put items not only from the top, but from the sides too, to give player more interaction-freedom (Which might have to be seperated into a different commit, if only two insertion points were very much intended)

Shurgent commented 2 years ago

Thanks for finding the bug. I'm actually surprised that someone is still using my mod. I abandoned it many years ago. What about the ability to load items from the side. As you say - only two insertion points were very much intended. :)

LudwigVonChesterfield commented 2 years ago

Now it only loads from the top.

Regarding mod use, it is used in TerraFirmaPunk pack, which I am fond of, and so I am trying to fix up all the crashes in it.