Extended Crafting

Extended Crafting

34M Downloads

[BUG] Crash while replaceing crafting table with automation interface.

Water-Moon opened this issue ยท 1 comments

commented

Game Version: 1.12.2
Mod Version: 1.5.2 (current latest version)
Crash Report: https://paste.dimdev.org/alakugepil.mccrash

The operation that causes this:

  1. Put an extended crafting table. (For example, tire 2 crafting table)
  2. Put an automation interface, and set the recipe.
  3. After doing some crafting via the automation interface, break the crafting table and replace it with a table of another kind. (In this case, replace tire 2 table with tire 4 table)
    -> After a short while, the game crashed. According to the crash report, it is due to index out of boundary.

What actually happened:
Game crashed. On a server, the server will crash. The map cannot be opened anymore.
Loading the map with this issue will cause the game to crash again.

What is expected:
Items just drop on to the ground or at least the game does not crash.

Edit Reason: formatting and fixing typo

commented

Fixing suggestion:
Add a variable on tile Automation Interface program that saves which kind of table is being used for the recipe.
Then, in "CheckStackSmartly" method, if current table != table saved, return false.