[10910] [fix] Spell::EffectQuestComplete

[KiriX]

Сейчас разбирался с квестом и наткнулся на один спелл:
http://ru.wowhead.com/spell=31336
По логике спелла и данным с ВХ - данный спелл должен быть скастован НПС. Попробовал реализовать через EvemtAI, однако квест не засчитался. Механика квеста и самого спелла не позволяет сделать грамотно так, чтобы этот спелл кастовался игроком (квест может выполняться группой игроков (и должен), таргетов у НПС может быть несколько и не обязательно это будет игрок). Решил покопаться и понял.
Спелла 31336 имеет второй эффект - квест комплит и целью его должны стать цели: TARGET_ALL_ENEMY_IN_AREA. Почему же не происходит комплит?
Стал копать обработки и наткнулся на то, что для мангоса спеллы с комплитом кастует или только игрок или они не обрабатываются вообще, что неверно, учитывая указание цели в самом спелле.
Доработал обработчик следующим образом - тогда получается всё верно и есть страховка от ошибки если цель - не игрок:

PHP код:

diff --git a/src/game/SpellEffects.cpp b/src/game/SpellEffects.cpp
index 096dae3..70ea439 100644
--- a/src/game/SpellEffects.cpp
+++ b/src/game/SpellEffects.cpp
@@ -7182,10 +7182,10 @@ void Spell::EffectReputation(SpellEffectIndex eff_idx)
 
 void Spell::EffectQuestComplete(SpellEffectIndex eff_idx)
 {
-    if(m_caster->GetTypeId() != TYPEID_PLAYER)
+    if(unitTarget->GetTypeId() != TYPEID_PLAYER)
         return;
 
-    Player *_player = (Player*)m_caster;
+    Player *_player = (Player*)unitTarget;
 
     uint32 quest_id = m_spellInfo->EffectMiscValue***91;eff_idx***93;;
     _player->AreaExploredOrEventHappens(quest_id); 


[Vinolentus]

Здорово.
А случаев, когда кастер - игрок, чей квест должен завершиться, но цель - не игрок, быть не может?

[KiriX]

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

Здорово.
А случаев, когда кастер - игрок, чей квест должен завершиться, но цель - не игрок, быть не может?

Не могу себе такого представить - это раз.
Два: неоднократно видел спеллы, завершающие квест, которые должны кастоваться НПС, но базе и скриптам приходится выёживаться вызывая каст спелла именно игроком именно из-за текущей реализации.
Три: цель не игроком быть не может, т.к. не игрок не может иметь квеста.

[Vinolentus]

Ну почему же. Такое бы даже корректно работало раньше. Впрочем, подобных заклинаний нет, и, видимо, так быть не должно.

[virusav]

Было бы неплохо узнать, как поведет себя ядро с данным патчем в случае таких целей с данным эффектом:
http://ru.wowhead.com/spell=19719 - TARGET_SELF
http://ru.wowhead.com/spell=23658 - TARGET_SINGLE_FRIEND
http://ru.wowhead.com/spell=24875 - TARGET_DUELVSPLAYER
http://ru.wowhead.com/spell=25201 - TARGET_CASTER_COORDINATES, TARGET_AREAEFFECT_INSTANT
http://ru.wowhead.com/spell=30790 - TARGET_CASTER_COORDINATES, TARGET_ALL_ENEMY_IN_AREA
http://ru.wowhead.com/spell=31377 - TARGET_CASTER_COORDINATES, TARGET_ALL_FRIENDLY_UNITS_AROUND_CASTER
http://ru.wowhead.com/spell=44124 - TARGET_MASTER, TARGET_AREAEFFECT_PARTY

Для квеста на поимку Матиса есть спеллы:
http://ru.wowhead.com/spell=31336
http://ru.wowhead.com/spell=31337
http://ru.wowhead.com/spell=31339
http://ru.wowhead.com/spell=31377

Может, просто не тот спелл кастуется?
Нужен снифф.

[KiriX]

Цели для комплита квеста будут определяться точно также как и при касте любого другого спелла с подобными целями.
Да и в любом случае - целью спелла должны быть именно те цели, которые указаны в спелле в дбц. Что-то я не часто вижу, чтобы в спеллах подменялись цели, которые указаны у них в дбц, почему же для этого эффекта делается исключение?

Спеллы для поимки Матиса:
В текущем варианте работы ядра все эти спеллы должны быть обязательно скастованы от игрока, что даже скриптом SD2 реализовать не так просто будет + ни один из приведённых спеллов не имеет для эффекта комплита цели self, что было бы логично если бы спелл должен был быть скастован игроком.

[virusav]

31339 - TARGET_DUELVSPLAYER
Может, госсип? Тут уже нужна точная инфа по данному квесту.

[KiriX]

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

31339 - TARGET_DUELVSPLAYER
Может, госсип? Тут уже нужна точная инфа по данному квесту.

Это всё равно сути не меняет. Почему для данного эффекта игнорируется цель и сущестует лишь одна цель: если кастер игрок - цель - кастер. Если кастре не игрок - эффект просто не обрабатывается, хотя срабатывал бы совершенно нормально, без костылей в ЭвентАИ типа "каст этого спелла целью".

[KiriX]

Что разработчики скажут по поводу данного фикса?

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

Здорово.
А случаев, когда кастер - игрок, чей квест должен завершиться, но цель - не игрок, быть не может?

Есть такой случай.... http://www.wowhead.com/quest=12842
Вот пример квеста, когда игрок должен скастовать (использовать спелл - http://www.wowhead.com/spell=53428), но не на NPC или GO.... а на итем.....
не знаю как в мангосе, а в тринити последней ревизии данный квест невыполним (без ремонта ядра).
Вот сейчас чистое ядро мангоса откомпилю, посмотрим как там.
__________________________________________________ ______________

На мангосе данный квест выполняется нормально...

Подскажите где копать в ядре? Где обработка таких квестов находится?

[KiriX]

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

Есть такой случай.... http://www.wowhead.com/quest=12842
Вот пример квеста, когда игрок должен скастовать (использовать спелл - http://www.wowhead.com/spell=53428), но не на NPC или GO.... а на итем.....
не знаю как в мангосе, а в тринити последней ревизии данный квест невыполним (без ремонта ядра).
Вот сейчас чистое ядро мангоса откомпилю, посмотрим как там.
__________________________________________________ ______________

На мангосе данный квест выполняется нормально...

Подскажите где копать в ядре? Где обработка таких квестов находится?

Тут совершенно другое дело. Не путайте тёплое с мягким Патч давно внесён в ядро (и в мангос и в триньку) - кв прекрасно выполняется...

С офф репо триньки, после компила квест не выполняется... значит в офф не исправили данный косяк..

[KiriX]

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

С офф репо триньки, после компила квест не выполняется... значит в офф не исправили данный косяк..

Хм... Странно... Я точно знаю, что на триньке все кв ДК 100% проходимы - сам тестировал. Но в коде у них действительно нет того, что есть в мангосе.
SpellEffects.cpp:

PHP код:

                case 53343:                                 // Rune of Razorice
                {
                    // Runeforging Credit
                    m_caster->CastSpell(m_caster, 54586, true);
                    return;
                } 


54586 - этот спелл комплитит квест. Эффект комплита имеет цель - target_self. Вызов данного спелла идёт таким образом, что данный спелл кастует именно игрок. К данной теме ваш комментарий всё же не имеет никакого отношения
Кв этот и на тринити 100% работал, а значит был реализован скорее всего скриптом, советую вам копать в этом направлении.

Спс.... буду копать.... Что интересно.. включил логи - 3 и на мангосе и на тринити....
абсолютно идентичны все вызовы...((( и 53343 есть...

[KiriX]

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

Спс.... буду копать.... Что интересно.. включил логи - 3 и на мангосе и на тринити....
абсолютно идентичны все вызовы...((( и 53343 есть...

Я смотрел в коде триньки - там нет такого момента в коде, что по касту 53343 вызывается нужный. Но вообще это не форум триньки, так что давай по добру завершать данную тему, к тому же это оффтопик

[KiriX]

Тема висит давно без единого коммента. Ап!

[zergtmn]

В 10910. Спасибо