Локальные guid
 

На карту а со командой ставится Creature/Vehicle/Gameobject. На карту б командой ставится тот же тип. Если счетчик guidLow для этого типа совпадает у карты а и карты б - выходит большая задница, от перезаписи GameObject/Creature data до (если на какой-то из карт объект с таким гуид удаляют) оставшегося вообще без нее объекта на другой карте.
Странно, что никто не заметил до сих пор. О_о

Вы в этом уверены? Вроде как гуиды локальны для каждой карты и в глобальных контейнерах мы подобные объекты не держим. Поэтому удаление объекта 1 с карты А ни коим образом не влияет на объект В с таким же гуидом на карте 2. Так по крайней мере задумывалась сея феерическая система :)

Если они временные. Посмотрите на mGameObjectDataMap и mCreatureDataMap. Они глобальны в той степени, в которой могут быть глобальны переменные одиночки-ObjectMgr'a. Ключ - guid %)
Или, например, сюда:

Код:

---

void GameObject::SaveToDB(uint32 mapid, uint8 spawnMask, uint32 phaseMask)
{
    const GameObjectInfo *goI = GetGOInfo();

    if (!goI)
        return;

    // update in loaded data (changing data only in this place)
    GameObjectData& data = sObjectMgr.NewGOData(GetGUIDLow());

    // data->guid = guid don't must be update at save
    data.id = GetEntry();
    data.mapid = mapid;
    data.phaseMask = phaseMask;
    data.posX = GetPositionX();
    data.posY = GetPositionY();
    data.posZ = GetPositionZ();
    data.orientation = GetOrientation();
    data.rotation0 = GetFloatValue(GAMEOBJECT_PARENTROTATION+0);
    data.rotation1 = GetFloatValue(GAMEOBJECT_PARENTROTATION+1);
    data.rotation2 = GetFloatValue(GAMEOBJECT_PARENTROTATION+2);
    data.rotation3 = GetFloatValue(GAMEOBJECT_PARENTROTATION+3);
    data.spawntimesecs = m_spawnedByDefault ? (int32)m_respawnDelayTime : -(int32)m_respawnDelayTime;
    data.animprogress = GetGoAnimProgress();
    data.go_state = GetGoState();
    data.spawnMask = spawnMask;

    // updated in DB
    std::ostringstream ss;
    ss << "INSERT INTO gameobject VALUES ( "
        << GetGUIDLow() << ", "
        << GetEntry() << ", "
        << mapid << ", "
        << uint32(spawnMask) << ","                        // cast to prevent save as symbol
        << uint16(GetPhaseMask()) << ","                    // prevent out of range error
        << GetPositionX() << ", "
        << GetPositionY() << ", "
        << GetPositionZ() << ", "
        << GetOrientation() << ", "
        << GetFloatValue(GAMEOBJECT_PARENTROTATION) << ", "
        << GetFloatValue(GAMEOBJECT_PARENTROTATION+1) << ", "
        << GetFloatValue(GAMEOBJECT_PARENTROTATION+2) << ", "
        << GetFloatValue(GAMEOBJECT_PARENTROTATION+3) << ", "
        << m_respawnDelayTime << ", "
        << uint32(GetGoAnimProgress()) << ", "
        << uint32(GetGoState()) << ")";

    WorldDatabase.BeginTransaction();
    WorldDatabase.PExecuteLog("DELETE FROM gameobject WHERE guid = '%u'", GetGUIDLow());
    WorldDatabase.PExecuteLog("%s", ss.str().c_str());
    WorldDatabase.CommitTransaction();
}

---

Я вроде такое замечал - ставил моба на карту 0, затем такого же моба на карту 1. При этом моб с карты 0 таинственно исчезал из базы 0о

А если потом резетнуть серв то они "таинственно" оба появятся.

Вы не читали третий пост?

Мде, однако... Я бы подождал комментариев от Владимира по сему щепетильному вопросу перед тем как начать бегать и кричать "все пропало!". Т.к. как говорится, поспешные выводы - кратчайший путь к разочарованию (с).

Предварительно выглядит так, что переделали почти все, кроме запросов к БД и некоторых кусков кода, которые до сих пор расчитывают на уникальные GUIDы для объектов :bad: Душераздирающее зрелище (с)

Получается должно быть PRIMARY KEY (`guid`,`map`) ?

Нужно учитывать тогда и instanceID. Если это, конечно, еще не учтено...

Ага, и переделывать все *_script, npc_gossip и др. для поддержки.

зачем? эта вся возня периферию кода никак затронуть не должна.

Лорд имел ввиду что нужно будет для этих таблиц добавлять поле `map` так как гуиды будут одинаковые для разных карт, а в этих таблицах они как уникальные.

в гуиде полно места. можно пару байт на карту потратить, а в таблицах хранить не каунтер а фуллгуид.

Фуллгуид присваивается сервером?
Какую имеет длину поля?

Статические GUIDы (загружаемые из базы) глобальны - соответственно новый статический спавн _должен_ резервироваться для всех существующих карт (копий карт).
И должен нигде не быть занят в данный момент. Вообще-то спавн мобов из игры при локальности временых гуидов (которые используются на конкретной копии карты) и использовании следующего свободного гуида был всегда маразмом. Так как для нормальной базы гуиды статического заселения не должны идти с дырами в нумерации.

И так
1) все в структуре базы и коде работы с загружаемым статически заселение правильно
2) не работают нормально GM команды спавна
3) как возможное решение резервировать при загрузке специальный интервал свободных гуидов сразу же после статического заселения для спавна из игры. Гуиды призываемых мобов и т.д. выделяются в таком случае после резервированого интревала. И при его окончании выдавать ошибку. В таком случае можно требовать презагрузки сервера для перевыделения интервала - можно в конфиге сделать указания размера резерва. Для рабочего сервера он не нужен (минимален), для тестового - может быть необходим более большой.

[added]Я работаю над патчем.

[added2]Реализовано в [11310]