Ru-MaNGOS  

Вернуться   Ru-MaNGOS > Ядро > Патчи > Патчи на рассмотрении

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

Патчи на рассмотрении Рассматриваемые к принятию патчи

Ответ
 
Опции темы Поиск в этой теме Опции просмотра
Старый 25.03.2012, 23:42   #1
virusav
Администратор
 
Аватар для virusav
 
Регистрация: 19.02.2010
Сообщений: 491
Сказал(а) спасибо: 56
Поблагодарили 341 раз(а) в 154 сообщениях
virusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человекvirusav Реально хороший человек
Отправить сообщение для virusav с помощью ICQ
По умолчанию Удаление ненужной таблицы `npc_gossip`

Таблица `npc_gossip` больше не нужна, т.к. все меню можно реализовать через таблицы `gossip_...`.

Патч во вложении.
Запрос для удаления таблицы:
Код:
DROP TABLE IF EXISTS `npc_gossip`;
Вложения
Тип файла: diff NPC_GOSSIP_REMOVE.diff (6.6 Кб, 19 просмотров)
virusav вне форума   Ответить с цитированием
3 пользователя(ей) сказали cпасибо:
KiriX (26.03.2012), partizanes (06.04.2012), YuruY (26.03.2012)
Старый 05.04.2012, 11:06   #2
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Может ещё дропнешь лишние поля в quest_template - ReqSpellCast1-4
Уже обсуждали, что на оффе этого поля нет, ни по одному кэшу не приходят. Сейчас поле для хаков...
KiriX вне форума   Ответить с цитированием
Старый 05.04.2012, 11:40   #3
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 440 раз(а) в 183 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Ну в базе не обязаны быть только поля с данными присылаемыми клиентом. Другое дело что если они не используются или таже функциональность может быть реализованы более корекным путем.
__________________
Так как устал объяснять знайте ICQ не пользуюсь
Vladimir вне форума   Ответить с цитированием
Старый 05.04.2012, 14:33   #4
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Цитата:
Сообщение от Vladimir Посмотреть сообщение
Ну в базе не обязаны быть только поля с данными присылаемыми клиентом. Другое дело что если они не используются или таже функциональность может быть реализованы более корекным путем.
Все квесты, использующие эти поля имеют неверные данные, если смотреть на то, что приходит по кэшу.
Реализуется большинство таких кв через EventAI или скрипты, чаще всего spell_dummy_npc.
Короче, без этого поля можно легко обойтись, оно может быть полезно лишь для собственных кв, чтобы не заморачиваться с EventAI.

В YTDB я нашёл всего 20 квестов, использующих это поле.
KiriX вне форума   Ответить с цитированием
Старый 05.04.2012, 16:25   #5
schmoozerd
MaNGOS Dev
 
Регистрация: 17.11.2011
Сообщений: 99
Сказал(а) спасибо: 35
Поблагодарили 80 раз(а) в 26 сообщениях
schmoozerd Скоро придёт к известности
По умолчанию

About the npc_gossip table I also agree in general that it should be removed, but udb is not yet ready for this.

Actually I tend to agree with you guys that it might be well worth the extra effort to remove the ReqSpellCastX fields
Only problem I might see here, is that mobs that are killable would now just be killable, instead of requiring a spell-cast; do such cases exist?
schmoozerd вне форума   Ответить с цитированием
Старый 05.04.2012, 16:54   #6
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Цитата:
Сообщение от schmoozerd Посмотреть сообщение
Only problem I might see here, is that mobs that are killable would now just be killable, instead of requiring a spell-cast; do such cases exist?
Afraid I can't understand you
If I understand you correctly - this case is impossible.
In all quest there need cast something on the npc, ReqCreatureOrGOId contain entry of other npc, not entry npc which we should cast required spell on...

Sorry for my (and http://translate.google.com) English
KiriX вне форума   Ответить с цитированием
3 пользователя(ей) сказали cпасибо:
partizanes (06.04.2012), schmoozerd (05.04.2012), Vladimir (05.04.2012)
Старый 06.04.2012, 14:30   #7
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

At that moment -8 quests with using ReqSpellCast fields in YTDB (thx NeatElves). No problem with removing all hack quest which use ReqSpellCast fields.

So just 12 quest using ReqSpellCast fields now (YTDB). Work will be continue, I think
KiriX вне форума   Ответить с цитированием
Старый 06.04.2012, 15:50   #8
schmoozerd
MaNGOS Dev
 
Регистрация: 17.11.2011
Сообщений: 99
Сказал(а) спасибо: 35
Поблагодарили 80 раз(а) в 26 сообщениях
schmoozerd Скоро придёт к известности
По умолчанию

But 140 uses in udb still - so removal is not on high priority
schmoozerd вне форума   Ответить с цитированием
Старый 06.04.2012, 19:17   #9
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

Цитата:
Сообщение от schmoozerd Посмотреть сообщение
But 140 uses in udb still - so removal is not on high priority
As you wish
KiriX вне форума   Ответить с цитированием
Старый 04.10.2012, 01:52   #10
KiriX
Умный
Старожил
 
Аватар для KiriX
 
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 431 раз(а) в 181 сообщениях
Записей в дневнике: 4
KiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человекKiriX Реально хороший человек
По умолчанию

UP! Schmoo - cmangos really can drop npc_gossip table.
KiriX вне форума   Ответить с цитированием
Ответ

Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

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

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


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Структура таблицы character_stats tozter Прочая документация 18 30.05.2011 16:31
[Crash?] удаление GO zergtmn Баг-репорты 0 29.04.2010 22:31


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


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