OoT-Randomizer icon indicating copy to clipboard operation
OoT-Randomizer copied to clipboard

Various fixes to misc. location hints

Open fenhl opened this issue 1 year ago • 3 comments

  • The names for misc. location hint locations are changed for consistency with the item locations hinted on them. For example, “10 Skulltulas Reward Hint” is renamed to “Kak 10 Gold Skulltula Reward Hint”. This shouldn't break plandos because these aren't item locations.
  • The feature allowing hint distributions to customize the locations hinted on misc. location hints is removed, as it was never implemented properly: the hint text was not adjusted to account for the changed locations, leading to misleading text, and the “already hinted” system always assumed the default location. The alternative of making the customization work properly runs into issues with making arbitrary hint text fit into the various text boxes, so I don't think it's feasible to implement.
  • The logic for the “ZR Frogs Ocarina Game Hint” location (previously called “ZR Frogs Ocarina Minigame Hint”) is fixed to require access as child.
  • The misc. hints for unique merchant locations are incorporated into the misc. location hint system. This prevents them from being shuffled when text is “Shuffled except Important Text”, and marks them as “already hinted”, preventing gossip stones from giving hints that are made redundant by them. This part of the PR is based on #2170, special thanks to @flagrama for calling attention to these bugs and writing the initial fix.

Testing

This has not yet been tested. Two things need to be tested:

  1. The change to ZR Frogs Ocarina Game Hint should cause a frogs 2 hint to be generated on a gossip stone if child river is fully inaccessible
  2. The changes to unique merchant hints should stop gossip stones hinting these locations when reachable

fenhl avatar Jan 14 '25 13:01 fenhl

I usually put non-locations like drops/hints and things toward the bottom of location list in any given region. There's absolutely no reason for this (I guess helps to tell real checks from the fake ones) but now that we've got two people arbitrarily moving the location orders around we're gonna end up in an edit war lol.

r0bd0g avatar Jan 14 '25 16:01 r0bd0g

I'm curious what setting the frogs hint to is_child actually does. Like, does it free the location up to be hinted if it turns out that child can't access the river, is that what it does?

r0bd0g avatar Jan 14 '25 16:01 r0bd0g

I usually put non-locations like drops/hints and things toward the bottom of location list in any given region. There's absolutely no reason for this (I guess helps to tell real checks from the fake ones) but now that we've got two people arbitrarily moving the location orders around we're gonna end up in an edit war lol.

I moved them to be paired like this because they are closely related, and so you more easily understand how their access requirements relate.

I'm curious what setting the frogs hint to is_child actually does. Like, does it free the location up to be hinted if it turns out that child can't access the river, is that what it does?

Yeah, I guess that's the only actual implication here. I mostly just fixed it because the current logic doesn't match the actual in-game access requirements, and all the other ones have their actual access requirements rather than just True, so I believe this was an oversight.

fenhl avatar Jan 14 '25 16:01 fenhl