[fix 9973] Уровень npc на бг Альтерак

[Insider42]

Недавно после установки патча на рандомные бг, игроки наконец таки стали ходить на бг Альтерак. Но тут же вскрылась ооочень старая проблема, которая появилась после внесения патча на это бг в Мангос. Т.к. для заселения сейчас полностью используется база (а в оригинальном патче было вшито в ядро) - мобы не подстраиваются под уровень игроков (а точнее диапазон уровней).

Собственно запрос заключается в том, чтобы реализовать такое:
1) бг запустилось для игроков 60-69 уровней - сервер считает что инст normal сложности и спавнит кричеров с их оригинальным entry
2) бг запустилось для игроков 70-79 уровней - сервер считает что инст 10 heroic и спавнит кричера с ид из поля difficulty_entry_1 этого кричера
3) бг запустилось для игроков 80-85 уровней - сервер считает что инст 25 normal и спавнит кричера с ид из поля difficulty_entry_2 этого кричера

Пусть даже это будет хак вроде
if (бг = альтерак && уровень бг = 70-79) creature->entry=difficulty_entry_1;
ибо эта проблема полностью убивает играбельность этого бг для игроков 70+ уровня. Мобы попросту не попадают по ним :/

[LordJZ]

А разве это сейчас не реализовано?

[Insider42]

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

А разве это сейчас не реализовано?

нет, стоит ядро 9797+база YTDB 543, в колонках difficulty_entry# заполнение присутствует, но на бг всегда мобы 60-64 уровней.

В том то и дело, что у бг не определяется сложность... или попросту не используется

[selector]

Дабы не создавать еще 1 тему. У Альтерака и Арати имеются проблемы с текстами. Например, когда игрок Альянса захватывает кузницу, то идет сообщение о том что Орда захватила стойла и т.д.

Проверено пол часа назад:
Rev. 9824 (clean)
YTDB 545
SD2 last(+ insider42 mods)

[LordJZ]

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

нет, стоит ядро 9797+база YTDB 543, в колонках difficulty_entry# заполнение присутствует, но на бг всегда мобы 60-64 уровней.

В том то и дело, что у бг не определяется сложность... или попросту не используется

А сама база-то правильно заселена? На 30-й карте стоят НПС, у которых есть копии в difficulty_entryN?

Я просто точно помню как спорил с balrok-ом и Vladimir-ом в комментариях к коммиту, по поводу реализации как раз этой фичи.

Хотя, возможно, что-то перестало работать (например, после свитча).

[Insider42]

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

А сама база-то правильно заселена? На 30-й карте стоят НПС, у которых есть копии в difficulty_entryN?

Возьмем за пример моба 11657
creature_template
Поля со сложностями 1 - 22685, 2 - 32045. Эти два кричера также присутствуют в таблице. Ошибок в загрузке сервера нет
В таблице creature моб 11657 записан, spawnmask=15 (все сложности)

Или же в случае БГ должны стоять нпс ВСЕХ сложностей в creature (тупо добавлены новые записи в creature с id из difficulty_entry этого кричера) и spawnmask должен быть для каждой сложности свой, дабы не спавнить их всех разом?

[LordJZ]

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

...
Или же в случае БГ должны стоять нпс ВСЕХ сложностей в creature (тупо добавлены новые записи в creature с id из difficulty_entry этого кричера) и spawnmask должен быть для каждой сложности свой, дабы не спавнить их всех разом?

Попробуйте. Если это так, то это - баг.

[Insider42]

Провел эксперименты.

Эксперимент №1. Поставил копию этого нпс с ID героического моба через запрос - затем проварьировал spawnmask в базе (2,4,8 - ид всех, кроме первой сложности) - моб не появлялся, поставил spawnmask=1 - появился, с 15 тоже появляется
Эксперимент №2. Зашёл как обычно своим гмом 80 уровня на бг и тупо воткнул моба, смотрю в базу - у него spawnmask=1.
Вывод - бг не знает ни о каких сложностях. Будто обычная локация

Без вмешательства в ядро проблему не решить

[PSZ]

До перехода на 3.3 это 100% работало, у меня сейчас на сервере стоит 9119 - разные уровни мобов для разных брекетов работают. Проверил на последней ревизии - действительно сломалось...

По-моему это после какого-то коммита отвалилось.

Меня вот этот коммит смущает - [9597] http://github.com/mangos/mangos/comm...ab869bae45fada

[Insider42]

АП.
в функции
BattleGroundMap* MapInstanced::CreateBattleGroundMap(uint32 InstanceId, BattleGround* bg)

uint8 spawnMode = bracketEntry ? bracketEntry->difficulty : REGULAR_DIFFICULTY;
всегда равен 0, т.е. REGULAR_DIFFICULTY
вот она и проблема. Теперь осталось найти - почему?

[PSZ]

bracketEntry равен NULL?

Сорри, нет возможности посмотреть самому.

[zergtmn]

diff --git a/src/game/BattleGroundMgr.cpp b/src/game/BattleGroundMgr.cpp
index fb36219..dfa7302 100644
--- a/src/game/BattleGroundMgr.cpp
+++ b/src/game/BattleGroundMgr.cpp
@@ -1535,6 +1535,8 @@ BattleGround * BattleGroundMgr::CreateNewBattleGround(BattleGroundTypeId bgTypeI
             return 0;
     }
 
+    bg->SetBracket(bracketEntry);
+
     // will also set m_bgMap, instanceid
     sMapMgr.CreateBgMap(bg->GetMapId(), bg);

Ниже уже есть эта строчка, наверное ее можно удалить (после более детального изучения проблемы).

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

diff --git a/src/game/BattleGroundMgr.cpp b/src/game/BattleGroundMgr.cpp
index fb36219..dfa7302 100644
--- a/src/game/BattleGroundMgr.cpp
+++ b/src/game/BattleGroundMgr.cpp
@@ -1535,6 +1535,8 @@ BattleGround * BattleGroundMgr::CreateNewBattleGround(BattleGroundTypeId bgTypeI
             return 0;
     }
 
+    bg->SetBracket(bracketEntry);
+
     // will also set m_bgMap, instanceid
     sMapMgr.CreateBgMap(bg->GetMapId(), bg);

Ниже уже есть эта строчка, наверное ее можно удалить (после более детального изучения проблемы).

Проверил, работает! Зашел на Альтерак на 80-85 лвл, уровень мобов был 81!

diff --git a/src/game/BattleGroundMgr.cpp b/src/game/BattleGroundMgr.cpp
index 9236742..d904821 100644
--- a/src/game/BattleGroundMgr.cpp
+++ b/src/game/BattleGroundMgr.cpp
@@ -1535,6 +1535,8 @@ BattleGround * BattleGroundMgr::CreateNewBattleGround(BattleGroundTypeId bgTypeI
             return 0;
     }
 
+    bg->SetBracket(bracketEntry);
+
     // will also set m_bgMap, instanceid
     sMapMgr.CreateBgMap(bg->GetMapId(), bg);
 
@@ -1545,7 +1547,6 @@ BattleGround * BattleGroundMgr::CreateNewBattleGround(BattleGroundTypeId bgTypeI
 
     // start the joining of the bg
     bg->SetStatus(STATUS_WAIT_JOIN);
-    bg->SetBracket(bracketEntry);
     bg->SetArenaType(arenaType);
     bg->SetRated(isRated);

[Insider42]

Да, ваш фикс судя по всему подходит...

[LordJZ]

Внесли: [9973]

[YuruY]

Наконецто.

[Vladimir]

Не внесли а написали свой... ветку не читал до поста.

[YuruY]

Главное результат ... есть.

[Fedia22]

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

Собственно запрос заключается в том, чтобы реализовать такое:
1) бг запустилось для игроков 60-69 уровней - сервер считает что инст normal сложности и спавнит кричеров с их оригинальным entry
2) бг запустилось для игроков 70-79 уровней - сервер считает что инст 10 heroic и спавнит кричера с ид из поля difficulty_entry_1 этого кричера
3) бг запустилось для игроков 80-85 уровней - сервер считает что инст 25 normal и спавнит кричера с ид из поля difficulty_entry_2 этого кричера

не видел тут 50-59 лвлы

[YuruY]

Это образно.

[Insider42]

Просто смутило что там только заполнение для entry (как же без него),difficulty_entry_1 и difficulty_entry_2. Посчитал с конца.. вот и вышло. Видимо 70-79 нету, там 70-85

В любом случае это не играет большой роли. Проблема уже решена

А ещё у меня предположения что тут ещё и косяк БД (у меня YTDB 547) - нету заполнения difficulty_entry_3. В dbc 4 сложности (0,1,2,3)

[zergtmn]

В DBC:

1,30,0,51,60,0,
2,30,1,61,70,1,
3,30,2,71,79,2,
4,30,3,80,85,3,

[virusav]

Официально максимальный уровень игроков 80.

[Vladimir]

близы просто с запасом сделали так как проще единый шаг использовать по 5.
С точки зрения мангосам последний bracket не ограничен по максимуму