agilbert1412 / StardewArchipelago

Archipelago Implementation for Stardew Valley
11 stars 12 forks source link

Coffee shows archipelago icon when not needed #55

Closed GFreak01 closed 3 months ago

GFreak01 commented 3 months ago

My game currently has "Coffee Bean" set for "Harvest" and "Shipsanity". However, the coffee in my inventory shows the archipelago icon, as if it were necessary to ship. (The hover text says "Harvest Coffee Bean, Shipsanity: Coffee Bean")

I assume the "Coffee" is pulling the icon status from "Coffee beans" due to the names being so similar.

agilbert1412 commented 3 months ago

It is indeed due to names, as the matching for these icons is very crude. It's basically just "check if the text is contained within the location name", with a bit of fluff like exceptions.

But this case is an item that is indirectly related, but related nonetheless, and these, we have decided to keep them as they might be useful to someone (ex: deciding not to brew your next batch and ship some instead)