Skip to content

Comments

fix(ability): Parental Bond no longer reduces damage improperly#1404

Merged
DayKev merged 1 commit intobetafrom
fix/parental-bond
Dec 20, 2025
Merged

fix(ability): Parental Bond no longer reduces damage improperly#1404
DayKev merged 1 commit intobetafrom
fix/parental-bond

Conversation

@DayKev
Copy link
Contributor

@DayKev DayKev commented Dec 20, 2025

Port of pagefaultgames/pokerogue#6743

What are the changes the user will see?

Parental Bond will no longer erroneously consider hitting the second Pokémon on the field as the second strike of a move when using a spread move (meaning the damage will no longer be improperly reduced).

Why am I making these changes?

Fixing a bug.

What are the changes from a developer perspective?

The damage reduction of the second strike from Parental Bond is no longer handled by AddSecondStrikeAbAttr. Instead, Parental Bond uses the existing DamageBoostAbAttr with a multiplier of 0.25 with a condition function that checks if it's the second hit of the move.

How to test the changes?

pnpm test:silent parental-bond

Checklist

  • Otherwise: I'm using beta as my base branch
  • There is no overlap with another PR?
  • The PR is self-contained and cannot be split into smaller PRs?
  • Have I provided a clear explanation of the changes?
  • Are all unit tests still passing? (pnpm test:silent)
    • Have I created new automated tests (pnpm test:create) or updated existing tests related to the PR's changes?

Parental Bond will no longer erroneously consider hitting
the second Pokemon on the field as the second strike of a move
when using a spread move
(meaning the damage will no longer be improperly reduced)

Co-authored-by: Bertie690 <taylormw163@gmail.com>
@DayKev DayKev self-assigned this Dec 20, 2025
@DayKev DayKev added Ability For ability implementations/interactions P2 MINOR. Incorrect behavior but is not game breaking. labels Dec 20, 2025
@DayKev DayKev merged commit 0c9af74 into beta Dec 20, 2025
10 checks passed
@DayKev DayKev deleted the fix/parental-bond branch December 20, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ability For ability implementations/interactions P2 MINOR. Incorrect behavior but is not game breaking.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants