Ru-MaNGOS

Вернуться   Ru-MaNGOS > Ядро > Баг-репорты

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

Баг-репорты Описываем проблемы и ошибки работы ядра

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 02.06.2010, 22:27   #1
LomanChePolino
Гость
 
Сообщений: n/a
По умолчанию Счеты алтора=50359 (spell=71611)

Ядро: 10011 чистое
Серьга http://ru.wowhead.com/item=50359
Цитата:
Каждый раз, когда вы исцеляете союзника, с некоторой вероятностью вы можете мгновенно восстановить 5550 - 6450 ед. здоровья находящейся поблизости дружественной цели.
На деле:
1 происходит каст на всех дружественных целей поблизости(включая носителя и мобов).
2 срабатывает от нанесения переодического урона заклинаниями(легко воссоздается паладином, заклинанием Освящение)
По описанию эффекта, шанс срабатывания 30%, на деле это число намного выше.
  Ответить с цитированием
Старый 25.11.2010, 18:40   #2
Most
Гость
 
Сообщений: n/a
По умолчанию

2) В базу залить

-- (71611) Item - Icecrown 25 Normal Healer Trinket 2
DELETE FROM spell_proc_event WHERE entry = 71611;
INSERT INTO spell_proc_event VALUES
(71611, 0x00, 0x00, 0x00000000, 0x00000000, 0x00000000,0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00044000, 0x00000000, 0, 0, 0);
  Ответить с цитированием
Старый 25.11.2010, 20:06   #3
gmltA
Гость
 
Сообщений: n/a
По умолчанию

1) Когда-то это фиксилось так
Код:
diff --git a/src/game/Spell.cpp b/src/game/Spell.cpp
index 42cecec..e65935c 100644
--- a/src/game/Spell.cpp
+++ b/src/game/Spell.cpp
@@ -2066,6 +2066,11 @@ void Spell::SetTargetMap(SpellEffectIndex effIndex, uint32 targetMode, UnitList&
 
             FillRaidOrPartyHealthPriorityTargets(targetUnitMap, m_caster, target, radius, count, true, false, true);
             }
+            // Item - Icecrown 25 Heroic/Normal Healer Trinket 2
+            else if (m_spellInfo->Id == 71641 || m_spellInfo->Id == 71610)
+            {
+                FillRaidOrPartyHealthPriorityTargets(targetUnitMap, m_caster, m_caster, radius, 1, true, false, false);
+            }
             else
                 FillAreaTargets(targetUnitMap, m_targets.m_destX, m_targets.m_destY, radius, PUSH_DEST_CENTER, SPELL_TARGETS_FRIENDLY);
             break;
Автор: Wowka321

Последний раз редактировалось gmltA; 25.11.2010 в 20:54.
  Ответить с цитированием
Старый 25.11.2010, 22:06   #4
virusav
Администратор
 
Аватар для virusav
 
Регистрация: 19.02.2010
Сообщений: 492
Сказал(а) спасибо: 55
Поблагодарили 341 раз(а) в 154 сообщениях
virusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человек
Отправить сообщение для virusav с помощью ICQ
По умолчанию

Цели: TARGET_DUELVSPLAYER_COORDINATES, TARGET_ALL_FRIENDLY_UNITS_IN_AREA.
Из-за TARGET_ALL_FRIENDLY_UNITS_IN_AREA работает на всех дружественных целях.

Возможно, здесь должно быть сочетание данных целей, т.е. TARGET_DUELVSPLAYER_COORDINATES выбирается из TARGET_ALL_FRIENDLY_UNITS_IN_AREA.
virusav вне форума   Ответить с цитированием
Старый 26.11.2010, 05:18   #5
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Не придумывайте того чего не надо тем более когда это противоречит как цели выбираются во всех других комбнаций. Для арейной выборки целей нужны координаты, вот они первыми и указываются. Но кстати из описания не следует что дружественная цель (1) должна быть игроком из райда, и (2) и должна выбирать цель приоритетно. И кстати патч еще и выбирает цель вблизи кастера, а по описанию вроде имеется ввиду цель вблизи первой цели. Вообщем патч основан непонятно на чем. Тем более что изначальный код _корректен_ просто количество целей ограничено 1. Что уже в ядре много раз правилось - не какой-то перпетум мобилне.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 26.11.2010, 08:29   #6
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Цитата:
Сообщение от Vladimir Посмотреть сообщение
Не придумывайте того чего не надо тем более когда это противоречит как цели выбираются во всех других комбнаций. Для арейной выборки целей нужны координаты, вот они первыми и указываются. Но кстати из описания не следует что дружественная цель (1) должна быть игроком из райда, и (2) и должна выбирать цель приоритетно. И кстати патч еще и выбирает цель вблизи кастера, а по описанию вроде имеется ввиду цель вблизи первой цели. Вообщем патч основан непонятно на чем. Тем более что изначальный код _корректен_ просто количество целей ограничено 1. Что уже в ядре много раз правилось - не какой-то перпетум мобилне.
В комментария на ВХ всё чётко описано, как должен работать спелл...
Да, не указано, что цель должна быть игрокой из рейда, но не указано, что ближайшую от текущей цели цель. Читаем внимательнее:
Цитата:
с некоторой вероятностью вы можете мгновенно восстановить 5550 to 6450 ед. здоровья находящейся поблизости (кого? Не указано, значит поблизости читающего, значит поблизости кастера) дружественной цели (ну а это просто описание цели, какой цели? - дружественной цели).
И приоритет есть, два коммента с ВХ:
Цитата:
Хиляет игрока у кого меньше всего хп, включая самого хилера
Криты по 10-11К идут. Срабатывает с 1 тиком хота. Так у меня, у рестодру.
Цитата:
отличная тринька, мне нравится. Частые проки, вкусный бонус к спд, хилит того, кого нужно.
Ну и конечно - цель одна, а не все вокруг кастера...
KiriX вне форума   Ответить с цитированием
Старый 26.11.2010, 11:03   #7
rsa
Почетный флудер
Старожил
 
Аватар для rsa
 
Регистрация: 08.03.2010
Адрес: Мурманск, Россия
Сообщений: 788
Сказал(а) спасибо: 55
Поблагодарили 333 раз(а) в 151 сообщениях
Записей в дневнике: 1
rsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранитаrsa Как самоцвет среди гранита
По умолчанию

У меня вообще крайнее время создается впечатление, что TARGET_DUELVSPLAYER* названы так неправильно, и неправильно определен принцип их работы. Эти таргеты должны представлять из себя скрипт на выборку из второго таргета, или если его нет - то просто скрипт. Уж очень сложные зависимости у целей с этими таргетами...
rsa вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
KiriX (26.11.2010)
Ответ


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

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


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
[10416] fix 71611 proc and remove check for family name if no spell_proc_event is defined newsbot CMaNGOS Commits 0 28.08.2010 09:30


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


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