Skip to content

Keep DEFAULT shiny odds after reloading options - #335

Open
Roxas2712 wants to merge 3 commits into
UNDERdecoded:mainfrom
Roxas2712:codex/upstream-option-default-persistence
Open

Roxas2712 wants to merge 3 commits into
UNDERdecoded:mainfrom
Roxas2712:codex/upstream-option-default-persistence

Conversation

@Roxas2712

Copy link
Copy Markdown

Selecting SHINY ODDS → DEFAULT after ALWAYS removed the key from the live options table. The generation-aware options writer then retained the previous on-disk rate, so a fresh load could enable the extra shiny roll again. Store an explicit zero for DEFAULT; Pokemon.shinyOddsOf already treats zero as the cartridge rate.

The regression drives the upstream flat menu through its normal input and the real saveOptions/optionsFor persistence path. It covers Red, Gold, Silver, Crystal and Emerald, shared values and per-generation overrides, and preservation of shared/other-generation settings.

Validation on 4a988e2: 130/130 persistence assertions, 20/20 UI-layout assertions and 23/23 battle-fit assertions pass. No ROM, mod data, GPU or release qualification is included.

Contributions by Roxas2712 — https://github.com/Roxas2712. LICENSE-ROXAS2712.md retains existing rights and grants Cedric / UNDERdecoded the specific incorporation permission with visible credit.

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

Successfully merging this pull request may close these issues.

1 participant