[patch] Призыв объектов на транспорте

[Insider42]

Выяснил что на оффе нельзя призывать объекты во время движения транспорта.

diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 69b2fa1..4cb8691 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -5030,6 +5030,17 @@ SpellCastResult Spell::CheckCast(bool strict)
 
                 break;
             }
+            case SPELL_EFFECT_SUMMON_OBJECT_SLOT1:
+            case SPELL_EFFECT_SUMMON_OBJECT_SLOT2:
+            case SPELL_EFFECT_SUMMON_OBJECT_SLOT3:
+            case SPELL_EFFECT_SUMMON_OBJECT_SLOT4:
+            {
+                if (m_caster->GetTypeId() == TYPEID_PLAYER)
+                    if (((Player*)m_caster)->HasMovementFlag(MOVEFLAG_ONTRANSPORT))
+                    return SPELL_FAILED_CANT_DO_THAT_RIGHT_NOW;
+                    
+                break;
+            }
             case SPELL_EFFECT_SUMMON_PET:
             {
                 if(m_caster->GetPetGUID())                  //let warlock do a replacement summon

То, что я хочу додумать:
Сумоны объектов должны также быть запрещены и в стартовой трубе даларанской арены, но как запретить там? Может какой-то особый флаг зоны выставляется у близов? Или делать такой же как, который я применил в патче даларанской арены - т.е. проверять если зона = Даларанская арена, то отмерять дистанцию от трубы и не позволять кастовать если проходит условие...

[LordJZ]

В препарейшене на арене можно ставить портал локовский, исключение - оргриммарская арена, т.к. там лифт.

В Spell.cpp функция CheckCast есть кусок кода:

    for (int i = 0; i < MAX_EFFECT_INDEX; ++i)
    {
        // for effects of spells that have only one target
        switch(m_spellInfo->Effect[i])
        {
            case SPELL_EFFECT_INSTAKILL:
...

Я думаю сюда можно поставить четыре case-а, проверку и возвращать SPELL_FAILED_NOT_HERE при наличии транспорта.

[MaS0n]

1) спросить у игроков с оффа, о том какое сообщение точно выдается при попытке поставить круг или отснифать SMSG_CAST_FAILED

Перейти в Spell.cpp - CheckCast - найти это :

    for (int i = 0; i < MAX_EFFECT_INDEX; ++i)
    {
        // for effects of spells that have only one target
        switch(m_spellInfo->Effect[i])

вставить

            case SPELL_EFFECT_SUMMON_OBJECT_SLOT1:
            {
                if (m_caster->GetTypeId() == TYPEID_PLAYER)
                    if (((Player*)m_caster)->HasMovementFlag(MOVEFLAG_ONTRANSPORT) ||
                        m_caster->HasAura(SPELL_ARENA_PREPARATION))
                    return SPELL_FAILED_BAD_TARGETS;(здесь подставить корректное сообщение об ошибке)
                    
                    break;
            }

Возможно это не совсем верное решение, но офлайк прерывание с выводом еррор мессаджа можно сделать только здесь

[Insider42]

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

В препарейшене на арене можно ставить портал локовский, исключение - оргриммарская арена, т.к. там лифт.

В Spell.cpp функция CheckCast есть кусок кода:

    for (int i = 0; i < MAX_EFFECT_INDEX; ++i)
    {
        // for effects of spells that have only one target
        switch(m_spellInfo->Effect[i])
        {
            case SPELL_EFFECT_INSTAKILL:
...

Я думаю сюда можно поставить четыре case-а, проверку и возвращать SPELL_FAILED_NOT_HERE при наличии транспорта.

Мне говорят в Даларанской арене в трубе нельзя ставить, даже без препарации. Будто бы там какой-то флаг особый выставляется, или идёт проверка по зоне и расстоянии от трубы, как у меня в патче Даларанской арены :/

[LordJZ]

Хмм, я в 3.3 на арене почти не играл -- могли и убрать возможность в трубе ставить портал. До 3.3 можно было -- инфа 100%.

SPELL_EFFECT_SUMMON_OBJECT_SLOT1 <- может быть надо 2, 3 и 4 так же?

P.S. Насколько я помню, сообщение там You can't do that right now. Только не знаю что это в цифрах

[Insider42]

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

Хмм, я в 3.3 на арене почти не играл -- могли и убрать возможность в трубе ставить портал. До 3.3 можно было -- инфа 100%.

Тогда что с ним происходило когда начинало лить с трубы?

[unlucky]

на даларанской арене в препарейшене можно ставить портал проверял сегодня(офф).
после выброса\выпрыга из трубы - не проверял можно ли его заюзать(портануться туда) но уверен что нельзя, либо он уберется там при старте арены.
порт мона ставить в препорейте на любой другой арене (кроме оргриммарская арена) и он работает при старте арены (не проверял только на далараноской).