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

ER logic bug with shops behind the GTG entrance #2011

Open
ETR-BTF opened this issue Jun 18, 2023 · 2 comments
Open

ER logic bug with shops behind the GTG entrance #2011

ETR-BTF opened this issue Jun 18, 2023 · 2 comments
Labels
Component: ASM/C Changes some internals of the ASM/C libraries Component: Logic Non-trivial changes to the JSON logic files Status: Help Wanted Extra attention is needed Type: Bug Something isn't working

Comments

@ETR-BTF
Copy link

ETR-BTF commented Jun 18, 2023

Long known (and memed) about, but the 10 Rupee entrance fee could be a problem if behind that entrance is a shop or Business Scrub that requires a full Wallet with no way to farm Rupees. Neither logic nor the ER fill algorithm currently accounts for this. The easiest solutions are likely to either remove the entrance fee (optionally depending on ER settings), or to make opening GTG a permanent flag (optionally depending on ER settings).

@fenhl fenhl added the Type: Bug Something isn't working label Jun 18, 2023
@fenhl
Copy link
Collaborator

fenhl commented Jun 18, 2023

This bug is also tracked as Roman971#50, since it's currently only relevant on Dev-R. Since the plan is to merge mixed pools once the last two remaining blockers (grotto save/death/voidwarp weirdness and overworld entrance parity) are fixed, I feel like it makes sense to keep this issue open as well.

@DuskTheUmbreon
Copy link

Small note: This is a bug only for glitchless logic.

There's a fairly easy glitch (base logic on my fork) that allows bypassing both GF gates itemlessly.

@fenhl fenhl added Status: Help Wanted Extra attention is needed Component: Logic Non-trivial changes to the JSON logic files Component: ASM/C Changes some internals of the ASM/C libraries labels Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: ASM/C Changes some internals of the ASM/C libraries Component: Logic Non-trivial changes to the JSON logic files Status: Help Wanted Extra attention is needed Type: Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants