MineColonies

MineColonies

53M Downloads

[BUG] Items are continuously being transported back and forth between the warehouses, which could potentially lead to a crash.

fulmoonless opened this issue ยท 0 comments

commented

Is there an existing issue for this?

  • I have searched the existing issues

Are you using the latest MineColonies Version?

  • I am running the latest beta version of MineColonies for my Minecraft version.

Did you check on the Wiki? or ask on Discord?

  • I checked the MineColonies Wiki and made sure my issue is not covered there. Or I was sent from discord to open an issue here.

What were you playing at the time? Were you able to reproduce it in both settings?

  • Single Player
  • Multi Player

Minecraft Version

1.20.1

MineColonies Version

1.20.1-1.1.647

Structurize Version

1.20.1-1.0.742

Related Mods and their Versions

Forge 47.2.0
WinterColonies-1.11

Current Behavior

I recently built a new warehouse (Warehouse A). After this, my old warehouse (Warehouse B) started sending items to Warehouse A. Once Warehouse A receives these items, it immediately sends them back to Warehouse B. The items involved are those that Warehouse B has set to maintain a Minimum Stock but hasn't yet reached that level. It seems like Warehouse B is requesting items from Warehouse A, and Warehouse A's solution is to retrieve items from Warehouse B to send them back to Warehouse B (which is kind of funny, haha).

Additionally, there's an issue where, for example, the mine requests torches from Warehouse B, and Warehouse B first asks Warehouse A to send the torches to Warehouse B, which then sends them to the mine.
image
image
Also, when I tested using the reproduction steps below, the game crashed directly. I'm not sure why it didn't crash during normal gameplay.

Expected Behavior

"Prevent warehouses from endlessly sending items back and forth, or better yet, prohibit warehouses from delivering to each other entirely.

If Warehouse B identifies that only Warehouse A can fulfill a request, it should automatically let Warehouse A handle it directly, instead of first having Warehouse A send the items to Warehouse B, which then sends them out."

Reproduction Steps

1.Build two warehouses.
2.Set two minimum stacks of stone in Warehouse A.
3.Place one stack of stone in Warehouse A.
4.It should have already crashed.

Logs

https://gist.github.com/fulmoonless/d009dfb002ce316f9a9a10ae01dafeaa

Anything else?

  • Add a thumbs-up to the bug report if you are also affected. This helps the bug report become more visible to the team and doesn't clutter the comments.
  • Add a comment if you have any insights or background information that isn't already part of the conversation.