Ru-MaNGOS

Вернуться   Ru-MaNGOS > Ядро > Патчи > Отвергнутые патчи

Важная информация

Отвергнутые патчи Патчи, отвергнутые от приёма в GIT

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 22.08.2010, 17:23   #1
Lordronn
Умный
 
Регистрация: 17.06.2010
Сообщений: 397
Сказал(а) спасибо: 58
Поблагодарили 55 раз(а) в 38 сообщениях
Lordronn Скоро придёт к известности
По умолчанию [FIX] Escape artist

Правим работу гномского расового заклинания.
Автор - я(Heisenberg)
PHP код:
diff --git a/src/game/SpellEffects.cpp b/src/game/SpellEffects.cpp
--- a/src/game/SpellEffects.cpp
+++ b/src/game/SpellEffects.cpp
@@ -5497,11 +5497,11 @@
                 }
                 case 
20589:                                 // Escape artist
                 
{
-                    if (!
unitTarget)
+                    if (!
m_caster)

-                    
unitTarget->RemoveSpellsCausingAura(SPELL_AURA_MOD_ROOT);
-                    
unitTarget->RemoveSpellsCausingAura(SPELL_AURA_MOD_DECREASE_SPEED);
+                    
m_caster->RemoveSpellsCausingAura(SPELL_AURA_MOD_ROOT);
+                    
m_caster->RemoveSpellsCausingAura(SPELL_AURA_MOD_DECREASE_SPEED);
                     return;
                 }
                 case 
24590:                                 // Brittle Armor - need remove one 24575 Brittle Armor aura 

Последний раз редактировалось Lordronn; 22.08.2010 в 18:14.
Lordronn вне форума   Ответить с цитированием
Старый 22.08.2010, 18:02   #2
NeatElves
YTDB Dev
 
Аватар для NeatElves
 
Регистрация: 06.03.2010
Сообщений: 259
Сказал(а) спасибо: 28
Поблагодарили 280 раз(а) в 136 сообщениях
NeatElves Как самоцвет среди гранитаNeatElves Как самоцвет среди гранитаNeatElves Как самоцвет среди гранита
По умолчанию

И при чем тут пути триньки ? Форумом ошиблись ?
NeatElves вне форума   Ответить с цитированием
Старый 22.08.2010, 18:14   #3
Lordronn
Умный
 
Регистрация: 17.06.2010
Сообщений: 397
Сказал(а) спасибо: 58
Поблагодарили 55 раз(а) в 38 сообщениях
Lordronn Скоро придёт к известности
По умолчанию

Ой, простите. Это от патча для тринити. Там такая же проблема
Lordronn вне форума   Ответить с цитированием
Старый 22.08.2010, 19:57   #4
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию

А что, для m_caster->RemoveSpellsCausingAura(SPELL_AURA_MOD_DECREASE_S PEED);
наличие m_caster проверить не надо?
Или я вообще что-то не понял. Если нет m_caster но ы с него снимаем ауру. а вторую снимаем пофиг есть или нет... странно это все...

Последний раз редактировалось rsa; 22.08.2010 в 19:59.
rsa вне форума   Ответить с цитированием
Старый 22.08.2010, 20:05   #5
zergtmn
MaNGOS Dev
 
Аватар для zergtmn
 
Регистрация: 07.03.2010
Сообщений: 314
Сказал(а) спасибо: 30
Поблагодарили 153 раз(а) в 83 сообщениях
zergtmn Обладатель прекрасной аурыzergtmn Обладатель прекрасной ауры
По умолчанию

TARGET A = SELF, должен без изменений работать
rsa, у спеллов есть кастер всегда, это у аур может не быть.

Последний раз редактировалось zergtmn; 22.08.2010 в 20:21.
zergtmn вне форума   Ответить с цитированием
Старый 22.08.2010, 20:15   #6
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию

Тогда надо либо проверять для обоих снятий либо не проверять для обоих. return/break потерян что ли? Тем более для таргетселф как я понимаю m_caster=unittarget.
rsa вне форума   Ответить с цитированием
Старый 22.08.2010, 20:19   #7
virusav
Администратор
 
Аватар для virusav
 
Регистрация: 19.02.2010
Сообщений: 492
Сказал(а) спасибо: 55
Поблагодарили 341 раз(а) в 154 сообщениях
virusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человек
Отправить сообщение для virusav с помощью ICQ
По умолчанию

Спелл 20589 имеет цель TARGET_SELF, т.е.
Цитата:
Сообщение от rsa Посмотреть сообщение
m_caster=unittarget
В связи с этим интересно, что правит данный патч.
virusav вне форума   Ответить с цитированием
Старый 08.09.2010, 16:21   #8
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Проверено - в мангосе прекрасно работает в текущем виде.

Цитата:
Ой, простите. Это от патча для тринити. Там такая же проблема
Нет, это проблема именно кривом коде тринити. Я не знаю как они умудрились сломать выбор self-target спелов -но им это удалось для данного случая если спел у них не работает в исходном виде кода.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
2 пользователя(ей) сказали cпасибо:
Konctantin (08.09.2010), Laise (08.09.2010)
Ответ


Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.



Текущее время: 05:39. Часовой пояс GMT +3.


ru-mangos.ru - Русское сообщество MaNGOS
Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума Ru-MaNGOS будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot