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.

Sign up for GitHub

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

Jump to bottom

Triple Kick calls attackstring and ppreduce twice #3054

Closed
LOuroboros opened this issue Jun 12, 2023 · 0 comments · Fixed by #3074
Closed

Triple Kick calls attackstring and ppreduce twice #3054

LOuroboros opened this issue Jun 12, 2023 · 0 comments · Fixed by #3074
Labels
bug Bug category: battle-mechanic Pertains to battle mechanics category: move effect Pertains to move effects status: confirmed This bug has been confirmed to exist in the codebase

Comments

@LOuroboros
Copy link
Collaborator

Description

The battle script for Triple Kick's effect calls ppreduce and attackstring.
Then it jumps to BattleScript_HitFromCritCalc which also calls both of them.

Version

upcoming (Edge)

Upcoming Version

8efbe82

Discord contact info

Lunos#4026
@LOuroboros LOuroboros added bug Bug category: battle-mechanic Pertains to battle mechanics status: unconfirmed This bug has not been reproduced yet status: confirmed This bug has been confirmed to exist in the codebase category: move effect Pertains to move effects and removed status: unconfirmed This bug has not been reproduced yet labels Jun 12, 2023
@AlexOn1ine AlexOn1ine mentioned this issue Jun 25, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Bug category: battle-mechanic Pertains to battle mechanics category: move effect Pertains to move effects status: confirmed This bug has been confirmed to exist in the codebase
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Clear Amulet visual bug + Tests AlexOn1ine/pokeemerald-expansion-1
1 participant
@LOuroboros