vmangos / core

Progressive Vanilla Core aimed at all versions from 1.2 to 1.12
GNU General Public License v2.0
692 stars 492 forks source link

Fix idol of the moon bonus damage #2802

Closed Gamemechanicwow closed 1 month ago

Gamemechanicwow commented 1 month ago

🍰 Pullrequest

Changes how Idol of the moon bonus damage is calculated. Note: I can't find any evidence that this idol added 17% as stated by the item tooltip - more info linked further below.

Before this PR:

After this PR:

For rank 10 moonfire this equals to:

Proof

Patch 1.12.1 Known Issues Spell damage awarded from Idol of the Moon is inconsistent with it's tooltip.

Posted on Druid classic discord: https://discord.com/channels/253205420225724416/253206622522834945/743124572475490316 image Image represents Rank 10 moonfire direct damage. From my interpretation that looks like 4-5 damage difference. No raw data available.

From wowhead comments: image (note: edited for easier readability) This equals an increase by 4.325 damage - since the tests mention crits, we can deduce that this is the direct damage part.

image Tick damage.

Counterproof

There are also comments on druid classic discord that mention its 20 + 20 damage. 20 direct damage and 4 x 5 dot damage. None of those comments are backed up by evidence, and seems to be a constant copy/pasta response based on initial testing. Since the direct damage portion of the spell varies by 36 damage - tests with low samplesize could be missleading.

image Mentions of a GM talking about how this spell could negate other bonusses. I can't deduce any math formula that would negate the "Improve Moonfire" talents and give the expected results.

It's not possible to test this ERA PTR, since Blizzard changed the item to its TBC version which adds +33 damage.

Issues

How2Test

Todo / Checklist