Удаление ненужной таблицы `npc_gossip`

[virusav]

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

Патч во вложении.
Запрос для удаления таблицы:

DROP TABLE IF EXISTS `npc_gossip`;

[KiriX]

Может ещё дропнешь лишние поля в quest_template - ReqSpellCast1-4
Уже обсуждали, что на оффе этого поля нет, ни по одному кэшу не приходят. Сейчас поле для хаков...

[Vladimir]

Ну в базе не обязаны быть только поля с данными присылаемыми клиентом. Другое дело что если они не используются или таже функциональность может быть реализованы более корекным путем.

[KiriX]

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

Ну в базе не обязаны быть только поля с данными присылаемыми клиентом. Другое дело что если они не используются или таже функциональность может быть реализованы более корекным путем.

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

В YTDB я нашёл всего 20 квестов, использующих это поле.

[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?

[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]

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

[schmoozerd]

But 140 uses in udb still - so removal is not on high priority

[KiriX]

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

But 140 uses in udb still - so removal is not on high priority

As you wish

[KiriX]

UP! Schmoo - cmangos really can drop npc_gossip table.