|
CMaNGOS Commits Изменения, принятые в официальный репозиторий ядра CMaNGOS. git://github.com/cmangos/mangos-wotlk.git |
|
Опции темы | Поиск в этой теме | Опции просмотра |
03.09.2010, 02:18 | #1 |
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
[9312] Implement DB storage of new quest_template field RewHonorMultiplier
http://github.com/mangos/mangos/comm...5e91e9e2b7f6e1
Тут возникли проблемы с базой, не даётся хонор за квесты. Стал копаться... Накопался и меня заинтересовал данный коммит. Что-то здесь не логично... Переименовали вполне логично называющееся поле RewHonorableKills в мало что говорящее RewHonorAddition и ввели новое поле RewHonorMultiplier, однако, количество хонра за квест по прежнему вычисляется так, что берутся данные из поля RewHonorAddition в качестве количества честьприносящих убийств. Не логичнее ли тогда вернуть старое понятное название или всё-таки начать пересчитывать количество хонора с использованием нового поля RewHonorMultiplier? |
03.09.2010, 05:25 | #2 | |
MaNGOS Dev
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
|
Цитата:
__________________
Так как устал объяснять знайте ICQ не пользуюсь |
|
03.09.2010, 05:33 | #3 |
Супер-модератор
Регистрация: 07.03.2010
Сообщений: 647
Сказал(а) спасибо: 100
Поблагодарили 252 раз(а) в 123 сообщениях
|
По-сути RewHonorMultiplier это и есть RewHonorableKills. А значение 62 уже устарело.
|
03.09.2010, 05:36 | #4 |
MaNGOS Dev
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
|
Возможно... хонором не занимался давно...
__________________
Так как устал объяснять знайте ICQ не пользуюсь |
03.09.2010, 10:08 | #6 |
Супер-модератор
Регистрация: 07.03.2010
Сообщений: 647
Сказал(а) спасибо: 100
Поблагодарили 252 раз(а) в 123 сообщениях
|
Если изменили название поля не значит что его назначение осталось прежним.
|
04.09.2010, 14:12 | #7 | |
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Цитата:
Поэтому и возник вопрос. Мультиплиер - нужен лишь для отображения количества хонора в клиента при просмотре квеста - и всё. Награда хонором берётся и вычисляется ТОЛЬКО из соседнего поля. |
|
05.09.2010, 20:56 | #8 | |
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Пообщался с Юрием... Он утверждает что все данные для квестов приходят со снифов и кэшей и заполненно 100% корректно. Но RewHonorAddition заполнено только для одного (неиспользуемого) квеста. Во всех остальных приходит только значение RewHonorMultiplier, а значит с текущей обработкой в коде и заполненными по оффу данными для базы ни за один квест получить хонор нельзя. Юра в чём-то сомневается и зачем-то оправляет меня поговорить с Владимиром. Вот только о чём - я так и не понял:
Цитата:
Хм... Как я понял, подсчёт реально получаемого хонора должен был быть поправлено позже, но до этого так и не дошло С 9312 коммита Или я неправильно понял перевод коммента коммита и он означал, что и реальная награда теперь считает корректно, однако видно, что данный коммит реальную награду хонора совершенно никак не затрагивает. http://github.com/mangos/mangos/comm...5e91e9e2b7f6e1 |
|
05.09.2010, 21:05 | #9 |
Супер-модератор
Регистрация: 07.03.2010
Сообщений: 647
Сказал(а) спасибо: 100
Поблагодарили 252 раз(а) в 123 сообщениях
|
|
05.09.2010, 21:17 | #10 |
MaNGOS Dev
Регистрация: 07.03.2010
Сообщений: 314
Сказал(а) спасибо: 30
Поблагодарили 153 раз(а) в 83 сообщениях
|
http://ru-mangos.ru/showthread.php?t=1701 вот это надо реализовать, тут ничего сложного.
|
Пользователь сказал cпасибо: | KiriX (05.09.2010) |
05.09.2010, 21:22 | #11 | |
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Цитата:
|
|
05.09.2010, 21:46 | #12 |
MaNGOS Dev
Регистрация: 07.03.2010
Сообщений: 314
Сказал(а) спасибо: 30
Поблагодарили 153 раз(а) в 83 сообщениях
|
Я сделал, но не мог проверить свою гипотезу о записях после 100.
|
06.09.2010, 10:30 | #14 |
MaNGOS Dev
Регистрация: 07.03.2010
Сообщений: 314
Сказал(а) спасибо: 30
Поблагодарили 153 раз(а) в 83 сообщениях
|
|
06.09.2010, 15:21 | #16 | |
YTDB Dev
Регистрация: 01.02.2010
Сообщений: 288
Сказал(а) спасибо: 125
Поблагодарили 97 раз(а) в 53 сообщениях
|
Цитата:
А "RewHonorMultiplier" у всех с кеша заполнена идентично (+/- кто когда последний раз обновлял это поле). |
|
06.09.2010, 17:39 | #17 | |
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Цитата:
На кого давить - должны решать "ядерщики"... |
|
06.09.2010, 18:23 | #18 |
YTDB Dev
Регистрация: 01.02.2010
Сообщений: 288
Сказал(а) спасибо: 125
Поблагодарили 97 раз(а) в 53 сообщениях
|
Кстате в сниффе квестов с Катаклизма, ситуация с этими полями такаяже, в первом поле значение есть только для трех квестов (видимо уже не используется как сейчас так и в будущем), а во втором .... много данных. )
|
|
|
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
ReqSpellCast [quest_template] | KiriX | Патчи | 18 | 11.09.2012 14:48 |
[10621] Add new field RequiredClasses for quest_template | newsbot | CMaNGOS Commits | 0 | 19.10.2010 01:42 |
[10207] Implement ITEM_FLAGS2_EXT_COST_REQUIRES_GOLD use instead sign of ExtendedCost field. | newsbot | CMaNGOS Commits | 0 | 17.07.2010 18:30 |
[9752] Add storage for min/maxgold of gameobject | newsbot | CMaNGOS Commits | 0 | 16.04.2010 09:50 |
[9589] Add additional field for storage of speed_run rate of creature | newsbot | CMaNGOS Commits | 0 | 15.03.2010 17:00 |