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

Holy Wrath ignores spell damage if the drawn card is 0-cost #21

Closed
NightKev opened this issue Aug 23, 2015 · 6 comments
Closed

Holy Wrath ignores spell damage if the drawn card is 0-cost #21

NightKev opened this issue Aug 23, 2015 · 6 comments
Labels
Milestone

Comments

@NightKev
Copy link
Member

When casting Holy Wrath and the drawn card has a casting cost of 0 (such as the Wisp or Target Dummy), the damage done by the spell remains at 0 even if the player has Spell Damage. Using Holy Wrath on a minion with Divine Shield in this situation will not break Divine Shield.

This bug is similar to 2 bugs that has already been fixed, namely using Shield Slam with 0 Armor and using Shadowflame on a 0-attack minion while having Spell Damage used to deal 0 damage incorrectly.

@Patashu
Copy link
Member

Patashu commented Aug 24, 2015

This bug has not been verified in a long time, so it may or may not have been fixed in the mean time.

@jleclanche
Copy link
Member

I believe we verified this in 2.6 or 2.7.

@Patashu Patashu added this to the 2.6.0.8834 milestone Aug 24, 2015
@Patashu
Copy link
Member

Patashu commented Aug 24, 2015

OK, let's say 2.6 then and we'll see if it reproduces.

@Patashu
Copy link
Member

Patashu commented Aug 27, 2015

@Patashu
Copy link
Member

Patashu commented Oct 3, 2015

Confirmed in 3.1. (Well, specifically that it doesn't get improved by Spell Damage +1. We also checked Divine Shield, it does not pop them. See for yourself.)

@Patashu Patashu modified the milestones: 3.1.0.10357, 2.6.0.8834 Oct 3, 2015
@culinko culinko modified the milestones: 4.1.0.10956, 3.1.0.10357 Jan 4, 2016
@culinko culinko modified the milestones: 5.0.0.12574, 4.1.0.10956 Apr 25, 2016
@culinko
Copy link

culinko commented Apr 25, 2016

This bug has been fixed.

@culinko culinko closed this as completed Apr 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

4 participants