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

bugfix_line_spell: prevent narrow line spells from being stopped early #40242

Merged
merged 7 commits into from
May 12, 2020

Conversation

OrenAudeles
Copy link
Contributor

@OrenAudeles OrenAudeles commented May 6, 2020

Summary

SUMMARY: Bugfixes "Fix Narrow Line Spell Effects"

Purpose of change

Line attack spells with narrow AOE (0 or 1) were being prevented from
actually working as intended. The exception added to the condition
prevents exiting early if the spell is a line attack.

Fixes #39955

Describe the solution

Add an exception for line spells to spell_effect_area which generates the final list of points to have damage applied.

Testing

In the referenced issue spells such as mana beam and lightning bolt would at most produce a single sound, and damage a single monster. With the exception applied the same spells were able to damage all zombies covered by the targeting UI.

Line attack spells with narrow AOE (0 or 1) were being prevented from
actually working as intended. The exception added to the condition
prevents exiting early if the spell is a line attack.
@OrenAudeles OrenAudeles requested a review from KorGgenT as a code owner May 6, 2020 06:02
@KorGgenT KorGgenT added <Bugfix> This is a fix for a bug (or closes open issue) Mods: Magiclysm Anything to do with the Magiclysm mod labels May 6, 2020
Splits spell effect area calculation out of `spell_effect_area` to
allow independent testing without calling the drawing function
`explosion_handler::draw_custom_explosion`
spell_type factory is not empty prior to running this test, clearing it
messes up other tests.
following convention across headers in repo

Co-authored-by: ZhilkinSerg <ZhilkinSerg@users.noreply.github.com>
@kevingranade kevingranade merged commit 4dfa4f0 into CleverRaven:master May 12, 2020
@OrenAudeles OrenAudeles deleted the bugfix_line_spell branch May 13, 2020 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Mods: Magiclysm Anything to do with the Magiclysm mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Line attack spells only damage the enemy at target
4 participants