|
|
|
|||||||
| Патчи Если кто-то хочет выложить не свой готовый патч - не забудьте указать автора и источник.
Если кто-то хочет задать вопрос по патчу - велкам. |
 |
|
|
Опции темы | Поиск в этой теме | Опции просмотра |
30.11.2010, 22:04
|
#1 |
|
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
 |
ReqSpellCast [quest_template]
Давно уже думаю об этом, и вот, кажется, настала пора написать это.
Эти поля не нужны. База 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, а зачёт каста реализуется иным образом... Надеюсь, общая идея ясна. Владимир, хотелось бы услышать ваши комментарии по данному поводу. |
|
|
|
01.12.2010, 15:56
|
#2 |
|
Новичок
Регистрация: 07.03.2010
Сообщений: 11
Сказал(а) спасибо: 126
Поблагодарили 0 раз(а) в 0 сообщениях
|
и зачем бежать впереди поезда?
в уже работающую схему, придумывать велосипед... как по мне, реализуются оставшиеся квесты, килл поля и забыли. |
|
|
|
|
01.12.2010, 20:49
|
#3 | |
|
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Цитата:
Зачем оставлять место для хаков, когда с этим и в коде, и в базе всегда ведётся яростная борьба? Господа модераторы, нехорошо редактировать чужие сообщения без малейшего комментария... |
|
|
|
|
|
03.12.2010, 14:59
|
#4 |
|
Новичок
Регистрация: 07.03.2010
Сообщений: 11
Сказал(а) спасибо: 126
Поблагодарили 0 раз(а) в 0 сообщениях
|
аааа, так и напиши, хочу реализовать вариант..... пока не пнешь не полетит
 .В таком случаи абсолютно за, более того минус затраты на чтение дополнительных полей |
|
|
|
|
18.05.2011, 23:36
|
#6 |
|
Гость
Сообщений: n/a
|
Да, нужно удалить эту таблицу, но перед этим, также, нужно реализовать, наконец-то, грамотно, все скрипты, чтобы действительно, не пришлось никому возвращаться к ней, а то, многие используют её, для того, чтобы реализовать квесты, к примеру, такие как http://ru.wowhead.com/quest=9685. Не смотря на то, что это является хаком.
То есть, как было сказано в посте №2, зачем торопиться? Думаю, надо сначала приложить усилия, чтобы освободить таблицу, а потом и её удалять. |
|
|
|
19.05.2011, 07:06
|
#7 |
|
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
1) Приведённый в пример вами квест никак не должен быть - у него есть скрипт в СД2
2) В YTDB (596) к настоящему времени всего 58 кв. использующих это поле. И я вовсе не уверен, что данные кв работают правильно или работают вообще. Проект так не любит хаки и оставляет такую дыру... 58 квестов. Если сейчас они реализованы через это глупое поле, значит через ЭвентАИ или скрипты их реализовать не составит труда. Но нахрен никто не пошевелится что-то менять, пока есть прекрасная возможность сделать это хаковым способом... |
|
|
|
|
20.05.2011, 00:32
|
#8 |
|
Гость
Сообщений: n/a
|
Абсолютно с вами солидарен, даже на счёт того, что тот приведённый мною пример есть в Sd2, притом, настолько корявый, что все палы орды, устали уже изощряться в ругательствах на этот счёт....
Как я заметил, всем лень, и никто не хочет ничего делать, из того, что действительно важно.... Каждый делает лишь то, чего им так хочется делать... Ну неужели так трудно, целенаправленно перелопатить эти 58 квестов? А также, по дороге прихватить то, что до сих пор, не могут осилить Sd2? P.S. Не буду упоминать целую стопку других, не менее важных проблем, которыми никто не хочет заниматься..... |
|
|
|
20.05.2011, 04:17
|
#9 |
|
Пользователь
Регистрация: 23.03.2010
Сообщений: 51
Сказал(а) спасибо: 14
Поблагодарили 32 раз(а) в 15 сообщениях
|
|
|
|
|
|
20.05.2011, 07:47
|
#10 |
|
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Vinolentus, убрать лишние поля из базы и их обработчик в коде - до минут 15. Я мог бы это сделать. Но сначала я хочу убедиться, что это будет принято. Если это нафиг никому не надо и всё прекрасно и так - я не стану тратить свои даже 15 минут...
|
|
|
|
|
20.05.2011, 22:55
|
#11 |
|
MaNGOS Dev
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
 |
В UDB используется в 150-ти. Как не будут использоваться или мало останется так и удалим. Борьба с хаками это хорошо, но без экстремизма. Пример с SD2 - да могу дропнуть хоть сейчас GetGUID() - и что - девелоперы SD2 просто уписаются от счастья поставленные перед фактомм наличия нескольких тысяч ошибок сборки.
И сколько времени после этого не будет собьираться SD2, а может девелоперы просто плюнут и уйдут.
__________________
Так как устал объяснять знайте ICQ не пользуюсь |
|
|
|
|
21.05.2011, 08:22
|
#12 |
|
Умный
Старожил
Регистрация: 06.03.2010
Сообщений: 886
Сказал(а) спасибо: 698
Поблагодарили 433 раз(а) в 181 сообщениях
Записей в дневнике: 4
|
Однако где гарантии, что сейчас девелоперы UDB стараются сделать эти кв правильно? А вот если бы изменения были - им пришлось бы отрыть информацию, что же действительно должно быть в квесте.
Я уже и не хотел упоминать, что это поле используется в кв где, к примеру, нужно излечить 10 кого-то там. Да, с этим полем это легко реализовать. И сделать потом тоже легко 10 раз на одного НПС кастануть. Да и кто будет задумываться как правильно надо, если жалоб разрабам о этих кв не поступает - они ведь работают!  В общем печально, что разработчики ядра избрали именно такую позицию 
|
|
|
|
|
21.05.2011, 08:35
|
#13 |
|
MaNGOS Dev
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
|
Ожидать что кто будет исправлять 150 квестов за раз не реально. И я думаю новых использований давно не добавлялось. Просто надо время чтобы исправить оставшиеся - я лично устраивать нездоровые сенсации и ставить девелоперов UDB пред фактом гемороя не собираюсь. Само по себе наличие полей не мешает ничему. Небудут нужны - удалим.
__________________
Так как устал объяснять знайте ICQ не пользуюсь |
|
|
|
|
21.05.2011, 21:58
|
#14 |
|
YTDB Dev
Регистрация: 01.02.2010
Сообщений: 288
Сказал(а) спасибо: 125
Поблагодарили 97 раз(а) в 53 сообщениях
|
Vladimir
А иногда полезно пнуть. Сколько времени можно ждать (больше года) дроп полей в "creature_movement" т.к. в "creature_movement_scripts" они дублируются? Там делов на час по "перегнать", по базе (могу помочь перегнать). |
|
|
|
|
23.05.2011, 14:21
|
#15 | |
|
Новичок
Регистрация: 07.03.2010
Сообщений: 22
Сказал(а) спасибо: 33
Поблагодарили 2 раз(а) в 2 сообщениях
|
Цитата:
|
|
|
|
|
|
11.09.2012, 13:00
|
#17 |
|
MaNGOS Dev
Регистрация: 17.11.2011
Сообщений: 99
Сказал(а) спасибо: 35
Поблагодарили 80 раз(а) в 26 сообщениях
|
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 Последний раз редактировалось schmoozerd; 11.09.2012 в 13:13. |
|
|
|
| Пользователь сказал cпасибо: | KiriX (11.09.2012) |
|
|
|
Похожие темы
|
||||
| Тема | Автор | Раздел | Ответов | Последнее сообщение |
| [9312] Implement DB storage of new quest_template field RewHonorMultiplier | KiriX | CMaNGOS Commits | 18 | 31.10.2010 15:48 |
| [10657] Separate quest_template.QuestFlags from SpecialFlags | newsbot | CMaNGOS Commits | 0 | 30.10.2010 02:17 |
| [10621] Add new field RequiredClasses for quest_template | newsbot | CMaNGOS Commits | 0 | 19.10.2010 01:42 |
| [9852] Disable forced alteration of specialFlags in quest_template at startup. | newsbot | CMaNGOS Commits | 3 | 11.05.2010 08:40 |
При правильном поведении пользователей я вообще добрый и пушистый, как на аватарке. 
Линейный вид
