[fixed][BUG] In GetBaseSpellPowerBonus

[Wowka321]

Hi everyone.

After 9896, attack power not included in calculation of Unit::SpellDamageBonusDone for not mana-based classes, cuz GetBaseSpellPowerBonus for those classes in current code is 0 and DoneAdvertisedBenefit 0, so don't pass if (DoneAdvertisedBenefit)......

If you look into Unit::SpellDamageBonusDone - you'll see it.

So, maybe we should to divide GetBaseSpellPowerBonus - for mana-based classes (like priest, warlock, shaman), AP-based (rogue, DK, war), and mixed like druid in forms and get right bonus for form... hm, but don't know about hunter with mana, and scaling stats from AP...

Maybe i mistake, but we need discuss this issue, cuz AP-based classes is sucks, like you do damage without your gear.

hm, but don't know about hunter with mana, and scaling stats from AP

[VladimirMangos]

It must be fixed now. Thank you for reporting... (fixed by your github post )