search

LeFauxMatt / StardewMods

Stardew Valley mods
MIT License
23 stars 38 forks source link

Fix for AutoOrganize only organizing some chests #92

Closed Belaith closed 6 months ago

Belaith commented 6 months ago

AutoOrganize removed containersFrom-chests when it couldn't insert into a to containerTo-chest once. When for example the containerTo-chest had a filter setting that didn't match the items in the containersFrom-chest the containersFrom-chest stopped checking with other chests.

TobiasChen commented 6 months ago

Seconding this, because of this, AutoOrganize really only works with one set of machines at the same time. Would be a great help to get this fix in.

As far as I can see, is the current conflict caused by the additional log changes. Would @LeFauxMatt you prefere, if that commit is removed / is there anything else I could do to help this fix along?

LeFauxMatt commented 6 months ago

Sorry I missed this PR and thank you for sharing the fix. I've incorporated the one line that fixes the issue into the next update of Better Chests.