|
Принятые патчи Иногда выкладывают патчи, которые потом в итоге все-таки принимают в ядро.
Повод для гордости. |
|
Опции темы | Поиск в этой теме | Опции просмотра |
16.06.2010, 21:04 | #1 |
Администратор
|
[patch]Temper Quel'Delar[69922]
Игрок кастует спелл на нпц, а тот в ответ кастует спелл с эффектом создания итема (квест http://ru.wowhead.com/quest=24560).
Патч работает, сравнил визуальный эффект с видео с офа - совпадает. Код:
diff --git a/src/game/SpellEffects.cpp b/src/game/SpellEffects.cpp index d8f6aac..58b7c53 100644 --- a/src/game/SpellEffects.cpp +++ b/src/game/SpellEffects.cpp @@ -1770,6 +1770,14 @@ void Spell::EffectDummy(SpellEffectIndex eff_idx) m_caster->CastSpell(m_caster, spell_id, true); return; } + case 69922: // Temper Quel'Delar + { + if (!unitTarget || unitTarget->GetTypeId() != TYPEID_UNIT) + return; + // Return Tempered Quel'Delar + unitTarget->CastSpell(m_caster, 69956, true); + return; + } } break; } |
16.06.2010, 21:12 | #2 |
MaNGOS Dev
Регистрация: 07.03.2010
Сообщений: 314
Сказал(а) спасибо: 30
Поблагодарили 153 раз(а) в 83 сообщениях
|
return пропустил и проверку unitTarget на NULL
|
16.06.2010, 21:17 | #3 |
Администратор
|
Проверку специально не стал делать, т.к. у спелла цель каста TARGET_SCRIPT, задается в `spell_script_target` + кастовать нельзя в бою.
Насчет return согласен, пропустил, первый пост поправил. |
16.06.2010, 21:25 | #4 |
Администратор
|
Добавил проверку в первый пост.
|
17.06.2010, 12:50 | #6 |
Администратор
|
Не пробовал.
|
17.06.2010, 14:17 | #8 |
Ученый
Регистрация: 07.03.2010
Сообщений: 138
Сказал(а) спасибо: 200
Поблагодарили 143 раз(а) в 49 сообщениях
|
С какого перепугу думми-эффект спелла реализовывать через базу?
|
17.06.2010, 14:26 | #9 |
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Чисто руководствуясь логикой - зачем править код (тем более такие патчи достаточно долго рассматриваются), если это вполне реализуемо силами базы?
Кроме того думми - серверный скрипт и не важно, что его обработка будет выводится с пом скрипты EventAI из базы |
17.02.2011, 15:56 | #11 |
Администратор
|
Принято в 11180.
|