Core/Spells: Fixed misdirection ability of hunters to end 4 seconds after first damage proc - fixes #114

src/server/game/Entities/Unit/Unit.cpp

@@ -6598,6 +6598,13 @@ bool Unit::HandleDummyAuraProc(Unit* victim, uint32 damage, AuraEffect* triggere
                     return true;
                 }
             }
+            // Misdirection
+            if (dummySpell->Id == 34477)
+            {
+                triggered_spell_id = 35079; // 4 sec buff on self
+                target = this;
+                break;
+            }
             break;
         }
         case SPELLFAMILY_PALADIN:

src/server/game/Spells/Auras/SpellAuraEffects.cpp

@@ -4965,10 +4965,17 @@ void AuraEffect::HandleAuraDummy(AuraApplication const* aurApp, uint8 mode, bool
                     }
                     break;
                 case SPELLFAMILY_HUNTER:
-                    // Misdirection
+                    switch (GetId())
-                    if (GetId() == 34477)
+                    {
+                    case 34477: // Misdirection
+                        if (aurApp->GetRemoveMode() != AURA_REMOVE_BY_EXPIRE)
                             target->SetReducedThreatPercent(0, 0);
                         break;
+                    case 35079: // Misdirection proc
+                        target->SetReducedThreatPercent(0, 0);
+                        break;
+                    }
+                    break;
                 case SPELLFAMILY_DEATHKNIGHT:
                     // Summon Gargoyle (Dismiss Gargoyle at remove)
                     if (GetId() == 61777)

src/server/game/Spells/SpellMgr.cpp

@@ -3063,6 +3063,7 @@ void SpellMgr::LoadDbcDataCorrections()
             case 57761: // Fireball!
             case 39805: // Lightning Overload
             case 64823: // Item - Druid T8 Balance 4P Bonus
+            case 34477: // Misdirection
             case 44401: // Missile Barrage
                 spellInfo->procCharges = 1;
                 break;