pokerogue icon indicating copy to clipboard operation
pokerogue copied to clipboard

[Bug] Elite Four for Gen 8 is missing, so Leon is messing up the Elite Four -> Champion order for Gen 9

Open jaimefd opened this issue 9 months ago • 3 comments

  • Normal Gen 9 elite four has Kieran as champion. (should be Nemona/Geeta)
  • DLC Gen 9 elite four has Leon (yes, Leon, lol), as champion. (should be Kieran)
  • Nemona/Geeta don't appear.

This is likely caused because in the code in src/battle.ts, line 326, the array for champions lists 10 options, while the elite four arrays only list 9 options.

  • This happens because there's 9 Elite Forus (one for each gen, -1 because no Gen 8, +1 because Gen 9 has two)
  • But we have 10 champion sets (one for each gen, +1 because Gen 9 has two)

In other words, Leon is just messing up the whole thing.

I didn't make a Pull Request on this issue because there's many ways to fix this, and I'm not sure how you want to implement it. But I'd suggest just adding an elite four for Gen 8, with Marnie, Hop, Bede and a random choice between a few Gym leaders, like it happens in the real game. Or just add that bad steel type guy. But I don't think Leon should be polluting the pools of other generations.

EDIT: Also, Kieran has no dialogue, unlike the other champions.

jaimefd avatar May 18 '24 22:05 jaimefd