[9312] Implement DB storage of new quest_template field RewHonorMultiplier
 

http://github.com/mangos/mangos/comm...5e91e9e2b7f6e1

Тут возникли проблемы с базой, не даётся хонор за квесты. Стал копаться...
Накопался и меня заинтересовал данный коммит.
Что-то здесь не логично...
Переименовали вполне логично называющееся поле RewHonorableKills в мало что говорящее RewHonorAddition и ввели новое поле RewHonorMultiplier, однако, количество хонра за квест по прежнему вычисляется так, что берутся данные из поля RewHonorAddition в качестве количества честьприносящих убийств. Не логичнее ли тогда вернуть старое понятное название или всё-таки начать пересчитывать количество хонора с использованием нового поля RewHonorMultiplier?

P.S. Не навижу дебильное "слишком короткое сообщение"

По-сути RewHonorMultiplier это и есть RewHonorableKills. А значение 62 уже устарело.

Возможно... хонором не занимался давно...

RewHonorableKills стало RewHonorAddition, вы хотите сказать, что сейчас в базе по сути два совершенно одинаковых поля?

Если изменили название поля не значит что его назначение осталось прежним.

Но в том-то и дело, что его назначение осталось прежним - оно указывает награду в виде количества убийств игроком, которые принесут ему честь (убийств эквивалентных уровню игрока персов).
Поэтому и возник вопрос.
Мультиплиер - нужен лишь для отображения количества хонора в клиента при просмотре квеста - и всё. Награда хонором берётся и вычисляется ТОЛЬКО из соседнего поля.

Пообщался с Юрием... Он утверждает что все данные для квестов приходят со снифов и кэшей и заполненно 100% корректно. Но RewHonorAddition заполнено только для одного (неиспользуемого) квеста. Во всех остальных приходит только значение RewHonorMultiplier, а значит с текущей обработкой в коде и заполненными по оффу данными для базы ни за один квест получить хонор нельзя. Юра в чём-то сомневается и зачем-то оправляет меня поговорить с Владимиром. Вот только о чём - я так и не понял:
(речь идёт о том, что во всех базах заполнено только поле RewHonorMultiplier, а второе остаётся пустым.

Хм... Как я понял, подсчёт реально получаемого хонора должен был быть поправлено позже, но до этого так и не дошло =) С 9312 коммита =) Или я неправильно понял перевод коммента коммита и он означал, что и реальная награда теперь считает корректно, однако видно, что данный коммит реальную награду хонора совершенно никак не затрагивает.
http://github.com/mangos/mangos/comm...5e91e9e2b7f6e1

KiriX, ну так вы сами ответили на все ваши вопросы.
Просто надо это исправить и всё.

Добавлено: этот прост был до edit-а предыдущего

http://ru-mangos.ru/showthread.php?t=1701 вот это надо реализовать, тут ничего сложного.

Хм... Судя по всему действительно ничего сложного, тем более что Том даже дал весь необходимый код, который осталось только прикрутить в нужное место. Но почему, если это так просто, это ещё не сделано? Где-то подвох =)))

Я сделал, но не мог проверить свою гипотезу о записях после 100.

Если сделан коммит, можно ссылочку, чтобы не делать двойную работу?

Коммита нет, оно в свалке со всякими моими наработками.

ЖаЛ =) Буду ваять сам подглядывая в твой код ;)

Для поля "RewHonorAddition" данные только в трех квестах с фулл кеша, см. любую базу.
А "RewHonorMultiplier" у всех с кеша заполнена идентично (+/- кто когда последний раз обновлял это поле).

Суть не в том. Я тебе уже говорил, как это считается в ядре. Благодаря zergtmn стало понятно, что давно пора принять одно обновление, которое до сих пор не вносят и хз когда внесут. А если не внесут и база будет упёрта в кэш и снифы - значит никогда не будет даваться хонор за квесты.
На кого давить - должны решать "ядерщики"...

Кстате в сниффе квестов с Катаклизма, ситуация с этими полями такаяже, в первом поле значение есть только для трех квестов (видимо уже не используется как сейчас так и в будущем), а во втором .... много данных. )

А воз и ныне там =( Обидно =(
Пошёл допиливать по ссылке из 10 поста и тестировать...