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

Fix Battlemage's Cry and Redblade Banner not working correctly #6301

Merged

Conversation

Lilylicious
Copy link
Contributor

Fixes #6208

Description of the problem being solved:

Applying the warcry power overrides in the actor misc section, after a lot of other stuff, caused some issues. Notably, it caused the increased spell damage from Battlemage's Cry to not apply correctly without manually setting warcry power, even when Redblade Banner was equipped.

This just moves said overrides to apply immediately prior to where warcry power is first used.

Steps taken to verify a working solution:

  • Checked Battlemage's Cry after change, noted that stats don't change between blank and 50000 warcry power in config

Link to a build that showcases this PR:

https://pobb.in/2f_873aTBknM

@QuickStick123 QuickStick123 added the bug: behaviour Behavioral differences label Aug 9, 2023
@Lilylicious Lilylicious force-pushed the redbladeBannerInfinity branch from 0d85bde to d687a63 Compare August 9, 2023 06:32
@Lilylicious Lilylicious force-pushed the redbladeBannerInfinity branch from d687a63 to e52898b Compare August 9, 2023 06:55
@LocalIdentity LocalIdentity changed the title Correctly apply warcry power overrides Fix Battlemage's Cry and Redblade Banner not working correctly Aug 13, 2023
@LocalIdentity LocalIdentity merged commit dc1ea3a into PathOfBuildingCommunity:dev Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: behaviour Behavioral differences
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Battlemage's Cry and Redblade Banner half-working
3 participants