From b9dce735736222ffd52997ea0540961d39145151 Mon Sep 17 00:00:00 2001 From: SnorlaxMonster <7100450+SnorlaxMonster@users.noreply.github.com> Date: Sun, 28 Apr 2024 03:16:34 +1000 Subject: [PATCH] Rename `MON_ID` to `MON_OT_ID` (#1114) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * Rename MON_ID to MON_TRAINER_ID The constant name MON_ID implies it relates to the ID number of the Pokémon itself. However, it actually refers to the Trainer ID number of the Pokémon's Original Trainer. Renaming it to MON_TRAINER_ID makes it substantially more clear what its purpose is. The original definition of MON_ID is preserved in macros/legacy.asm * Remove MON_ID from macros/legacy.asm Co-authored-by: Sylvie <35663410+Rangi42@users.noreply.github.com> * Apply rename to missed file * Rename to `MON_OT_ID` --------- Co-authored-by: SnorlaxMonster Co-authored-by: Sylvie <35663410+Rangi42@users.noreply.github.com> Co-authored-by: vulcandth --- constants/pokemon_data_constants.asm | 2 +- engine/battle/core.asm | 10 +++++----- engine/battle/effect_commands.asm | 4 ++-- engine/events/daycare.asm | 2 +- engine/events/halloffame.asm | 2 +- engine/events/lucky_number.asm | 2 +- engine/events/magikarp.asm | 4 ++-- engine/events/poke_seer.asm | 2 +- engine/pokemon/breeding.asm | 2 +- engine/pokemon/search_owned.asm | 6 +++--- 10 files changed, 18 insertions(+), 18 deletions(-) diff --git a/constants/pokemon_data_constants.asm b/constants/pokemon_data_constants.asm index 365d4a9038d..a31ad058ae3 100644 --- a/constants/pokemon_data_constants.asm +++ b/constants/pokemon_data_constants.asm @@ -77,7 +77,7 @@ rsreset DEF MON_SPECIES rb DEF MON_ITEM rb DEF MON_MOVES rb NUM_MOVES -DEF MON_ID rw +DEF MON_OT_ID rw DEF MON_EXP rb 3 DEF MON_STAT_EXP rw NUM_EXP_STATS rsset MON_STAT_EXP diff --git a/engine/battle/core.asm b/engine/battle/core.asm index 56eb0c674fa..77ed507fb9c 100644 --- a/engine/battle/core.asm +++ b/engine/battle/core.asm @@ -3864,9 +3864,9 @@ InitBattleMon: ld a, MON_SPECIES call GetPartyParamLocation ld de, wBattleMonSpecies - ld bc, MON_ID + ld bc, MON_OT_ID call CopyBytes - ld bc, MON_DVS - MON_ID + ld bc, MON_DVS - MON_OT_ID add hl, bc ld de, wBattleMonDVs ld bc, MON_POKERUS - MON_DVS @@ -3950,9 +3950,9 @@ InitEnemyMon: ld hl, wOTPartyMon1Species call GetPartyLocation ld de, wEnemyMonSpecies - ld bc, MON_ID + ld bc, MON_OT_ID call CopyBytes - ld bc, MON_DVS - MON_ID + ld bc, MON_DVS - MON_OT_ID add hl, bc ld de, wEnemyMonDVs ld bc, MON_POKERUS - MON_DVS @@ -7074,7 +7074,7 @@ GiveExperiencePoints: call Divide ; Boost Experience for traded Pokemon pop bc - ld hl, MON_ID + ld hl, MON_OT_ID add hl, bc ld a, [wPlayerID] cp [hl] diff --git a/engine/battle/effect_commands.asm b/engine/battle/effect_commands.asm index 4051f497b99..82398f7c06f 100644 --- a/engine/battle/effect_commands.asm +++ b/engine/battle/effect_commands.asm @@ -655,9 +655,9 @@ BattleCommand_CheckObedience: and a ret nz - ; If the monster's id doesn't match the player's, + ; If the Pokémon's Trainer ID doesn't match the player's, ; some conditions need to be met. - ld a, MON_ID + ld a, MON_OT_ID call BattlePartyAttr ld a, [wPlayerID] diff --git a/engine/events/daycare.asm b/engine/events/daycare.asm index 00675502b0e..ec386185f4a 100644 --- a/engine/events/daycare.asm +++ b/engine/events/daycare.asm @@ -491,7 +491,7 @@ DayCare_GiveEgg: call AddNTimes ld b, h ld c, l - ld hl, MON_ID + 1 + ld hl, MON_OT_ID + 1 add hl, bc push hl ld hl, MON_MAXHP diff --git a/engine/events/halloffame.asm b/engine/events/halloffame.asm index 8a160a184cb..60070eeda28 100644 --- a/engine/events/halloffame.asm +++ b/engine/events/halloffame.asm @@ -172,7 +172,7 @@ GetHallOfFameParty: ld [de], a inc de - ld hl, MON_ID + ld hl, MON_OT_ID add hl, bc ld a, [hli] ld [de], a diff --git a/engine/events/lucky_number.asm b/engine/events/lucky_number.asm index 375b817b84f..46f073c2bdd 100644 --- a/engine/events/lucky_number.asm +++ b/engine/events/lucky_number.asm @@ -149,7 +149,7 @@ CheckForLuckyNumberWinners: .done pop hl push hl - ld de, MON_SPECIES - MON_ID + ld de, MON_SPECIES - MON_OT_ID add hl, de ld a, [hl] pop hl diff --git a/engine/events/magikarp.asm b/engine/events/magikarp.asm index 09d75967e5e..acd46c24f58 100644 --- a/engine/events/magikarp.asm +++ b/engine/events/magikarp.asm @@ -11,7 +11,7 @@ CheckMagikarpLength: cp MAGIKARP jr nz, .not_magikarp - ; Now let's compute its length based on its DVs and ID. + ; Now let's compute its length based on its DVs and Trainer ID. ld a, [wCurPartyMon] ld hl, wPartyMon1Species ld bc, PARTYMON_STRUCT_LENGTH @@ -22,7 +22,7 @@ CheckMagikarpLength: ld d, h ld e, l pop hl - ld bc, MON_ID + ld bc, MON_OT_ID add hl, bc ld b, h ld c, l diff --git a/engine/events/poke_seer.asm b/engine/events/poke_seer.asm index 9ad8d71dd5e..9e5e7a04e94 100644 --- a/engine/events/poke_seer.asm +++ b/engine/events/poke_seer.asm @@ -107,7 +107,7 @@ ReadCaughtData: ld a, SEERACTION_TRADED ld [wSeerAction], a - ld a, MON_ID + ld a, MON_OT_ID call GetPartyParamLocation ld a, [wPlayerID] cp [hl] diff --git a/engine/pokemon/breeding.asm b/engine/pokemon/breeding.asm index da58d8480e7..73b04570bb0 100644 --- a/engine/pokemon/breeding.asm +++ b/engine/pokemon/breeding.asm @@ -296,7 +296,7 @@ HatchEggs: ld [hli], a ld a, [de] ld [hl], a - ld hl, MON_ID + ld hl, MON_OT_ID add hl, bc ld a, [wPlayerID] ld [hli], a diff --git a/engine/pokemon/search_owned.asm b/engine/pokemon/search_owned.asm index 2edb987eb10..c4960a86e4b 100644 --- a/engine/pokemon/search_owned.asm +++ b/engine/pokemon/search_owned.asm @@ -191,7 +191,7 @@ CheckOwnMon: ; wScriptVar should contain the species we're looking for ; outputs: -; sets carry if monster matches species, ID, and OT name. +; sets carry if monster matches species, Trainer ID, and OT name. push bc push hl @@ -206,9 +206,9 @@ CheckOwnMon: cp b jr nz, .notfound - ; check ID number + ; check Trainer ID number - ld bc, MON_ID + ld bc, MON_OT_ID add hl, bc ld a, [wPlayerID] cp [hl]