[9312] Implement DB storage of new quest_template field RewHonorMultiplier

[KiriX]

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

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

[Vladimir]

// rewarded honor points
data << uint32(pQuest->GetRewHonorAddition());
data << float(pQuest->GetRewHonorMultiplier()); // new reward honor (multiplied by ~62 at client side)

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

[LordJZ]

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

[Vladimir]

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

[KiriX]

Сообщение от LordJZ

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

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

[LordJZ]

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

[KiriX]

Сообщение от LordJZ

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

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

[KiriX]

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

Поговори с Владимиром для начала, не могли все до единой базы ошибиться одинакого.

(речь идёт о том, что во всех базах заполнено только поле RewHonorMultiplier, а второе остаётся пустым.

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

[LordJZ]

KiriX, ну так вы сами ответили на все ваши вопросы.

Сообщение от KiriX

...
Награда хонором берётся и вычисляется ТОЛЬКО из соседнего поля.

Просто надо это исправить и всё.

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

[zergtmn]

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

[KiriX]

Сообщение от zergtmn

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

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

[zergtmn]

Сообщение от KiriX

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

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

[KiriX]

Сообщение от zergtmn

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

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

[zergtmn]

Сообщение от KiriX

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

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

[KiriX]

Сообщение от zergtmn

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

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

[YuruY]

речь идёт о том, что во всех базах заполнено только поле RewHonorMultiplier, а второе остаётся пустым.

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

[KiriX]

Сообщение от YuruY

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

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

[YuruY]

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

[KiriX]

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