Refined Storage

Refined Storage

77M Downloads

Mod interaction bug: Yeta wrench can rotate cables into invalid positions

falkreon opened this issue ยท 1 comments

commented

Issue description:

Importers, exporters, and remote storage cables all have an orientation determining which side the "port" appears on. The YetaWrench, from EnderIO, uses a straightforward Block::rotateBlock implementation (source here) to rotate blocks which understand rotation inherently. Importers, exporters and remote storage cables respond to these requests, but can arrive at invalid states by doing so.

What happens:

Rotating from one state to another functions appropriately, until you arrive at an invalid (for the cable arrangement) rotation state. An example is the cable facing the chest, and the port facing one of the adjacent cables the tile used to be attached to. Invalid states appear to crash the server. Luckily, the server is able to restart fine, and the offending cable configuration can then be cleared.

What you expected to happen:

Rotation should skip invalid states, or, failing that, unworking states would sever network nodes requiring that tile.

Steps to reproduce:

  1. Place and power a controller
  2. Place a chest near the controller
  3. Place an external storage cable so that it attaches to the chest
  4. Connect the controller to the external storage cable using regular cable
  5. Using the yeta or any similar wrench, rotate the external storage until the port faces the cable connecting it to the controller.

At this point the game will crash.

Version (Make sure you are on the latest version before reporting):

(this is a SkyFactory 3.06 environment)

  • Minecraft: 1.10.2
  • Forge: 12.18.3.2234
  • Refined Storage: 1.2.18

Does this issue occur on a server? Yes

If a (crash)log is relevant for this issue, link it here:

(I appologize, I know a stack trace would be incredibly useful, but I'm not the server owner. If there are problems tracking this down I can try and repro in a single-player world)

commented

Whoops, this is probably a duplicate of the fixed #915. Gonna close this one out.