fireplace icon indicating copy to clipboard operation
fireplace copied to clipboard

Arcane Blast should not double its own base damage

Open beheh opened this issue 10 years ago • 5 comments

Arcane Blast currently deals:

  • 4 damage without Spellpower increase
  • 6 damage with +1 Spellpower
  • 8 damage with +2 Spellpower

It should deal:

  • 2 damage without Spellpower increase
  • 4 damage with +1 Spellpower (2 + 1*2)
  • 6 damage with +2 Spellpower (2 + 2*2)

beheh avatar Nov 26 '15 10:11 beheh

Test case.

beheh avatar Nov 26 '15 13:11 beheh

We should also add a test for Prophet Velen/Arcane Blasst interaction.

beheh avatar Nov 27 '15 09:11 beheh

The problem is here that we defer damage amplification to controller.

What should be happening is something like:

# calculate the actual additional spellpower
spellpower = self.spellpower
if receives_double_spelldamage_bonus:
   spellpower *= 2
# add spellpower to base damage
amount = base_damage + spellpower
# double the full damage amount when Velen is in play
amount <<= spellpower_double

beheh avatar Nov 27 '15 10:11 beheh

Fixed?

jleclanche avatar Feb 08 '16 02:02 jleclanche

No, Arcane Blast still deals 4 damage with no spell power on board.

beheh avatar Feb 08 '16 08:02 beheh

fixed

shinoi2 avatar Dec 14 '23 00:12 shinoi2