architdate / PKHeX-Auto-Legality-Mod

Modded Showdown import function that can be sideloaded for automatic Pokemon Legality
MIT License
39 stars 11 forks source link

Enhancement suggestions to the trainerdata.json file #76

Closed optizard closed 6 years ago

optizard commented 6 years ago

Enhancement suggestions to the trainerdata.json file

DISCLAIMER This will be a very long and full of grammar and syntax errors post, cause English is not my native language and I want to explain everything with examples and clarity before all.

I'm very satisfied by the addition of trainerdata.json: it helps me on using another game save file and have a different ID that isn't the default one I put in the trainerdata.txt when I gen a Pokémon. I want to suggest some more enhancements to it.

A feature that I would like to have is to gen a Pokémon with a trainer data linked to the game the mod attributes to the Pokémon and not based on the savegame that we are editing (the current behaviour of the trainerdata.json).

Just suppose to work on a Ultra Moon savegame trying to gen this Snorlax:

Snorlax @ Figy Berry Ability: Gluttony Level: 50 Shiny: Yes EVs: 244 HP / 212 Def / 52 SpD Sassy Nature IVs: 4 Spe

  • Recycle
  • Return
  • Stockpile

We are going to use this trainerdata.json (Black and White)

{ "21":{ "TID":"12345", "SID":"61234", "OT":"Black", "Gender":"M", "Country":"Canada", "SubRegion":"Alberta", "3DSRegion":"Americas (NA/SA)" }, "20":{ "TID":"23455", "SID":"23444", "OT":"White", "Gender":"F", "Country":"Singapore", "SubRegion":"Singapore", "3DSRegion":"Americas (NA/SA)" } }

and this trainerdata.txt

TID:34534 SID:11123 OT:Moon2 Gender:F Country:Canada SubRegion:Alberta 3DSRegion:Americas (NA/SA)

It will of course have Black as it's origin game:

Snorlax1

but the mod will give to this Snorlax the trainer data of the trainerdata.txt and not the Black one:

Snorlax2

Before the implementation of trainerdata.txt I succesfully edited the part of code regarding the default trainer data, letting me obtain a Snorlax with the Black trainer data in a Ultra Moon game only if it's origin was set to Black by the mod. The editing of the code was fairly simple (my knowledge of coding is only copy/paste other code sections and the "if/else") and so I presume that this kind of behaviour could be achieved in the new trainerdata.json, probably adding some new values or keys.

Just to get things more clear I'll try to suppose a particular scenario with these files and a particular "goal":

Values that I suggest can be used in the trainerdata.json (but there can be more of them, it's just an example):

Conclusions

If you read till this point you are really good and I appreciate it at a lot cause this wall of text surely is scary 😱. I hope that this enhancement suggestion will be accepted and transformed into reality. If it isn't clear I love this Mod so much that I took a lot of time to write and polish this post just to make sure everyone who reads this understand what I precisely think and need about it and maybe let you assume the same vision that I have about my "meticolous" 😅 conception of genning.

Thank you for your attention, Optimus Zard

architdate commented 6 years ago

Already addressed in 71c28b9.