Fix AddCheckToLogic bombchu handling (#7274)

1. consistently include shop bombchus
2. fix boolean flip typo, that was filtering out ITEMTYPE_SHOP branch that coerces matching LogicVals
This commit is contained in:
Philip Dubé
2026-09-28 16:16:27 +00:00
committed by GitHub
parent 4f546bc87e
commit c42004bc42
@@ -423,12 +423,12 @@ bool AddCheckToLogic(LocationAccess& locPair, GetAccessibleLocationsStruct& gals
ApplyOrStoreItem(location, gals, addToPlaythrough);
}
// If we want to ignore bombchus, only add if bombchu is not in the name
else if (IsBombchus(ignore) && !IsBombchus(locItem, true)) {
else if (IsBombchus(ignore, true) && !IsBombchus(locItem, true)) {
ApplyOrStoreItem(location, gals, addToPlaythrough);
}
// We want to ignore a specific Buy item. Buy items with different RandomizerGets are recognised by a
// shared GetLogicVal
else if (ignore != RG_GOLD_SKULLTULA_TOKEN && IsBombchus(ignore)) {
else if (ignore != RG_GOLD_SKULLTULA_TOKEN && !IsBombchus(ignore, true)) {
if ((type == ITEMTYPE_SHOP && Rando::StaticData::GetItemTable()[ignore].GetLogicVal() !=
location->GetPlacedItem().GetLogicVal()) ||
type != ITEMTYPE_SHOP) {