ReqSpellCast [quest_template]

[KiriX]

Давно уже думаю об этом, и вот, кажется, настала пора написать это.
Эти поля не нужны.
База YTDB уже давно перешла на почти полностью безхаковый режим работы (насколько это вообще возможно, конечно) и все данные берутся из кэшей клиента.
Как результат: сейчас в базе всего 73 квеста, где задействованы данные поля... 73 из 9456. И эти 73 квеста ни что иное как просто ещё не попавшиеся в кэшах.
Следовательно, любое использование данного поля - хак в базе. Раз любое - значит подобные поля попросту не нужны.
Да и силами EventAI сейчас всё очень неплохо решается, оставшееся - силами SD2.
0,77% квестов использующих данные поля - по-моему достаточно свидетельство об их ненадобности...
Ну и напоследок пример:
квест http://ru.wowhead.com/?quest=8346
Раньше как решались подобные задачи? ID маназмея в ReqCreatureOrGOId1, ID спелла в ReqSpellCast1. Но по данным с кэша в ReqCreatureOrGOId1 стоит http://ru.wowhead.com/npc=15468, а зачёт каста реализуется иным образом...
Надеюсь, общая идея ясна. Владимир, хотелось бы услышать ваши комментарии по данному поводу.

[ANTOMA]

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

[KiriX]

и зачем бежать впереди поезда?
в уже работающую схему, придумывать велосипед...
как по мне, реализуются оставшиеся квесты, килл поля и забыли.
Эмс... Что-то мне подсказывает, что они (эти оставшиеся квесты) реализуются правильным путём горааааздо быстрее, если поля килл сейчас...
Зачем оставлять место для хаков, когда с этим и в коде, и в базе всегда ведётся яростная борьба?

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

[ANTOMA]

аааа, так и напиши, хочу реализовать вариант..... пока не пнешь не полетит :).
В таком случаи абсолютно за, более того минус затраты на чтение дополнительных полей

[KiriX]

Владимир, можно всё-таки услышать ваш комментарий по этому поводу?

[Humort]

Да, нужно удалить эту таблицу, но перед этим, также, нужно реализовать, наконец-то, грамотно, все скрипты, чтобы действительно, не пришлось никому возвращаться к ней, а то, многие используют её, для того, чтобы реализовать квесты, к примеру, такие как http://ru.wowhead.com/quest=9685. Не смотря на то, что это является хаком.

То есть, как было сказано в посте №2, зачем торопиться? Думаю, надо сначала приложить усилия, чтобы освободить таблицу, а потом и её удалять.

[KiriX]

1) Приведённый в пример вами квест никак не должен быть - у него есть скрипт в СД2
2) В YTDB (596) к настоящему времени всего 58 кв. использующих это поле. И я вовсе не уверен, что данные кв работают правильно или работают вообще.
Проект так не любит хаки и оставляет такую дыру...
58 квестов. Если сейчас они реализованы через это глупое поле, значит через ЭвентАИ или скрипты их реализовать не составит труда. Но нахрен никто не пошевелится что-то менять, пока есть прекрасная возможность сделать это хаковым способом...

[Humort]

Абсолютно с вами солидарен, даже на счёт того, что тот приведённый мною пример есть в Sd2, притом, настолько корявый, что все палы орды, устали уже изощряться в ругательствах на этот счёт....
Как я заметил, всем лень, и никто не хочет ничего делать, из того, что действительно важно....
Каждый делает лишь то, чего им так хочется делать...
Ну неужели так трудно, целенаправленно перелопатить эти 58 квестов? А также, по дороге прихватить то, что до сих пор, не могут осилить Sd2?

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

[Vinolentus]

никто не хочет ничего делать, из того, что действительно важно....
Каждый делает лишь то, чего им так хочется делать...Если кто-то что-либо делает, то он считает, что именно это на данный момент наиболее важно/актуально/интересно/etc. Если у вас иные приоритеты - вы можете делать то, что _вы_ считаете более важным. Если не делаете - значит, не так оно и важно даже для вас.

[KiriX]

Vinolentus, убрать лишние поля из базы и их обработчик в коде - до минут 15. Я мог бы это сделать. Но сначала я хочу убедиться, что это будет принято. Если это нафиг никому не надо и всё прекрасно и так - я не стану тратить свои даже 15 минут...

[Vladimir]

В UDB используется в 150-ти. Как не будут использоваться или мало останется так и удалим. Борьба с хаками это хорошо, но без экстремизма. Пример с SD2 - да могу дропнуть хоть сейчас GetGUID() - и что - девелоперы SD2 просто уписаются от счастья поставленные перед фактомм наличия нескольких тысяч ошибок сборки.
И сколько времени после этого не будет собьираться SD2, а может девелоперы просто плюнут и уйдут.

[KiriX]

Однако где гарантии, что сейчас девелоперы UDB стараются сделать эти кв правильно? А вот если бы изменения были - им пришлось бы отрыть информацию, что же действительно должно быть в квесте.
Я уже и не хотел упоминать, что это поле используется в кв где, к примеру, нужно излечить 10 кого-то там. Да, с этим полем это легко реализовать. И сделать потом тоже легко 10 раз на одного НПС кастануть.
Да и кто будет задумываться как правильно надо, если жалоб разрабам о этих кв не поступает - они ведь работают! =)
В общем печально, что разработчики ядра избрали именно такую позицию =(

[Vladimir]

Ожидать что кто будет исправлять 150 квестов за раз не реально. И я думаю новых использований давно не добавлялось. Просто надо время чтобы исправить оставшиеся - я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь. Само по себе наличие полей не мешает ничему. Небудут нужны - удалим.

[YuruY]

Vladimir
А иногда полезно пнуть. Сколько времени можно ждать (больше года) дроп полей в "creature_movement" т.к. в "creature_movement_scripts" они дублируются? Там делов на час по "перегнать", по базе (могу помочь перегнать).

[Dragon]

я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь.
Можно предупредить сейчас и через некоторое время удалить, вариант?

[KiriX]

Shmoo, what you think about drop field ReqSpellCast???

[schmoozerd]

udb still has a bunch of quests with it.

But yes, this should be dropped :)

Edit: However it seems many of them can be dropped directly

[NeatElves]

+ http://www.ru-mangos.ru/showthread.php?t=5267 ))

[KiriX]

shmoo, and stop to look at UDB - they really slowly!!! =)))