YTDB

Вернуться   YTDB > Новости / News > CMaNGOS Commits

CMaNGOS Commits Изменения, принятые в официальный репозиторий ядра CMaNGOS.
git://github.com/cmangos/mangos-wotlk.git

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 17.03.2011, 22:00   #1
newsbot
News Bot
 
Аватар для newsbot
 
Регистрация: 20.11.2009
Сообщений: 1,290
Сказал(а) спасибо: 0
Поблагодарили 5 раз(а) в 5 сообщениях
newsbot На верном пути
По умолчанию [11261] Check gaps in quest RewChoiceItem* data.

автор: VladimirMangos


<pre>m src/game/ObjectMgr.cpp
</pre>
<pre style='white-space:pre-wrap;width:81ex'>[11261] Check gaps in quest RewChoiceItem* data.

Client can crash if data not continues.

Thanks Chero and Joey for help in research problem ^^</pre>




Дальше...
newsbot вне форума   Ответить с цитированием
Старый 17.03.2011, 22:13   #2
YuruY
YTDB dev
 
Аватар для YuruY
 
Регистрация: 05.11.2009
Адрес: Thunder Bluff
Сообщений: 3,625
Сказал(а) спасибо: 7,839
Поблагодарили 3,255 раз(а) в 498 сообщениях
YuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспорима
По умолчанию

Смысл?
__________________
Правила написания баг-репортов.
Общие правила поведения - рекомендуемые исключительно всем для прочтения.
YuruY вне форума   Ответить с цитированием
Старый 17.03.2011, 22:40   #3
virusav
Брянский волк
 
Аватар для virusav
 
Регистрация: 06.11.2009
Сообщений: 588
Сказал(а) спасибо: 30
Поблагодарили 433 раз(а) в 260 сообщениях
virusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человек
Отправить сообщение для virusav с помощью ICQ
По умолчанию

Код:
2011-03-17 22:29:43 Quest 316 has `RewChoiceItemId4` = 0 but `RewChoiceItemId5` = 33079, client can crash at like data.
2011-03-17 22:29:43 Quest 316 has `RewChoiceItemId2` = 0 but `RewChoiceItemId3` = 22114, client can crash at like data.
Поставил итемы и количества по-порядку - ошибки исчезли.

Но если данные придут в таком порядке из кеша, то при следующем обновлении данными из кеша все вернется назад, снова будут ошибки в логе.
virusav вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
Vladimir (18.03.2011)
Старый 18.03.2011, 00:47   #4
Vladimir
MaNGOS Dev
 
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
Vladimir Скоро придёт к известностиVladimir Скоро придёт к известности
По умолчанию

Вы уверены что такие данные есть? Я проверял по данным из UDB - там таких случаев нет. Если такие данные есть то я откачу, конечно, тем более у меня подтверждения для 3.x ytт креша - но клиент 1.12.1 явно крешится на таких данных.
Vladimir вне форума   Ответить с цитированием
Старый 18.03.2011, 01:10   #5
NeatElves
YTDB dev
 
Аватар для NeatElves
 
Регистрация: 05.11.2009
Сообщений: 5,920
Сказал(а) спасибо: 91
Поблагодарили 5,383 раз(а) в 4,847 сообщениях
NeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспоримаNeatElves Репутация неоспорима
По умолчанию

Есть там предметы по кешу)):
Код:
REPLACE INTO quest_template VALUES ('enGB', 316, 415, 316, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 7, 50, 60, 0, 0, 0, 0, 0, 8, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 20371, 1, 20651, 1, 22114, 1, 22780, 1, 33079, 1, 0, 0, 469, 0, 0, 0, 0, 7, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, '<UNUSED 1>', '', '', '', 'Return to Test Squirrel.', 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 2688, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, '', '', '', '');
NeatElves вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
Vladimir (18.03.2011)
Старый 18.03.2011, 01:44   #6
Vladimir
MaNGOS Dev
 
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
Vladimir Скоро придёт к известностиVladimir Скоро придёт к известности
По умолчанию

f*ck. Я уже revert сделал
Vladimir вне форума   Ответить с цитированием
Старый 18.03.2011, 12:21   #7
virusav
Брянский волк
 
Аватар для virusav
 
Регистрация: 06.11.2009
Сообщений: 588
Сказал(а) спасибо: 30
Поблагодарили 433 раз(а) в 260 сообщениях
virusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человек
Отправить сообщение для virusav с помощью ICQ
По умолчанию

Не надо торопиться.
Цитата:
Но если данные придут в таком порядке из кеша, то при следующем обновлении данными из кеша все вернется назад, снова будут ошибки в логе.
Цитата:
Вы уверены что такие данные есть?
Я не успел ответить, т.к. ночь была на дворе, спать пошел.

Сегодня прогнал скриптом 9408 квестов из кеша, но не встретил ни одного квеста, где бы встречались итемы в качестве вознаграждения, не идущие в полях подряд.

В таком случае лучше вернуть коммит, т.к. он, как я понимаю, пойдет для 1.12.1 и мастер-ветки.
virusav вне форума   Ответить с цитированием
Старый 18.03.2011, 17:10   #8
Vladimir
MaNGOS Dev
 
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
Vladimir Скоро придёт к известностиVladimir Скоро придёт к известности
По умолчанию

Ну для 1.x он просто необходим так как клиент крешится на такие данные, а для 3.x/2.x видимо верну/добавлю раз помогло поймать глюк в данных.
Vladimir вне форума   Ответить с цитированием
Пользователь сказал cпасибо:
virusav (18.03.2011)
Старый 18.03.2011, 19:50   #9
YuruY
YTDB dev
 
Аватар для YuruY
 
Регистрация: 05.11.2009
Адрес: Thunder Bluff
Сообщений: 3,625
Сказал(а) спасибо: 7,839
Поблагодарили 3,255 раз(а) в 498 сообщениях
YuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспорима
По умолчанию

Может данные нужны по какомуто квесту, проверить?
335а, (401 убил, но на работе остался вроде) 403а, 406а? (фулл есть с нескольких клиентов)
__________________
Правила написания баг-репортов.
Общие правила поведения - рекомендуемые исключительно всем для прочтения.
YuruY вне форума   Ответить с цитированием
Старый 18.03.2011, 23:08   #10
Vladimir
MaNGOS Dev
 
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
Vladimir Скоро придёт к известностиVladimir Скоро придёт к известности
По умолчанию

Я откатил - т.е. передобавил проверку, так что если какие еще случаи будут это будет сообщенно ядром при загрузке.
Vladimir вне форума   Ответить с цитированием
Старый 22.03.2011, 03:28   #11
Vladimir
MaNGOS Dev
 
Регистрация: 20.11.2009
Сообщений: 25
Сказал(а) спасибо: 17
Поблагодарили 93 раз(а) в 35 сообщениях
Vladimir Скоро придёт к известностиVladimir Скоро придёт к известности
По умолчанию

Цитата:
Я не успел ответить, т.к. ночь была на дворе, спать пошел.
Мои эмоции касались исключительно моей торопыжкости
Vladimir вне форума   Ответить с цитированием
Старый 24.03.2011, 13:07   #12
YuruY
YTDB dev
 
Аватар для YuruY
 
Регистрация: 05.11.2009
Адрес: Thunder Bluff
Сообщений: 3,625
Сказал(а) спасибо: 7,839
Поблагодарили 3,255 раз(а) в 498 сообщениях
YuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспоримаYuruY Репутация неоспорима
По умолчанию

Код:
 UPDATE `quest_template` SET `RewChoiceItemId1` = 20371, `RewChoiceItemId2` = 20651, `RewChoiceItemId3` = 22114, `RewChoiceItemId4` = 22780, `RewChoiceItemId5` = 33079 WHERE `entry` = 316;
вот так по кешу, но второго и четвертого предмета нет в базе (квест не юзабельный, в кеше они не приходят), если их занулить - ядро даст ошибку
__________________
Правила написания баг-репортов.
Общие правила поведения - рекомендуемые исключительно всем для прочтения.
YuruY вне форума   Ответить с цитированием
Ответ


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

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


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
[10590] Check if GO is requirement of quest to activate sparkle if it is newsbot CMaNGOS Commits 0 08.10.2010 14:21
[10015] More redundant data check in `spell_proc_event` newsbot CMaNGOS Commits 0 01.06.2010 04:58
[9334] Check if baseLevel for XP is valid before attempt use data from store newsbot CMaNGOS Commits 0 09.02.2010 02:22
[9275] Quest reward data rearranged and comments added. newsbot CMaNGOS Commits 0 31.01.2010 19:30
[9218] Fixed check in CMSG_QUEST_POI_QUERY proccessing for max quest amount. newsbot CMaNGOS Commits 0 20.01.2010 07:50


Текущее время: 00:35. Часовой пояс GMT +4.


YTDB - MaNGOS DataBase

Русский форум Мангос - официальный форум RMDC

Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума YTDB будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot
Punk's On-Air - Punk, Folk, Blues, Ska Bascinet - реконструкция denw IL-2 Training Книготорговая Компания «А-5» - художественная, учебная литература, скидки, опт и мелкий опт, доставка, дешево, низкие цены издательства