Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Honey in BAGI isn't displayed properly when ready to be harvested. #21

Open
voreisthescore opened this issue Apr 15, 2021 · 2 comments

Comments

@voreisthescore
Copy link

Honey as an Item ready to be harvested shows the default honey icon, yet once harvested and in inventory the correct asset is shown. Unfortunately I can't narrow down what could be causing this, as an uneducated guess possibly due to the fact that from Bee Houses, you only get one item i.e. Honey, yet with kegs you can have multiple possible outputs hence multiple assets. This issue is solely relegated to Bee Houses and Honey, everything works as intended with Kegs, Casks etc.

@voltaek
Copy link

voltaek commented Feb 24, 2024

I know this issue is going on 3 years old, but I just wanted to let you know this is due to how the game handles the harvesting of honey. Unlike a keg or preserve bin, where the item to be harvested from it is known as soon as it becomes ready to harvest, the type of honey you collect is (in the base game's logic, at least) not determined until you actually remove the honey from the bee house. Per the wiki: "The honey type is determined when harvested, rather than when it becomes ready".

So the bee house's item-ready-to-be-harvested is technically the base honey until you pull it out, at which point the game checks for nearby flowers and if it finds a qualifying one, modifies the honey's name and price as it puts it into your inventory. Therefore the mod is showing the image/icon for the base honey, since that's technically what's there at the moment.

That's not to say that this or some other mod couldn't alter that logic, but I assume that checking for nearby flowers over a swath of nearby grid tiles would be somewhat computationally expensive, so you wouldn't want to be running that check too often.

@voltaek
Copy link

voltaek commented Mar 10, 2024

@voreisthescore I created a standalone mod to handle this: Honey Harvest Sync

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants