Changeset 157

Timestamp:

11/19/08 13:41:36 (17 years ago)

Author:

yumileroy

Message:

[svn] Fix hunter's frozen trap, half duration when pvp.
Use vector to store linked spell information to support multiple effects.

Original author: megamage
Date: 2008-11-03 17:20:35-06:00

Location:

trunk/src/game

Files:

• Spell.cpp (modified) (2 diffs)
• SpellEffects.cpp (modified) (1 diff)
• SpellMgr.cpp (modified) (1 diff)
• SpellMgr.h (modified) (2 diffs)
• Unit.cpp (modified) (1 diff)

trunk/src/game/Spell.cpp

@@ -1056 +1056 @@
         ((Creature*)m_caster)->AI()->SpellHitTarget(unit, m_spellInfo);
 
-    if(int32 spell_triggered = spellmgr.GetSpellLinked(m_spellInfo->Id, 1))
-    {
-        if(spell_triggered > 0)
-            unit->CastSpell(unit, spell_triggered, true, 0, 0, m_caster->GetGUID());
-        else
-            unit->RemoveAurasDueToSpell(-spell_triggered);
+    if(const std::vector<int32> *spell_triggered = spellmgr.GetSpellLinked(m_spellInfo->Id + 1000000))
+    {
+        for(std::vector<int32>::const_iterator i = spell_triggered->begin(); i != spell_triggered->end(); ++i)
+        {
+            if(spell_triggered < 0)
+                unit->RemoveAurasDueToSpell(-(*i));
+            else
+                unit->CastSpell(unit, *i, true, 0, 0, m_caster->GetGUID());
+        }
     }
 }
@@ -2253 +2256 @@
     }
 
-    if(int32 spell_triggered = spellmgr.GetSpellLinked(m_spellInfo->Id, 0))
-    {
-        if(spell_triggered > 0)
-            m_caster->CastSpell(m_targets.getUnitTarget() ? m_targets.getUnitTarget() : m_caster, spell_triggered, true);
-        else
-            m_caster->RemoveAurasDueToSpell(-spell_triggered);
+    if(const std::vector<int32> *spell_triggered = spellmgr.GetSpellLinked(m_spellInfo->Id))
+    {
+        for(std::vector<int32>::const_iterator i = spell_triggered->begin(); i != spell_triggered->end(); ++i)
+        {
+            if(spell_triggered < 0)
+                m_caster->RemoveAurasDueToSpell(-(*i));
+            else
+                m_caster->CastSpell(m_targets.getUnitTarget() ? m_targets.getUnitTarget() : m_caster, *i, true);
+        }
     }
 

trunk/src/game/SpellEffects.cpp

@@ -2119 +2119 @@
     // Now Reduce spell duration using data received at spell hit
     int32 duration = Aur->GetAuraMaxDuration();
-    unitTarget->ApplyDiminishingToDuration(m_diminishGroup,duration,m_caster,m_diminishLevel);
+    unitTarget->ApplyDiminishingToDuration(m_diminishGroup,duration,caster,m_diminishLevel);
     Aur->setDiminishGroup(m_diminishGroup);
 

trunk/src/game/SpellMgr.cpp

@@ -2088 +2088 @@
         }
 
-        SpellLinkedSpell linkedSpell;
-        linkedSpell.spell = effect;
-        linkedSpell.type = type;
-        mSpellLinkedMap[trigger] = linkedSpell;
+        if(type) //we will find a better way when more types are needed
+            trigger += 1000000;
+        mSpellLinkedMap[trigger].push_back(effect);
 
         ++count;

trunk/src/game/SpellMgr.h

@@ -649 +649 @@
 typedef std::map<uint32, SpellExtraAttribute> SpellExtraAttrMap;
 
-struct SpellLinkedSpell
-{
-    int32 spell;
-    uint32 type;
-};
-typedef std::map<int32, SpellLinkedSpell> SpellLinkedMap;
+typedef std::map<int32, std::vector<int32> > SpellLinkedMap;
 
 class SpellMgr
@@ -863 +858 @@
         }
 
-        int32 GetSpellLinked(int32 spell_id, uint32 type) const
+        const std::vector<int32> *GetSpellLinked(int32 spell_id) const
         {
             SpellLinkedMap::const_iterator itr = mSpellLinkedMap.find(spell_id);
-            if(itr != mSpellLinkedMap.end())
-                return itr->second.type == type ? itr->second.spell : 0;
-            else
-                return 0;
+            return itr != mSpellLinkedMap.end() ? &(itr->second) : NULL;
         }
 

trunk/src/game/Unit.cpp

@@ -4155 +4155 @@
 
 
-    if(int32 spell_triggered = spellmgr.GetSpellLinked(-(int32)Aur->GetSpellProto()->Id, 0))
-    {
-        if(spell_triggered > 0)
-        {
-            if(Unit* caster = Aur->GetCaster())
-                CastSpell(this, spell_triggered, true, 0, 0, caster->GetGUID());
-        }
-        else
-            RemoveAurasDueToSpell(-spell_triggered);
+    if(const std::vector<int32> *spell_triggered = spellmgr.GetSpellLinked(-(int32)Aur->GetSpellProto()->Id))
+    {
+        for(std::vector<int32>::const_iterator i = spell_triggered->begin(); i != spell_triggered->end(); ++i)
+        {
+            if(spell_triggered < 0)
+                RemoveAurasDueToSpell(-(*i));
+            else if(Unit* caster = Aur->GetCaster())
+                CastSpell(this, *i, true, 0, 0, caster->GetGUID());
+        }
     }