[mod] Vehicles

[Gen1us2k]

патч реализует работу транспортных средств в игре. на английском vehicles

автор: Много авторов, первыми были NetSky, Rastikzzz, потом подхватил Wojta
Форк на гитхабе: http://github.com/Tasssadar/Valhalla.../tree/vehicle/ // иногда автор забрасывает дело
Оригинальная тема на getmangos.com: http://getmangos.com/community/showthread.php?11759-[dev]Vehicles

как получить?

git checkout -b vehicle
git pull git://github.com/Tasssadar/Valhalla-Project.git vehicle
git diff -p master vehicle > vehicle.patch

либо слив аттач и

patch -p1 -l < vehicle_9542.patch

Вариант основанный на порте с трините от zergtmn:

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

Вот патч, который реализует многоместные маунты: http://paste2.org/p/880155
Бранч: http://github.com/zergtmn/mangos/com...ltiseat_mounts
В базу:

ALTER TABLE creature_template
  ADD COLUMN `VehicleId` mediumint(8) unsigned NOT NULL default '0' AFTER `PetSpellDataId`;

VehicleId из снифов: http://paste2.org/p/880137, тут не только маунты.
Теоретически на них можно сажать кричеров.

[timmit]

Ну вы сделайте, выложите, а по ходу думаю уже мысли и как лучше и что изменить скажут

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

ну вот как то

это только часть кода.....
enum VehicleFLags
{
    VF_NONE							= 0x000000000,
    VF_UNK1		                        = 0x000000001,                 // 
    VF_UNK2			            = 0x000000002,                 // 
    VF_UNK3				 = 0x000000004, 	   //
    VF_UNK4				 = 0x000000008,                 // FOR VEHICLE_ID = 34
    VF_UNK5			            = 0x000000010,                 // 
    VF_UNK6				= 0x000000020,                 // FOR VEHICLE_ID = 186
    VF_UNK7				= 0x000000040,                 // 
    VF_UNK8				 = 0x000000080,                 // 
    VF_UNK9		                      = 0x000000100,                 // VF_FLYING
    VF_UNK10	                                  = 0x000000200,                 // 
    VF_UNK11				= 0x000000400,                 // 
    VF_UNK12			            = 0x000000800,		   //
    VF_UNK13				= 0x000001000,                // 
    VF_UNK14				= 0x000002000,                // 
    VF_UNK15				= 0x000004000,                // FOR VEHICLE_ID = 264
    VF_UNK16			            = 0x000008000,		   // FOR VEHICLE_ID = 295
    VF_UNK17			            = 0x000010000,		   //
    VF_UNK18			            = 0x000020000,		   // FOR VEHICLE_ID = 218
    VF_UNK19			            = 0x000040000,		   //
    VF_DESPAWN_AT_LEAVE	            = 0x000080000,		   // FOR VEHICLE_ID = 17
    VF_UNK21			            = 0x000100000,		   //
    VF_UNK22			            = 0x000200000,		   //
    VF_UNK23			            = 0x000400000,		   // FOR VEHICLE_ID = 210 chopper
    VF_UNK24			            = 0x000800000,		   //
    VF_UNK25			            = 0x001000000,		   //
    VF_UNK26			            = 0x002000000,		   //
    VF_UNK27			            = 0x004000000,		   //
    VF_UNK28			            = 0x008000000,		   //
    VF_UNK29			            = 0x010000000,		   //
    VF_UNK30			            = 0x020000000,		   //
    VF_UNK31			            = 0x040000000,		   // FOR VEHICLE_ID = 328
    VF_UNK32				= 0x080000000
};

о значении флагов можно только догадываться
сейчас я занят отделением хаковых решений от не хаковых и флагами некада заниматься

Да, тоже заглянул в эти флаги и просто ужаснулся! ни чего не понятно, что куда и зачем. Даже сделал для себя табличку с vehId и флагами, плодов не принесло!
Единствееное что я могу предположить что VF_UNK24 - это запрет на перемещения npc (пушки и всякое такое, но уверен лишь на 3%)! Да и к томуже точных данных та нету у кого какой vehId и т.д. так что и с флагами будет проблематично!

Кому интересно можете глянуть вложение(Excel таблица (2007) лист 1 - это то, что известно правда может и коряво, а 2 это все сразу), может у вас идеи появятся! Да и вообще как можно разобраться в флагах?

[lucky]

Сообщение от Rage Hunter

Да, тоже заглянул в эти флаги и просто ужаснулся! ни чего не понятно, что куда и зачем. Даже сделал для себя табличку с vehId и флагами, плодов не принесло!
Единствееное что я могу предположить что VF_UNK24 - это запрет на перемещения npc (пушки и всякое такое, но уверен лишь на 3%)! Да и к томуже точных данных та нету у кого какой vehId и т.д. так что и с флагами будет проблематично!

Кому интересно можете глянуть вложение(Excel таблица (2007)), может у вас идеи появятся! Да и вообще как можно разобраться в флагах?

Возможно я чего то не понял, но о флагах реализованных в данном патче можно прочитать здесь в коментах http://ru-mangos.ru/showpost.php?p=2086&postcount=25 src\game\SharedDefines.h
Если это не то, тогда извиняюсь

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

Возможно я чего то не понял, но о флагах реализованных в данном патче можно прочитать здесь в коментах http://ru-mangos.ru/showpost.php?p=2086&postcount=25 src\game\SharedDefines.h
Если это не то, тогда извиняюсь

Я говорю не про хак флаги которые были добавлены от болды, а о флагах соодержащихся в dbc файлах!

Кстати Пакет SMSG_PET_SPELLS в void Vehicle::BuildVehicleActionBar(Player *plr)
там есть такая строка

data << uint16(m_VehicleData ? m_VehicleData->v_spells[i] : NULL) << uint8(0) << uint8(i+8);

так вот на сколько мне известно мак значение uint16 = ~65000
сейчас же спелы есть с id выше 65К
ее стоит заменить на

data << uint32(MAKE_UNIT_ACTION_BUTTON(m_VehicleData ? m_VehicleData->v_spells[i] : NULL,i+8));

на сколько это правильно? может как-то по другому?

[Konctantin]

а размер поля в пакете?

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

а размер поля в пакете?

??????????????????
стоит же WorldPacket data(SMSG_PET_SPELLS, 8+2+4+4+4*10+1+1);
и так 4*10

[Кот ДаWINчи]

смотрю данный мод. В Сишники не сильно силен.
Но вижу в патче вот такую конструкцию:

diff --git a/src/game/GameObject.cpp b/src/game/GameObject.cpp
index c6a4a00..3aa1a05 100644
--- a/src/game/GameObject.cpp
+++ b/src/game/GameObject.cpp
@@ -418,11 +418,7 @@ void GameObject::Update(uint32 /*p_time*/)
if(!m_respawnDelayTime)
return;

- if(!m_spawnedByDefault)
- {
- m_respawnTime = 0;
- return;
- }
+ m_respawnTime = m_spawnedByDefault ? time(NULL) + m_respawnDelayTime : 0;

// since pool system can fail to roll unspawned object, this one can remain spawned, so must set respawn nevertheless
m_respawnTime = time(NULL) + m_respawnDelayTime;

Разве красная строка не перекрывает вычисления сделанные в зеленой?

Вообще-то там вот так, не знаю где вы это нашли

diff --git a/src/game/GameObject.cpp b/src/game/GameObject.cpp
index b4d7507..a8f82e8 100644
--- a/src/game/GameObject.cpp
+++ b/src/game/GameObject.cpp
@@ -421,14 +421,8 @@ void GameObject::Update(uint32 /*p_time*/)
             if(!m_respawnDelayTime)
                 return;
 
-            if(!m_spawnedByDefault)
-            {
-                m_respawnTime = 0;
-                return;
-            }
-
             // since pool system can fail to roll unspawned object, this one can remain spawned, so must set respawn nevertheless
-            m_respawnTime = time(NULL) + m_respawnDelayTime;
+            m_respawnTime = m_spawnedByDefault ? time(NULL) + m_respawnDelayTime : 0;
 
             // if option not set then object will be saved at grid unload
             if(sWorld.getConfig(CONFIG_BOOL_SAVE_RESPAWN_TIME_IMMEDIATLY))

[PSZ]

Неа ). В ветке Тасссадара, из которой делают патчи так и есть.

            m_respawnTime = m_spawnedByDefault ? time(NULL) + m_respawnDelayTime : 0;

            // since pool system can fail to roll unspawned object, this one can remain spawned, so must set respawn nevertheless
            m_respawnTime = time(NULL) + m_respawnDelayTime;

[Кот ДаWINчи]

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

Неа ). В ветке Тасссадара, из которой делают патчи так и есть.

            m_respawnTime = m_spawnedByDefault ? time(NULL) + m_respawnDelayTime : 0;

            // since pool system can fail to roll unspawned object, this one can remain spawned, so must set respawn nevertheless
            m_respawnTime = time(NULL) + m_respawnDelayTime;

но смысла тогда не вижу. m_respawnTime должно вычисляться один раз.

[NeatElves]

А тут не смысл, тут руки автора ветки при накате патча Лютика на респ сундуков.)

[YuruY]

Пока двери/сундуки и порталлы, онии у себя не ревлизовали..

Тут вопросик назрел! Что вообще из себя вехиклы должны предстовлять? creature? или все же отдельный класс?

[KiriX]

Сообщение от Rage Hunter

Тут вопросик назрел! Что вообще из себя вехиклы должны предстовлять? creature? или все же отдельный класс?

Ну на то они и вехиклы, что отдельный класс, а то назывались бы по-другому.
Более всего похожи на пэтов. Сначала это кричер самый обыкновенный. При клике на него кастуется спелл, который вызывает vehicle, который более всего похож на пэта.

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

Ну на то они и вехиклы, что отдельный класс, а то назывались бы по-другому.

Ха, тогда другой вопрос! Пример петы, npс, go! Для всех в базу существует отдельные таблицы, по вашему отпределенинию раз вехиклы отдельный класс стоит и для них создавать отдельные таблицы? vehicle_template, vehiсle и т.д. (ведь по сути они должны появляться уже как vehicle в мире, а не сумонится).

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

При клике на него кастуется спелл, который вызывает vehicle, который более всего похож на пэта.

я на офе такого не видел, клик на него и поехал, ни чего там не вызывается!

[timmit]

Вы видать никогда не использовали аддон от вовхеда? там прекрасно видно всегда, что вехикл это другой нпс, и номер у него другой, хоть вам и кажется, что один и тот же

[KiriX]

Сообщение от Rage Hunter

1) Ха, тогда другой вопрос! Пример петы, npс, go! Для всех в базу существует отдельные таблицы, по вашему отпределенинию раз вехиклы отдельный класс стоит и для них создавать отдельные таблицы? vehicle_template, vehiсle и т.д. (ведь по сути они должны появляться уже как vehicle в мире, а не сумонится).

2) я на офе такого не видел, клик на него и поехал, ни чего там не вызывается!

1) Так и есть в данном патче. Но на самом деле 90% данных, что в таблицах - можно вытянуть из ДБЦ. Но без некоторых правок БД всё же не обойтись - требуется указывать кто какой именно вехикл (вызывает! - см. код и спеллы вехиклов).
2) Ну вам уже ответили... Может, вы и имеете опыт игры на оффе, но ваша подпись весьма красноречиво свидетельствует о том, что спорить с тем, что вам говорят более опытные люди, с вашей стороны весьма глупо...

тиммит, о каком именно аддоне от ВХ идёт речь?

[Neggod]

http://ru.wowhead.com/client

[timmit]

Ссылку дали выше.
Хороший такой аддон, собирают всю информацию, показывает номера Го,нпс. Разговоры по квестам, диалоги некоторые собирает для отправки на вовхед. Именно с ним очень хорошо было да и есть работать с фазами.

[KiriX]

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

Ссылку дали выше.
Хороший такой аддон, собирают всю информацию, показывает номера Го,нпс. Разговоры по квестам, диалоги некоторые собирает для отправки на вовхед. Именно с ним очень хорошо было да и есть работать с фазами.

А, вы про этот. Им я пользуюсь, скоро скину в базу всё, что насобирал. Уже 3,5Мб кэша и ВХлутер заполнен тоже должен быть неплохо. С нуля до 25 левела человек-чернокнижник.

[timmit]

/wl id дает показывает идшники

Принцип такой:
1) стоит обычный моб с флагом UNIT_NPC_FLAG_SPELLCLICK
2) При клике на моба, кастуется спелл с эффектом SPELL_EFFECT_SUMMON и типом SUMMON_PROP_GROUP_VEHICLE
3) Суммонится моб класса Vehicle
В этом плане патч корректен.

[xmolex]

Кто-нибудь знает, кто-нибудь из офф девов работает над реализацией вехиклов?

[KiriX]

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

Кто-нибудь знает, кто-нибудь из офф девов работает над реализацией вехиклов?

Насколько мне известно - никто.
И этот же самый никто даже не изъявляет желания заниматься этим...
Думаю, всё в руках юзеров. Было уже сказано, что патч из этой темы в данном виде даже рассматриваться не будет, пока все возможные данные не будут браться из ДБЦ, а не переносится оттуда в БД.
Так что надо взяться за переписку.

[xmolex]

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

Было уже сказано, что патч из этой темы в данном виде даже рассматриваться не будет, пока все возможные данные не будут браться из ДБЦ

Это я понимаю, просто если кто-то уже работает, то толку браться тоже нет, т.к. всеравно свое будут доделывать. А с другой стороны, можно связаться и узнать чем помочь можно, вдруг затык на чем-нибудь.

[KiriX]

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

Это я понимаю, просто если кто-то уже работает, то толку браться тоже нет, т.к. всеравно свое будут доделывать. А с другой стороны, можно связаться и узнать чем помочь можно, вдруг затык на чем-нибудь.

Первая часть сообщения - по моим данным никто не работает. Хотя когда-то, как я помню, один перспективный человечек занимался...
Так что НАДО браться самим и что-то делать. Кстати, в триньке очень много берётся из ДБЦ - можно подглядеть их реализацию.
Можно связаться с главным разрабом Валгхаллы и подтолкунть его к этим же мыслям - ему проще, он этот патч уже должен знать вдоль и поперёк.

[madbroths]

Собственно патч под реву 10003 (он на самом деле под 9983) но накатывается без ошибок.

Переделывал не я.

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

Собственно патч под реву 10003 (он на самом деле под 9983) но накатывается без ошибок.

Переделывал не я.

а что собственно переделано?

[madbroths]

Ничего. просто подогнано под ревизию

[tempura]

что подогнано, спрашивают?

Йоу, бро, твая савсем нипанимай вапросав? Каммон?

[DiffuSer]

Переведите, пожалуйста, на русский язык, что предыдущий оратор сказал?

[anvd24]

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

Переведите, пожалуйста, на русский язык, что предыдущий оратор сказал?

Возможно уважаемый имел ввиду вопрос, что конкретно вы изменяли в патче, если изменяли что-либо.
Какую работу проделали,если таковая имела место быть.
Что именно было добавлено или исправлено,если конечно было.

И если ничего кроме как изменения имени файла исправлено не было с вашей стороны,то что вы имели ввиду под словами "переделано не мною", а кем?Есть информация?
Возникает резонный вопрос,где вы это взяли,и кто автор.
Если данная информация отсутствует,то это совсем другое дело.

Возможно я ошибаюсь, но скорее всего именно эти вопросы волнуют общественность.

[tempura]

Благодарю вас, о уважаемый и почтеннейший anvd24!

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

кто знает как написать в гите сразу 3 строчки

git clone git://github.com/mangos/mangos.git
cd mangos
git pull git://github.com/Tasssadar/Valhalla-Project.git vehicle

вставь в командную строку (Меню>Изменить>Вставить)

[tempura]

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

кто знает как написать в гите сразу 3 строчки

git clone git://github.com/mangos/mangos.git
cd mangos
git pull git://github.com/Tasssadar/Valhalla-Project.git vehicle

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

вставь в командную строку (Меню>Изменить>Вставить)

Вообще-то - по очереди. "вставить три сразу" - это что-то из немецких фильмов, совершенно определенного содержания.

В Git Bash можно так

git clone git://github.com/mangos/mangos.git && \
cd mangos && \
git pull git://github.com/Tasssadar/Valhalla-Project

[KiriX]

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

Вообще-то - по очереди. "вставить три сразу" - это что-то из немецких фильмов, совершенно определенного содержания.

Сразу вспомнилось: Файл "Жена" -> контекстное меню -> "Изменить"

[rsa]

Эх... У Тассадара лучше патч больше не брать. Судя по плюхам в коде он его поддерживает из последних сил, уж больно сильно изменилось ядро с тех пор как эти вехиклы были сделаны... Еще пара резких смен чего-либо и все.