[patch] Improved Icy Touch

[Insider42]

Суть проблемы: Талант "Улучшенное ледяное прикосновение" не увеличивает урон наносимый способностью "Ледяное прикосновение"

Таблетка

diff --git a/src/game/Unit.cpp b/src/game/Unit.cpp
index 4d52b88..daa9fd0 100644
--- a/src/game/Unit.cpp
+++ b/src/game/Unit.cpp
@@ -6407,6 +6407,21 @@ uint32 Unit::SpellDamageBonusDone(Unit *pVictim, SpellEntry const *spellProto, u
                         break;
                     }
                 }
+
+                // Icy Touch
+                if (spellProto->SpellFamilyFlags & UI64LIT(0x0000000000000002))
+                {
+                    // Improved Icy Touch
+                    Unit::AuraList const& dummyAuras = GetAurasByType(SPELL_AURA_DUMMY);
+                    for(Unit::AuraList::const_iterator i = dummyAuras.begin(); i != dummyAuras.end(); ++i)
+                    {
+                        if ((*i)->GetSpellProto()->SpellIconID == 2721)
+                        {
+                            DoneTotalMod *= ((*i)->GetModifier()->m_amount+100.0f) / 100.0f;
+                            break;
+                        }
+                    }
+                }
             }
             // Death Coil (bonus from Item - Death Knight T8 DPS Relic)
             else if (spellProto->SpellFamilyFlags & UI64LIT(0x00002000))

Патч бегло протестирован, то, что от него требуется он выполняет.

[xex]

А так не правильнее?

            // Icy Touch and Howling Blast
            if (spellProto->SpellFamilyFlags & UI64LIT(0x0000000200000002))
            {
                // search disease
                bool found = false;
                Unit::AuraMap const& auras = pVictim->GetAuras();
                for(Unit::AuraMap::const_iterator itr = auras.begin(); itr!=auras.end(); ++itr)
                {
                    if(itr->second->GetSpellProto()->Dispel == DISPEL_DISEASE)
                    {
                        found = true;
                        break;
                    }
                }
                if(!found)
                    break;

                // search for Glacier Rot dummy aura
                Unit::AuraList const& dummyAuras = GetAurasByType(SPELL_AURA_DUMMY);
                for(Unit::AuraList::const_iterator i = dummyAuras.begin(); i != dummyAuras.end(); ++i)
                {
                    if ((*i)->GetSpellProto()->EffectMiscValue[(*i)->GetEffIndex()] == 7244)
                    {
                        DoneTotalMod *= ((*i)->GetModifier()->m_amount+100.0f) / 100.0f;
                        break;
                    }
                }
            }

[Insider42]

Сообщение от xex

А так не правильнее?

            // Icy Touch and Howling Blast
            if (spellProto->SpellFamilyFlags & UI64LIT(0x0000000200000002))
            {
                // search disease
                bool found = false;
                Unit::AuraMap const& auras = pVictim->GetAuras();
                for(Unit::AuraMap::const_iterator itr = auras.begin(); itr!=auras.end(); ++itr)
                {
                    if(itr->second->GetSpellProto()->Dispel == DISPEL_DISEASE)
                    {
                        found = true;
                        break;
                    }
                }
                if(!found)
                    break;

                // search for Glacier Rot dummy aura
                Unit::AuraList const& dummyAuras = GetAurasByType(SPELL_AURA_DUMMY);
                for(Unit::AuraList::const_iterator i = dummyAuras.begin(); i != dummyAuras.end(); ++i)
                {
                    if ((*i)->GetSpellProto()->EffectMiscValue[(*i)->GetEffIndex()] == 7244)
                    {
                        DoneTotalMod *= ((*i)->GetModifier()->m_amount+100.0f) / 100.0f;
                        break;
                    }
                }
            }

невижу разницы со стандартным кодом мангоса в том, что вы запостили

[Warlord123]

У xex , похоже, была идея совместить просмотр dummy аур. Что-то в этом роде...

------------------------------ src/game/Unit.cpp ------------------------------
index 2614123..c0137d2 100644
@@ -6369,23 +6369,22 @@ uint32 Unit::SpellDamageBonusDone(Unit *pVictim, SpellEntry const *spellProto, u
                     if(itr->second->GetSpellProto()->Dispel == DISPEL_DISEASE)
                     {
                         found = true;
                         break;
                     }
                 }
-                if(!found)
-                    break;
 
-                // search for Glacier Rot dummy aura
+                // search for Glacier Rot dummy aura & Improved Icy Touch
                 Unit::AuraList const& dummyAuras = GetAurasByType(SPELL_AURA_DUMMY);
                 for(Unit::AuraList::const_iterator i = dummyAuras.begin(); i != dummyAuras.end(); ++i)
                 {
-                    if ((*i)->GetSpellProto()->EffectMiscValue[(*i)->GetEffIndex()] == 7244)
+                    if (((*i)->GetSpellProto()->EffectMiscValue[(*i)->GetEffIndex()] == 7244 && found ) //Glacier Rot
+                        ||((*i)->GetSpellProto()->SpellIconID == 2721 && spellProto->SpellFamilyFlags & UI64LIT(0x0000000000000002)) //Improved Icy Touch
+                        )
                     {
                         DoneTotalMod *= ((*i)->GetModifier()->m_amount+100.0f) / 100.0f;
-                        break;
                     }
                 }
             }
             // Death Coil (bonus from Item - Death Knight T8 DPS Relic)
             else if (spellProto->SpellFamilyFlags & UI64LIT(0x00002000))
             {

[Insider42]

Сообщение от Warlord123

У xex , похоже, была идея совместить просмотр dummy аур. Что-то в этом роде...

------------------------------ src/game/Unit.cpp ------------------------------
index 2614123..c0137d2 100644
@@ -6369,23 +6369,22 @@ uint32 Unit::SpellDamageBonusDone(Unit *pVictim, SpellEntry const *spellProto, u
                     if(itr->second->GetSpellProto()->Dispel == DISPEL_DISEASE)
                     {
                         found = true;
                         break;
                     }
                 }
-                if(!found)
-                    break;
 
-                // search for Glacier Rot dummy aura
+                // search for Glacier Rot dummy aura & Improved Icy Touch
                 Unit::AuraList const& dummyAuras = GetAurasByType(SPELL_AURA_DUMMY);
                 for(Unit::AuraList::const_iterator i = dummyAuras.begin(); i != dummyAuras.end(); ++i)
                 {
-                    if ((*i)->GetSpellProto()->EffectMiscValue[(*i)->GetEffIndex()] == 7244)
+                    if (((*i)->GetSpellProto()->EffectMiscValue[(*i)->GetEffIndex()] == 7244 && found ) //Glacier Rot
+                        ||((*i)->GetSpellProto()->SpellIconID == 2721 && spellProto->SpellFamilyFlags & UI64LIT(0x0000000000000002)) //Improved Icy Touch
+                        )
                     {
                         DoneTotalMod *= ((*i)->GetModifier()->m_amount+100.0f) / 100.0f;
-                        break;
                     }
                 }
             }
             // Death Coil (bonus from Item - Death Knight T8 DPS Relic)
             else if (spellProto->SpellFamilyFlags & UI64LIT(0x00002000))
             {

Тогда будет накладываться только один бонус Ведь мы всего 1 раз проходим это место для нанесения урона, а там у вас стоит "или", если хотите чтобы такой работал то нужно цикл на 2 раза мутить или break сносить...

[Warlord123]

Сообщение от Insider42

Тогда будет накладываться только один бонус Ведь мы всего 1 раз проходим это место для нанесения урона, а там у вас стоит "или", если хотите чтобы такой работал то нужно цикл на 2 раза мутить или break сносить...

В патчике как раз стоит -break; в 2 местах. Цикл чуть длиннее, но наверняка быстрее чем 2-х кратный поиск dummy аур и 2 цикла.