[Fix][8206] Rake DoT Damage change in 8173

[nos4r2zod]

What bug does the patch fix? What features does the patch add?

The change to Rake's DoT ap coefficient in [8173] is incorrect.

• * The coefficient there applies per tick, so 0.06 (times the number of ticks) would give the correct 0.18 ap bonus to the total DoT done.
  * This coefficient was added in [7499], long after the switch to cllient 3.0.9 (rev. 7298), so the "change" in 3.0.2 was already considered.
  * I tested this with client just to be 100% sure, and now (without this patch) the DoT portion adds 0.54 * AP damage.
  

For which repository revision was the patch created?

8175

Who has been writing this patch? Please include either forum user names or email addresses.

Myself

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 44558e2..9acd2bd 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -4308,8 +4308,8 @@ void Aura::HandlePeriodicDamage(bool apply, bool Real)
                // Rake
                if (m_spellProto->SpellFamilyFlags & UI64LIT(0x0000000000001000) && m_spellProto->Effect[2]==SPELL_EFFECT_ADD_COMBO_POINTS)
                {
-                    // $AP*0.18 bonus per tick
-                    m_modifier.m_amount += int32(caster->GetTotalAttackPowerValue(BASE_ATTACK) * 18 / 100);
+                    // $AP*0.06 bonus per tick
+                    m_modifier.m_amount += int32(caster->GetTotalAttackPowerValue(BASE_ATTACK) * 6 / 100);
                    return;
                }
                // Lacerate

[emsy]

this is indeed correct http://www.wowwiki.com/Attack_power_coefficient#Druid_coefficients

edit:

Now just gotta update [8175] which forgot to add Maul, which should be considered to have MECHANIC_BLEED just like Shred

[VladimirMangos]

In [8206]. Thank you You are correct and i must read spell descriptions more carefully...