Fixed Autobazaar retrying to buy indefinitely when the inventory is full #383
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixed a problem where, when the inventory is full and the full-inventory failsafe is bypassed (idk how), AutoSprayonator Autobazaar would keep buying spray material indefinitely. This happened because it couldn't detect the material in the inventory, and continued until the player either ran out of money or an evacuation occurred.
I was unable to replicate how AutoBazaar bypassed the full-inventory failsafe, but I only could reproduce it by setting the failsafe stop delay to the maximum. In that case, AutoBazaar tries to buy spray material before the failsafe stops all actions, even when the inventory is full. The instantly bought spray material is then transferred to the stash, causing Autobazaar to repeat the process indefinitely.
This happened to me once while I was AFK, Autobazaar kept instantly buying spray material until an evacuation occurred, leaving me with 300M worth of plant matter.
This issue is resolved by performing an inventory check before executing AutoSprayonator AutoBazaar, which prevents any actions if the inventory is full.
Let me know if you need logs or clip from the evacuation