Реагенты триггерных спеллов

[virusav]

Есть спелл 40958, который триггерит 40962.
Последний спелл требует реагенты, но при касте они не забираются.
Каст 40958 осуществляется госсипом у го.

Написал патч на спелл 40962:

diff --git a/src/game/SpellEffects.cpp b/src/game/SpellEffects.cpp
index c566fd1..caca0e6 100644
--- a/src/game/SpellEffects.cpp
+++ b/src/game/SpellEffects.cpp
@@ -1288,6 +1288,22 @@ void Spell::EffectDummy(SpellEffectIndex eff_idx)
                     DoCreateItem(eff_idx, newitemid);
                     return;
                 }
+                case 40962:                                 // Blade's Edge Terrace Demon Boss Summon Branch
+                {
+                    if (!unitTarget || unitTarget->GetTypeId() != TYPEID_PLAYER)
+                        return;
+
+                    uint32 spell_id = 0;
+                    switch (urand(1,4))
+                    {
+                        case 1: spell_id = 40957; break;    // Blade's Edge Terrace Demon Boss Summon 1
+                        case 2: spell_id = 40959; break;    // Blade's Edge Terrace Demon Boss Summon 2
+                        case 3: spell_id = 40960; break;    // Blade's Edge Terrace Demon Boss Summon 3
+                        case 4: spell_id = 40961; break;    // Blade's Edge Terrace Demon Boss Summon 4
+                    }
+                    unitTarget->CastSpell(unitTarget, spell_id, true, NULL, NULL, unitTarget->GetObjectGuid(), m_spellInfo);
+                    return;
+                }
                 case 42287:                                 // Salvage Wreckage
                 {
                     if (m_caster->GetTypeId() != TYPEID_PLAYER)

Сначала использовал m_caster, т.к. по словам НФ это предпочтительнее, если таргет=кастер.
Потом сделал, как в коммите https://github.com/mangos/mangos/com...0122e399ea5d4a

В итоге реагенты так и остались в инвентаре.

Каким образом триггерный спелл 40962 может забрать реагенты у кастера?

[rsa]

Надо в конце-концов что-то делать с таргетом 25. Если в 40958 оригиналкастером будет игрок, то и реагенты заберутся правильно...
Кста, что-то я в ДБЦ никакого требования реагентов на 40962 не вижу. Следовательно все равно надо хакать.

[virusav]

Если в `gossip_scripts` указать каст спелла 40962, то реагенты забираются, а в случае с 40958 - нет.
Тип каста равен 1, т.к. все остальные варианты дают ошибку в логе.

Какие еще варианты?

[rsa]

ни в 40962 ни в его триггерных ингридиенты не забираются. так что сначала выясните кто же у вас их забирает, тогда будет ясно что делать

[virusav]

Спеллворк не показал требования реагентов.

Решил проверить спеллы 43723 и 43753, при касте которых в конечном итоге реагенты забираются, но в спеллворке не увидел данных по требуемым итемам.

У го есть спеллфокус, у госсипа прописано требование итема в количестве 35 штук.

Если не спелл 40962 забирает реагенты, то кто + в чем тогда разница между кастом спелла напрямую и через основной спелл в таком случае?

[MaS0n]

Не пробовал

unitTarget->CastSpell(unitTarget, spell_id, false, NULL, NULL, unitTarget->GetObjectGuid(), m_spellInfo);


Для триггерных спеллов вроде не берутся реагенты

[virusav]

Недавно был коммит, где триггерный спелл забирал реагенты, ссылка в первом посте, но не прокатило.
Если не прокатит с true, то тогда буду пробовать с false, но не уверен, что так должно быть в случае каста триггероного спелла.

[rsa]

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

У го есть спеллфокус, у госсипа прописано требование итема в количестве 35 штук.

Если не спелл 40962 забирает реагенты, то кто + в чем тогда разница между кастом спелла напрямую и через основной спелл в таком случае?

вывод - реагенты забирает госсип при успешном касте спелла. осталось только выяснить, почему каст 40962 считается успешным, а предыдущего по цепочке - нет. все остальное трогать не надо, уже работает.

[virusav]

Если не ошибаюсь, то госсип не может забирать реагенты.

Спелл 40962 требует итемы, это можно увидеть в обновленном спеллворке:
Reagents: 32569x35

[virusav]

У спелла 40958 эффект SPELL_EFFECT_FORCE_CAST.
В Spell::EffectForceCast каст происходит таким образом:

unitTarget->CastSpell(unitTarget, spellInfo, true, NULL, NULL, m_originalCasterGUID);

Чтобы реагенты триггерных спеллов забирались, нужно (взято и коммита НФ):

unitTarget->CastSpell(unitTarget, spell_id, true, NULL, NULL, unitTarget->GetObjectGuid(), m_spellInfo)

1. Чем может быть чревата замена каста в Spell::EffectForceCast на:

unitTarget->CastSpell(unitTarget, spellInfo->Id, true, NULL, NULL, m_originalCasterGUID, spellInfo);

т.е. чтобы реагенты забирались сразу при касте триггерных спеллов?
Возможно, косяк будет, если кастер итем, хз.
2. Почему в Spell::EffectForceCast каст всегда идет тартетом на себя, а не кастером на таргет?

[rsa]

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

[virusav]

Раз уж речь о триггерных спеллах: 47722 триггерит 47724, у которого SPELL_EFFECT_SCRIPT_EFFECT=каст 50239 на себя (создание итема).
Определенное время назад в конечном итоге у игрока в инвентаре оказывался итем после каста 50239.
После какого-то коммита целью 50239 является нпц, на которого идет каст 47722.

[virusav]

Проблему решил таким образом:

void Spell::EffectForceCast(SpellEffectIndex eff_idx)
...
-    unitTarget->CastSpell(unitTarget, spellInfo, true, NULL, NULL, m_originalCasterGUID);
+    unitTarget->CastSpell(unitTarget, spellInfo, true, NULL, NULL, m_originalCasterGUID, m_spellInfo);

Для триггерных спеллов как раз и применяется такой прием, т.к. именно при таком касте реагенты забираются.

Только все равно интересно, почему кастует именно таргет на таргет, а не кастер на таргет.

[Vladimir]

Поправлено в [11693]. Спасибо.

[virusav]

Все-таки остался открытым вопрос о том, что триггерный спелл кастует таргет, а не кастер.

unitTarget->CastSpell(unitTarget, spellInfo, true, NULL, NULL, m_originalCasterGUID, m_spellInfo);

Из-за этого не работают спеллы с:

Attributes: 0x20000000 (SPELL_ATTR_UNAFFECTED_BY_INVULNERABILITY)
TargetAuraState = 13 (AURA_STATE_HEALTHLESS_35_PERCENT)

47679 Blood Draw
47702 Unholy Union
47722 Frost Draw
50251 Blood Draw

Для примера возьмем спелл 47722:

Effect 0: Id 6 (SPELL_EFFECT_APPLY_AURA)
BasePoints = 1
Targets (25, 0) (TARGET_DUELVSPLAYER, NO_TARGET)
Aura Id 23 (SPELL_AURA_PERIODIC_TRIGGER_SPELL), value = 1, misc = 0 (0), miscB = 0, periodic = 8000
   Trigger spell (47724) Frost Draw. Chance = 101

Сейчас его кастует игрок с помощью итема на нпц, который в итоге кастует триггерный спелл 47724:

Effect 0: Id 77 (SPELL_EFFECT_SCRIPT_EFFECT)
BasePoints = 50239
Targets (1, 0) (TARGET_SELF, NO_TARGET)

Effect 1: Id 1 (SPELL_EFFECT_INSTAKILL)
BasePoints = 1
Targets (25, 0) (TARGET_DUELVSPLAYER, NO_TARGET)

У 47724 эффект каста спелла 50239, который создает итем, но т.к. кастер нпц, а таргет TARGET_SELF, то игрок ничего не получает.

Если бы триггерные спеллы кастовал не таргет, а кастер, то: игрок кастует 47722, он же кастует 47724 и он же кастует 50239, поэтому получает итем.

Посмотрел выборочно другие спеллы с кастом триггерных спеллов на случай смены кастера, чтобы ничего не поломалось, но проблем не нашел.

Чем чревата замена

unitTarget->CastSpell(unitTarget, spellInfo, true, NULL, NULL, m_originalCasterGUID, m_spellInfo);

на

кастер->CastSpell(unitTarget, spellInfo, true, NULL, NULL, m_originalCasterGUID, m_spellInfo);

?

[rsa]

ну например в вышеприведенном 47724 игрок вместе с итемом получит еще и EFFECT_INSTAKILL здесь в общем не совсем о том разговор, цепочка кривостей начинается с 47722, который должен каститься явно в обратном направлении. как я уже не раз писал, таргет 25 - совсем не то что у нас сейчас имеется, а скорее нечто вроде SCRIPT_TARGET, сильно зависящий от условий каста.

[virusav]

Насчет 47724 не согласен.
Если кастует игрок, то:
1. SPELL_EFFECT_SCRIPT_EFFECT и TARGET_SELF, т.е. эффект на себя.
2. SPELL_EFFECT_INSTAKILL и TARGET_DUELVSPLAYER, т.е. убить цель, противоположную кастеру, а это нпц, который является таргетом.