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

Fixes spread damage in double battles #5064

Merged
merged 4 commits into from
Jul 31, 2024

Conversation

AlexOn1ine
Copy link
Collaborator

@AlexOn1ine AlexOn1ine commented Jul 29, 2024

In double battles when B_POSITION_OPPONENT_LEFT fainted from the first hit of a spread move, the second hit dealt 100% of damage to B_POSITION_OPPONENT_RIGHT

Using gAbsentBattlerFlags fixes this because it is set when there is no other mon to replace the fainted mon with

@AlexOn1ine AlexOn1ine marked this pull request as draft July 29, 2024 22:05
@AlexOn1ine AlexOn1ine marked this pull request as ready for review July 29, 2024 22:33
@AlexOn1ine
Copy link
Collaborator Author

Force pushed because I found a very simple fix and didn't want to remove the changes one by one.

@AlexOn1ine
Copy link
Collaborator Author

actually I thought of a scenario where this might be still wrong

@AlexOn1ine AlexOn1ine marked this pull request as draft July 30, 2024 06:54
@AlexOn1ine
Copy link
Collaborator Author

actually I thought of a scenario where this might be still wrong

all good ready for review again.

@AlexOn1ine AlexOn1ine marked this pull request as ready for review July 30, 2024 14:26
@DizzyEggg DizzyEggg merged commit e6175b5 into rh-hideout:master Jul 31, 2024
1 check passed
@hedara90 hedara90 added category: battle-mechanic Pertains to battle mechanics bugfix Bugfixes labels Aug 17, 2024
@AlexOn1ine AlexOn1ine deleted the spread_damage branch November 21, 2024 22:34
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.

3 participants