[7782] "Aspect of the Viper" dummy procs

[Auntie Mangos]

Formula is taken from the comments on wowhead: http://www.wowhead.com/?spell=34074 The wiki is not very usable here :-/

Bug reports: http://getmangos.eu/community/viewtopic.php?id=3873&highlight=Viper ,but this is in fact reffering to the old mechanic of this spell.

From c8d5c1006597cbf6b516a89cf6a2113295d1eb70 Mon Sep 17 00:00:00 2001
From: pasdVn <[email protected]>
Date: Fri, 6 Feb 2009 17:47:45 +0100
Subject: [PATCH] added dummy procs for 34074

---
src/game/Unit.cpp |   13 ++++++++++++-
1 files changed, 12 insertions(+), 1 deletions(-)

diff --git a/src/game/Unit.cpp b/src/game/Unit.cpp
index 6788079..c87d189 100644
--- a/src/game/Unit.cpp
+++ b/src/game/Unit.cpp
@@ -5268,7 +5268,17 @@ bool Unit::HandleDummyAuraProc(Unit *pVictim, uint32 damage, Aura* triggeredByAu
        }
        case SPELLFAMILY_HUNTER:
        {
-            // Thrill of the Hunt
+            // Aspect of the Viper
+            if ( dummySpell->SpellFamilyFlags & 0x4000000000000LL )
+            {
+                uint32 maxmana = GetMaxPower(POWER_MANA);
+                basepoints0 = maxmana* GetAttackTime(RANGED_ATTACK)/1000.0f/100.0f;
+
+                target = this;
+                triggered_spell_id = 34075;
+                break;
+            }
+            // Thrill of the Hunt
            if ( dummySpell->SpellIconID == 2236 )
            {
                if(!procSpell)
@@ -10429,6 +10439,7 @@ void Unit::ProcDamageAndSpellFor( bool isVictim, Unit * pTarget, uint32 procFlag
                break;
            }
            case SPELL_AURA_MANA_SHIELD:
+            case SPELL_AURA_OBS_MOD_MANA:
            case SPELL_AURA_DUMMY:
            {
                sLog.outDebug("ProcDamageAndSpell: casting spell id %u (triggered by %s dummy aura of spell %u)", spellInfo->Id,(isVictim?"a victim's":"an attacker's"), triggeredByAura->GetId());
-- 
1.6.0.2.1172.ga5ed0

And the procflags because it is not a "standard" proc aura and also non-direct-damage spells can trigger the aura, as the wiki says:

INSERT INTO `spell_proc_event` (`entry`, `SchoolMask`, `SpellFamilyName`, `SpellFamilyMask0`, `SpellFamilyMask1`, `SpellFamilyMask2`, `procFlags`, `procEx`, `ppmRate`, `CustomChance`, `Cooldown`) VALUES
(34074, 0, 9, 522819, 8917121, 513, 0, 0, 0, 0, 0);

[przemratajczak]

works good TY

[Win 32`]

This patch in GIT

[Graftal]

Thanks, great patch, hunters really needed this, hope it gets offi soon

[ApoC]

Patch applied in [7782]

Thank You.

[germandgf]

Mana Regen from Aspect of the Viper shouldn't cause threat

[VladimirMangos]

It not generate threat, i not see by code wher it can...