Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2 bugs on BG EY #1380

Open
Thermaltake opened this issue Aug 16, 2013 · 8 comments
Open

2 bugs on BG EY #1380

Thermaltake opened this issue Aug 16, 2013 · 8 comments

Comments

@Thermaltake
Copy link

Суть. Первый баг. При захвате базы сквернобота альянсом, выводится не тот текст о захвате. Поправка https://gist.github.com/Thermaltake/6249235

Второй баг. После захвата этой же точки, нельзя поставить флаг на ней. Я долго копал, но так и не нашел, как поправить...
Короче, начнем с того, что https://github.com/mangosR2/mangos/blob/master/src/game/BattleGround/BattleGroundEY.h#L125 не правильное сейчас. На точке, на которой ставится флаг, триггер == 4515 .
Не верите? Встаньте прямо в этот маленький черный круг и .trig
Дальше.
https://github.com/mangosR2/mangos/blob/master/src/game/BattleGround/BattleGroundEY.cpp#L283

Ни при триггере 4515 ни при 4514 она не вызывается. Зато, на какой то хрен, вызывается при триггере 5866 который вообще непонятно зачем.
Вызывает https://github.com/mangosR2/mangos/blob/master/src/game/MiscHandler.cpp#L679 . Опять же, она на все триггеры точек на бг око бури срабатывает, даже 5866, но верный 4515 и неверный 4514, почему то не срабатывает. Я даже не нашел, откуда она должна вызываться, чтобы дальше искать, где проблема...

И эти баги, оказывается, есть и в cmangos .

И люди, играющие на других серверах, где вообще триня, тоже жалуются на второй баг с флагом.

@rsa
Copy link
Member

rsa commented Aug 16, 2013

хохма в том, что ареатриггера под скверноботом нет в ДБЦ, совсем. поэтому раньше он эмулировался хаком. сейчас хака нет (БГ полностью переделано) - нет и триггера... а 4515 это другой.

@Thermaltake
Copy link
Author

Вот, я тоже в dbc смотрел и не нашел. Предлагаю его захакать.
В само БГ это, в какую нибудь функцию запихать проверку на AreaTrigger и если она будет 4515(или 4514 все же оставить?), то вызывать HandleAreaTrigger . А в source что передавать? Игрока?

Но я так и не понял, где и как https://github.com/mangosR2/mangos/blob/master/src/game/MiscHandler.cpp#L679 вызывается? может как то проще все это сделать можно..

Или же как вы делали, на недостающие данные добавляли таблицы в базу, их заполнять и подгружать.. Но только из-за одного ареатриггера это делать, бред..

@boxa
Copy link
Member

boxa commented Aug 16, 2013

помню, что давно смотрел эту тему, сделал, чтобы засчитывался флаг:

enum EYPointsTrigger
...
// boxa
// bug in client/dbc AREATRIGGER_FEL_REAVER_RUINS_POINT = 4514, // also 4515
AREATRIGGER_FEL_REAVER_RUINS_POINT = 5866,
// boxa

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

с этим ареатриггером флаг засчитывается, но немного далековато от точки установки. но работает =)

в мейнстриме ядра этой правке делать нечего, но как затычка сойдёт.

скоро уже приеду с отдыха и продолжим работу (пишите баг репорты :)

@Thermaltake
Copy link
Author

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

@Thermaltake
Copy link
Author

а это https://gist.github.com/Thermaltake/6249235 все же можно сразу внести.

@Thermaltake
Copy link
Author

скоро уже приеду с отдыха и продолжим работу (пишите баг репорты :)

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

@rsa
Copy link
Member

rsa commented Aug 16, 2013

а это https://gist.github.com/Thermaltake/6249235 все же можно сразу внести.

а это кажется и был полухак, чтобы хотя бы БГ можно было закончить... надо глянуть.

@rsa
Copy link
Member

rsa commented Aug 16, 2013

rsa же до победного не верит

не беспокойтесь, я не верю ничему и никогда, пока сам лбом не стукнусь :) это нормально, издержки профессии...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants