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

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

Closed
jaimefd opened this issue May 18, 2024 · 3 comments · Fixed by #1246
Labels
enhancement New feature or request

Comments

@jaimefd
Copy link
Contributor

jaimefd commented May 18, 2024

  • 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.

@CodeTappert
Copy link
Contributor

Well according to https://bulbapedia.bulbagarden.net/wiki/Galar_League you fight

always Marnie, Hop, (Bede - he is officially not part so i dont know if we include it), Nessa, (Bea or Allister depending on the game), Raihan.

So it would be 5 fights even if we cut Bede (which we should...). I would also Cut Hop so the elite 4 all have a typ theme.

So it would be Marine (Dark), Nessa (Water), Bea(Fighting)/Allistar(Ghost), Raihan (Dragon).

Whats your opinion about that?

@jaimefd
Copy link
Contributor Author

jaimefd commented May 19, 2024

That sounds fine, I don't know about the type theme tho, I would avoid reusing gym leaders in elite four whenever possible to add more variety and potentially more egg vouchers

@CodeTappert
Copy link
Contributor

That sounds fine, I don't know about the type theme tho, I would avoid reusing gym leaders in elite four whenever possible to add more variety and potentially more egg vouchers

Have a look at Larry and or the dragon guy from kanto (dont really know the english name...). They have seperate trainer classes for their second apparence. Same could be done for the gym leaders of galar

@bennybroseph bennybroseph added the enhancement New feature or request label May 19, 2024
CodeTappert added a commit to CodeTappert/pokerogue that referenced this issue May 22, 2024
bennybroseph added a commit that referenced this issue May 25, 2024
Co-authored-by: Benjamin Odom <bennybroseph@gmail.com>
Icehawk78 added a commit to Icehawk78/pokerogue that referenced this issue May 25, 2024
* main: (43 commits)
  Added localization to the biomes. (pagefaultgames#1378)
  Fix French translation of Genesect's Drive (pagefaultgames#1379)
  Adding clause so right Pokemon in double battle cannot target itself with a spread move when left one dies with no replacement (pagefaultgames#1370)
  Fix incorrectly unselectable starter formes (pagefaultgames#1332)
  Made Lab 25% instead of 12.5% to encounter after Factory (pagefaultgames#1351)
  Fixed German localization for Starf Berry description and localized zippyZap effect (pagefaultgames#1373)
  Swapped Aron for Aggron on Iris pokemon pool (pagefaultgames#1374)
  Adds New BattleSceneEventTypes (pagefaultgames#1372)
  localized Honeygather again (pagefaultgames#1369)
  hotfix for alder bgm on double battle (pagefaultgames#1368)
  Implement Honey Gather (pagefaultgames#1360)
  Adding the option to have named trainers be able to have a double battle together (pagefaultgames#1318)
  Make iv scanner check against baby form IVs (pagefaultgames#1214)
  Issue pagefaultgames#1118 Added Galar Elite 4 (pagefaultgames#1246)
  Beat Up checks the user's party instead of always checking player's party (pagefaultgames#1268)
  Implement Quick Guard and other conditional team protection moves (pagefaultgames#1275)
  Implement Purify (pagefaultgames#1291)
  Update move.ts (pagefaultgames#1359)
  Fix Wind Rider (pagefaultgames#607)
  Implement Poison Puppeteer (pagefaultgames#320)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants