[11665][fix] Mage T10 2P Bonus

[Den]

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index 6e3579d..641938e 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -8859,16 +8859,15 @@ void SpellAuraHolder::HandleSpellSpecificBoosts(bool apply)
                 case 48108:                                 // Hot Streak (triggered)
                 case 57761:                                 // Fireball! (Brain Freeze triggered)
                 {
-                    // consumed aura
-                    if (!apply && m_removeMode != AURA_REMOVE_BY_EXPIRE)
+                    if (!apply)
                     {
                         Unit* caster = GetCaster();
                         // Item - Mage T10 2P Bonus
-                        if (!caster || !caster->HasAura(70752))
-                            return;
-
-                        cast_at_remove = true;
-                        spellId1 = 70753;                   // Pushing the Limit
+                        if (caster || caster->HasAura(70752))
+                        {
+                            cast_at_remove = true;
+                            spellId1 = 70753;                   // Pushing the Limit
+                        }
                     }
                     else
                         return;

Блин, промахнулся, фикс на мусарку...

add
Хотя, нет.

[Den]

бонус срабатывал только тогда, когда:
1) баф снимался правой кнопкой мыши(44401, 48108, 57761).
2) при повторном проке бафа который уже висел на игроке т.е 44401 или 57761.
С патчем работает правильно.

diff --git a/src/game/SpellAuras.cpp b/src/game/SpellAuras.cpp
index a854f80..f500fc7 100644
--- a/src/game/SpellAuras.cpp
+++ b/src/game/SpellAuras.cpp
@@ -8873,16 +8873,14 @@ void SpellAuraHolder::HandleSpellSpecificBoosts(bool apply)
                 case 48108:                                 // Hot Streak (triggered)
                 case 57761:                                 // Fireball! (Brain Freeze triggered)
                 {
-                    // consumed aura
-                    if (!apply && m_removeMode != AURA_REMOVE_BY_EXPIRE)
+                    if (!apply)
                     {
                         Unit* caster = GetCaster();
-                        // Item - Mage T10 2P Bonus
-                        if (!caster || !caster->HasAura(70752))
-                            return;
-
-                        cast_at_remove = true;
-                        spellId1 = 70753;                   // Pushing the Limit
+                        if (caster || caster->HasAura(70752))   // Item - Mage T10 2P Bonus
+                        {
+                            cast_at_remove = true;
+                            spellId1 = 70753;                   // Pushing the Limit
+                        }
                     }
                     else
                         return;
diff --git a/src/game/UnitAuraProcHandler.cpp b/src/game/UnitAuraProcHandler.cpp
index e2e38d9..a417eb4 100644
--- a/src/game/UnitAuraProcHandler.cpp
+++ b/src/game/UnitAuraProcHandler.cpp
@@ -2958,6 +2958,11 @@ SpellAuraProcResult Unit::HandleProcTriggerSpellAuraProc(Unit *pVictim, uint32 d
                     (((Creature*)pVictim)->GetCreatureInfo()->MechanicImmuneMask & (1 << (MECHANIC_STUN - 1))) == 0)
                     return SPELL_AURA_PROC_FAILED;
             }
+            else if (auraSpellInfo->SpellIconID == 3261)
+            {
+                if (HasAura(44401) || HasAura(57761))
+                    return SPELL_AURA_PROC_FAILED;
+            }
             break;
         case SPELLFAMILY_WARRIOR:
             // Deep Wounds (replace triggered spells to directly apply DoT), dot spell have familyflags

[Amaru]

+                        if (caster || caster->HasAura(70752))   // Item - Mage T10 2P Bonus

wtf

-                    if (!apply && m_removeMode != AURA_REMOVE_BY_EXPIRE)
+                    if (!apply && m_removeMode == AURA_REMOVE_BY_DEFAULT)

Весь фикс. Отмена ауры - AURA_REMOVE_BY_DEFAULT, замена таким же проком - AURA_REMOVE_BY_STACK, истечение срока - AURA_REMOVE_BY_EXPIRE.
Снятие аур обработчиков проков происходит с AURA_REMOVE_BY_DEFAULT - все, что нам нужно.

[Den]

Ну если так, то AURA_REMOVE_BY_EXPIRE потому что:

Таланты "Путь огня", "Заградительные стрелы" и "Заморозка мозгов" также повышают ваш рейтинг скорости. при поглощении их эффектов.

[Vladimir]

С патчем работает правильно.

Что-то я в дискуссии совершенно запутался - и когда он должен или не должен...

[Den]

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

когда он должен или не должен...

Должен по истечению срока аур 44401, 48108, 57761.

[Vladimir]

Т.е во всех других случаях таких как диспел, вытеснение аналогичным из стека, снятие на какое либо событие, отмена - НЕ должен.

[Amaru]

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

Т.е во всех других случаях таких как диспел, вытеснение аналогичным из стека, снятие на какое либо событие, отмена - НЕ должен.

да, поэтому, по сути

-                    if (!apply && m_removeMode != AURA_REMOVE_BY_EXPIRE)
+                    if (!apply && m_removeMode == AURA_REMOVE_BY_DEFAULT)

- фикс

Добавлено через 39 секунд

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

Должен по истечению срока аур 44401, 48108, 57761.

не по истечению срока, а по снятию проком

[Vladimir]

Исправление добавлено в [11665] в форме предложенной Amaru. Спасибо