[patch/dev] Lay on Hands (Возложение рук)

[Insider42]

С одним из последних патчей спел должен накладывать 25771 (Forbearance) и 61987 (Avenging Wrath Marker) на того, на кого было применено заклинание. Точно также как и остальные "бубли"

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 8a1c134..cbc04e3 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -2817,8 +2817,8 @@ void Spell::cast(bool skipCheck)
                 if (m_targets.getUnitTarget() && m_targets.getUnitTarget()->getVictim() != m_caster)
                     AddPrecastSpell(67485);                 // Hand of Rekoning (no typos in name ;) )
             }
-            // Divine Shield, Divine Protection or Hand of Protection
-            else if (m_spellInfo->SpellFamilyFlags & UI64LIT(0x0000000000400080))
+            // Divine Shield, Divine Protection, Hand of Protection or Lay on Hands
+            else if (m_spellInfo->SpellFamilyFlags & UI64LIT(0x0000000000408080))
             {
                 AddPrecastSpell(25771);                     // Forbearance
                 AddPrecastSpell(61987);                     // Avenging Wrath Marker

Осталась одна проблема - как заблокировать использование самой способности если на цели 25771 (Forbearance) висит? Я не нашёл кода (может и плохо глядел) отвечающего за это...

[Energy]

По аналогии с Hand of Protection?

upd: что-то не нашел отработки форбиранса вообще. Получается что сейчас это дело не кастится только засчет проверки в клиенте? оО

[Insider42]

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

По аналогии с Hand of Protection?

да, я так и не смог найти кусок кода предотвращающий использование спелов под Forbearance

[MaS0n]

Проверка только сервер, на клиенте не работает)

void Aura::HandleModMechanicImmunity(bool apply, bool /*Real*/)
{
    uint32 misc  = m_modifier.m_miscvalue;
    // Forbearance
    // in DBC wrong mechanic immune since 3.0.x
    if (GetId() == 25771)
        misc = MECHANIC_IMMUNE_SHIELD;

Это предотвращает использование любых спеллов с щитовой(29) механикой, а для возложения видимо придется писать отдельную проверку

[Energy]

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

Проверка только сервер, на клиенте не работает)

void Aura::HandleModMechanicImmunity(bool apply, bool /*Real*/)
{
    uint32 misc  = m_modifier.m_miscvalue;
    // Forbearance
    // in DBC wrong mechanic immune since 3.0.x
    if (GetId() == 25771)
        misc = MECHANIC_IMMUNE_SHIELD;

Это предотвращает использование любых спеллов с щитовой(29) механикой, а для возложения видимо придется писать отдельную проверку

Что на самом деле тоже неправильно, ибо форбиранс предотвращает использование _только_ бопа, шилдволла, и дивайн шилда. ну и лоха (если в себя)

[MaS0n]

Это, кстати, официальный код - возможно механику форбиранса нужно переписать более корректно. У меня в мыслях только Spell:CheckCast - там же, где чекаются aurastate, делать что-то вроде custom exclude check для подобных спеллов
Кстати, если не ошибаюсь, Истощение/Пресыщение у шаманов тоже не рабит без вмешательств

[Insider42]

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

Проверка только сервер, на клиенте не работает)

void Aura::HandleModMechanicImmunity(bool apply, bool /*Real*/)
{
    uint32 misc  = m_modifier.m_miscvalue;
    // Forbearance
    // in DBC wrong mechanic immune since 3.0.x
    if (GetId() == 25771)
        misc = MECHANIC_IMMUNE_SHIELD;

Это предотвращает использование любых спеллов с щитовой(29) механикой, а для возложения видимо придется писать отдельную проверку

Вот из-за этого приходится писать хаки на снятие аур по ID для исправления Сокрушающего броска

Рад, что наконец заставил хоть кого-то пошевелится на этом форуме xD
А то только и занят тем, что веду монологи