Probably breaks a thing or two... time to get testing after this new
commit.
Wrapping the stored 'savefile' into an object which handles its use
allows a clearer separation of function and easier maintenance or future
updates.
May eventually move all get/sets for each form to allow property
fetching from any window, all logic handled within the class.
This makes it so that the program can turn off these methods for every
instance unless the implemented method explicitly says otherwise.
Nullable boolzzz
Properly init offsets for inventory, and fix for genderless bit writing.
ORAS PokeDex editor does not save bools for the current entry when
closing the form; fix will be provided in a later commit.
There's a few ways to finish implementing, as the program needs a way to
access Trainer Information on the fly. Quick and dirty way is to add new
global variables and set them when a save file is loaded, and a cleaner
way would be to move the 'savefile' variable into the sav class and do
all accessing from there.
Not yet decided on which. For now, the added items and code still do
nothing.
Removed unneeded picturebox array (duplicate of slotpictureboxes)
Removed temporary variable 'picturebox', just straight up compare to
null.
Deleting now instantly sets the picture to null, instead of generating
the image. (might not have been the best idea if I somehow decided to
change slot filling)
Empty slots now instantly return null (empty) for their image instead of
doing layering.
It might be wise to just ball up all the results (as decrypted) to speed
up form loading (by just having to read one file instead of >1000 at a
time).
Maybe just scoop up all the files into the database and filter dupes,
but no way to remove stuff....
Indicator will now tell the source of the file last viewed
Focus fix pairs with the main form adding the focus argument to the
method; the window will no longer lose focus (ie, pass focus to the Main
form)
I don't have TradeFriendshipAffection called by anything; or any of
these methods for that matter. Need to find a solution so that the user
can turn ON/OFF these types of modifications (and ensure that PKHeX
doesn't overwrite any user data).
Required adding a pre-filtered method to the report form in order to
process data.
Probably gonna end up with stupid identifiers stretching everything, but
at least it shows where everything is. Best to export to a csv table
anyway.
When the SelectedIndex of the Version ComboBox is changed, the
Generation index will be set to zero (and vice versa) to prevent
conflicting search criteria.
Thanks RustInPeace for initiating the conversation.
Getting EggLoc needs to call getLocation with eggMet set (for HGSS
mons).
Fixed 0 index for Box/Slot in the identifier; BOX 1 => B1
Should only add the pkm to the list if its checksum is valid.
Big thanks to Siobonbon for finally clearing up the muck that had been
sitting around for so long 👍
The mysterious dex flags are much clearer now, there is no 'via trade'
flag (it's the forms owned flag)
