[patch] Warbringer 2nd part (Вестник войны 2часть)

[Insider42]

Суть проблемы: функционирует только 2 часть таланта, с со снятием препятствующих передвижению эффектов, и то неправильно. Первую часть в этой теме мы рассматривать не будем.

Автор: я

Фикс

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index f442c95..7b13f46 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -4992,7 +4992,7 @@ SpellCastResult Spell::CheckCast(bool strict)
             }
             case SPELL_EFFECT_CHARGE:
             {
-                if (m_caster->hasUnitState(UNIT_STAT_ROOT))
+                if (m_caster->hasUnitState(UNIT_STAT_ROOT) && !(m_spellInfo->Id == 3411 && m_caster->HasAura(57499)))
                     return SPELL_FAILED_ROOTED;
 
                 break;

в базу (запрос исправляет грубую ошибку в mangos.sql)

UPDATE `spell_proc_event` SET `SpellFamilyMaskA0`=0,`SpellFamilyMaskA1`=0,`SpellFamilyMaskA2`=0 where `entry`=57499;

И ещё: судя по всему реализация работы таланта в мангосе неверна. Это должно быть реализовано не через проки, а сниматься раньше чем будет произведен эффект!

В итоге что мы имеем? - При снятии корней (а ведь мы знаем, что если позволить использовать SPELL_EFFECT_CHARGE в корнях - игрок портуется к цели моментально) мы портуемся к цели (вместо того, чтобы добежать до неё) и снимаем корни.
Такого быть не должно, получается что мы игнорируем всякого рода препятствия (которые не мешают LoS) и также игнорируем mmaps (те, у кого этот мод стоит поймут)

По-моему тут 2 выхода:
1) Нужен какого-то рода preCast или захардкодить снятие в spell.cpp, там же где действует мой патч. Ну и избавиться от лишней записи в spell_proc_event + удалить реализацию 6953 оверрайда
2) Нужно переработать отвечающий за работу корней/приморозок и т.д. код, позволить двигаться на SPELL_EFFECT_CHARGE, а не портоваться сразу к цели