StrangeLoopGames / EcoIssues

131 stars 21 forks source link

USER ISSUE: Consolidate Inventories crash #23102

Closed madcatandrew closed 2 years ago

madcatandrew commented 2 years ago

\ User Id: (Not logged user) Version: 0.9.5.0 beta release-301

Steps to Reproduce: Accessed large lumber stockpile, various chests nearby and powered cart. Clicked consolidate inventories, crashed.

Expected behavior: Consolidated I guess.

Actual behavior: The following error message popped up:

Server encountered an exception:

<size=60.00%>Exception: DivideByZeroException
Message:Attempted to divide by zero.
Source:Eco.Gameplay

System.DivideByZeroException: Attempted to divide by zero.
   at Eco.Gameplay.Items.InventoryChangeSet.ConsolidateInventory(Inventory inventory, User user)
   at Eco.Gameplay.Components.LinkComponent.ConsolidateLinkedInventory(User user)
Outer Exceptions:
Exception has been thrown by the target of an invocation.
Exception has been thrown by the target of an invocation.</size>

Do you have mods installed? Does the issue happen when no mods are installed?: No mods

**Please provide log files. Instruction is here https://github.com/StrangeLoopGames/EcoIssues/wiki/Getting-the-Eco-game-log-file

madcatandrew commented 2 years ago

After rejoin, and persistent across server restarts, many benches, doors and chests near the stockpile which caused the crash are now completely invisible, but still accessible from other inventories in range.

Rejoining the server causes the exact benches and chests which are invisible to change to different ones.

SlayksWood commented 2 years ago

@madcatandrew Can you provide a couple of screenshots of your storages before clicking the button?

madcatandrew commented 2 years ago

Player.log Player-prev.log 20220428082618_1 20220428082636_1 20220428082649_1 20220428082657_1 20220428082701_1

madcatandrew commented 2 years ago

Leaving my server backup pre-update here to reproduce from. Backup was about 60m from house, looking towards it. Reproduction of the error reliably happens upon consolidating with any container near my house.

Also including a new log file since I can reproduce it anytime, and I think those prior logs were not from an active crash. This one definitely includes it.

World.zip Player.log

D3nnis3n commented 2 years ago

Should be fixed in 9.5.1, please reopen if it reoccurs.