BreakBB / ExtendedCharacterStats

Extended Character Stats: A WoW Classic addon
MIT License
7 stars 18 forks source link

Total Spell hit on Classic Era calculated incorrectly #326

Closed retrobloodfang closed 1 month ago

retrobloodfang commented 1 month ago

Bug description

On my mage ECS displays the following: Bonus: 7.14% Miss: 1% Miss (Lvl + 3): 9.86%

However, I know for a fact I’m at a way higher hit. i have the following hit contributions: Talents into ‘Elemental Precisioin’: 3/3 (6% hit) Total hit from gear: 8% Total hit aquired should be: 14%

It used to work fine but it broke since the WoW SoD patch. So I’m sure it’s the Addon.

Screenshots

image

Version

Classic Era ECS v3.3.4

BreakBB commented 1 month ago

Thanks for reporting this. It reminded me, that Blizzard changed the talent order and now I have to do all checks over again 🙈

retrobloodfang commented 1 month ago

I also commented on a post saying spell hit in SoD was incorrect and I noticed the same in Era. You may want to look at the SoD topic too. https://github.com/BreakBB/ExtendedCharacterStats/issues/325

BreakBB commented 1 month ago

hmmm... It is The talent is looking fine for me on SoD with my mage 🤔

/dump GetTalentInfo(3, 17) still is Elemental Precision

retrobloodfang commented 1 month ago

Cool, maybe not. Don’t play SoD, just Era and in Era there’s definitely an issue. Thanks for looking into it!

BreakBB commented 1 month ago

Found the issue. It affects both Era and SoD. Fix will be released shortly.

retrobloodfang commented 1 month ago

Thanks in advance for the hot fix!