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

Updated ability popups for Skill Swap, Mummy/Lingering Aroma, Worry Seed, Simple Beam, fix Doodle/Role Play bugs #5493

Merged
merged 7 commits into from
Oct 10, 2024

Conversation

PhallenTree
Copy link

@PhallenTree PhallenTree commented Oct 9, 2024

Updated Ability Popups for Skill Swap, Mummy/Lingering Aroma, Worry Seed and Simple Beam to show the previous abilities of the affected battler(s). Also added Skill Swap on ally interaction (no ability popups).
Fixed Doodle checking if target has a valid ally instead of checking if user has a valid ally (potentially gave fainted allies new abilities).
Fixed Role Play failing if ally's ability couldn't be suppressed + Doodle failing for this same reason even if ally was fainted.
Fixed Doodle playing its animation despite failing.

Added Skill Swap, Worry Seed, Simple Beam and Role Play tests.
Also reordered some tests that check specific abilities that cause Role Play to fail by ability number to make it easier to check.

Discord contact info

PhallenTree

Comment on lines 2352 to 2361
.if B_ABILITY_POP_UP == TRUE
setbyte sFIXED_ABILITY_POPUP, TRUE
showabilitypopup BS_TARGET
pause 60
sethword sABILITY_OVERWRITE, 0
updateabilitypopup BS_TARGET
pause 20
destroyabilitypopup
pause 40
.endif
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you think it would make sense to make a script for those to reduce duplicate scripts?

Copy link
Author

@PhallenTree PhallenTree Oct 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added BattleScript_AbilityPopupOverwriteThenNormal which requires gBattlerAbility to be set to the battler, to be within .if B_ABILITY_POPUP == TRUE ... .endif and sABILITY_OVERWRITE to be set before being called.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also added some missing recordability calls.

Copy link
Collaborator

@AlexOn1ine AlexOn1ine Oct 10, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added BattleScript_AbilityPopupOverwriteThenNormal which requires gBattlerAbility to be set to the battler, to be within .if B_ABILITY_POPUP == TRUE ... .endif and sABILITY_OVERWRITE to be set before being called.

Damn, I asked specifically because of the preprocs 😅

I let you decide if the change is still worth it otherwise or the previous state is better.

@AlexOn1ine AlexOn1ine added category: battle-mechanic Pertains to battle mechanics bugfix Bugfixes labels Oct 10, 2024
@AlexOn1ine AlexOn1ine merged commit 8d2d622 into rh-hideout:master Oct 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Bugfixes category: battle-mechanic Pertains to battle mechanics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants