Карта 0 (вост. королевства) не получает script_id

[rsa]

Сабж собственно. Причем скрипт грузится, но в структуре WorldTemplate для карты 0 script_id всегда = 0. Убедиться можно при помощи патча (это не фикс! просто иллюстрация), скрипт естественно должен быть присвоен:

diff --git a/src/game/Map.cpp b/src/game/Map.cpp
index ed9a7e7..5528a13 100644
--- a/src/game/Map.cpp
+++ b/src/game/Map.cpp
@@ -1155,8 +1155,22 @@ void Map::CreateInstanceData(bool load)
     {
         if (WorldTemplate const* mInstance = ObjectMgr::GetWorldTemplate(GetId()))
             i_script_id = mInstance->script_id;
+        DEBUG_LOG("Loading instance data for WorldMap: %u script: %u ", GetId(), i_script_id);
+        if (!i_script_id)
+        {
+            QueryResult* result = WorldDatabase.PQuery("SELECT ScriptName FROM world_template WHERE map = '%u'", GetId());
+            if (result)
+            {
+                Field* fields = result->Fetch();
+                const char* scriptname = fields[0].GetString();
+                i_script_id = sScriptMgr.GetScriptId(scriptname);
+                delete result;
+            }
+        }
+        DEBUG_LOG("Loading instance data for WorldMap: %u script: %u ", GetId(), i_script_id);
     }
 
+
     if (!i_script_id)
         return;

поиски источника бага привели в район преобразования форматов в SQLStorage и тут я сломался (насколько же ... хм... запутанная система присвоения script_id в мангосе однако. непонятно только зачем она вообще, все равно все время ищем по имени...)

[KiriX]

И в чём проблема? Для карты 0 пишем скрипт с номером 0

[rsa]

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

И в чём проблема? Для карты 0 пишем скрипт с номером 0

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

[wk23]

сделать по умолчанию -1, а не 0

diff --git a/src/game/Map.cpp b/src/game/Map.cpp
index fed3ab2..0dc52ae 100644
--- a/src/game/Map.cpp
+++ b/src/game/Map.cpp
@@ -74,7 +74,7 @@ Map::Map(uint32 id, time_t expiry, uint32 InstanceId, uint8 SpawnMode)
   m_VisibleDistance(DEFAULT_VISIBILITY_DISTANCE), m_persistentState(NULL),
   m_activeNonPlayersIter(m_activeNonPlayers.end()),
   i_gridExpiry(expiry), m_TerrainData(sTerrainMgr.LoadTerrain(id)),
-  i_data(NULL), i_script_id(0)
+  i_data(NULL), i_script_id(-1)
 {
     m_CreatureGuids.Set(sObjectMgr.GetFirstTemporaryCreatureLowGuid());
     m_GameObjectGuids.Set(sObjectMgr.GetFirstTemporaryGameObjectLowGuid());
diff --git a/src/game/ScriptMgr.cpp b/src/game/ScriptMgr.cpp
index 08a92b7..30a792f 100644
--- a/src/game/ScriptMgr.cpp
+++ b/src/game/ScriptMgr.cpp
@@ -942,13 +942,13 @@ uint32 ScriptMgr::GetScriptId(const char *name) const
     // use binary search to find the script name in the sorted vector
     // assume "" is the first element
     if (!name)
-        return 0;
+        return 1;
 
     ScriptNameMap::const_iterator itr =
         std::lower_bound(m_scriptNames.begin(), m_scriptNames.end(), name);
 
     if (itr == m_scriptNames.end() || *itr != name)
-        return 0;
+        return -1;
 
     return uint32(itr - m_scriptNames.begin());
 }

[rsa]

я не зря привел пример для просмотра. реальный номер скрипта у меня 1792 а вовсе не 0. он не прописывается в структуру WorldTemlate, а 0 значение работает правильно.

[SilverIce]

все загружается правильно
но SQLStorage LookupEntry возвращает null при нулевом entry.. споткнулись

[rsa]

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

все загружается правильно
но SQLStorage LookupEntry возвращает null при нулевом entry.. споткнулись

something like this I expected. Thanks.

[rsa]

Поскольку решение проблемы судя по всему никому не интересно, предлагаю собственное -
https://github.com/mangosR2/mangos/c...f66b38b4a5c61d
пока судя по всему ничего не испортилось. Глядя по коду тоже возможных проблем не нашел. Кто-то вообще может сказать на кой черт это исключение было изначально?

[zergtmn]

Потому что 0 используется в коде как пустой guid/entry.

// хотим получить nullptr при m_castItem = 0
if(ItemPrototype const* itemInfo = sItemStorage.LookupEntry(m_castItem))
    ...

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

[rsa]

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

Потому что 0 используется в коде как пустой guid/entry.

// хотим получить nullptr при m_castItem = 0
if(ItemPrototype const* itemInfo = sItemStorage.LookupEntry(m_castItem))
    ...

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

спасибо за наводку. но по моему такие случаи нужно отслеживать в коде обработчиков, а не делать хак в storage (который должен быть туп как пробка). и прототип в этом случае все равно NULL если итема 0 нет в базе.

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