Миникарта

[Softer]

Заметил проблему с миникартой:
Карта не обновляет местоположение членов группы.
Но когда кто-то летит с помощью флай мастера, то его путь на карте показывает членам пати.

Я не совсем понимаю: миникарта завязана на сервер или релиазована исключительно в клиенте?

[Vinolentus]

Миникарта реализована в клиенте. Местоположение членов группы "завязано на сервер".
Посмотри, когда ставится GROUP_UPDATE_FLAG_POSITION.

[zergtmn]

На чистом ядре проверял?

[Softer]

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

Как я понял отсылка (или формирование) пакета происходит в этом участке

709     if (mask & GROUP_UPDATE_FLAG_POSITION)
710         *data << uint16(player->GetPositionX()) << uint16(player->GetPositionY());

Файла GroupHandler.cpp, а реакция на запрос в этом куске

848     data << uint16(player->GetPositionX());                 // GROUP_UPDATE_FLAG_POSITION                                                                                       
849     data << uint16(player->GetPositionY());                 // GROUP_UPDATE_FLAG_POSITION

Верно?

На чистом ядре проверял?

Чистое, Insider, Rsa одинаково. Клиентов тоже пробовал менять :-D


ADD: Данные куски беру из репозитория Rsa

[Vinolentus]

Сказал же, посмотри,

когда ставится

, а не как обрабатывается поставленный.
Вот оно от rsa:

bool Player::SetPosition(float x, float y, float z, float orientation, bool teleport)
{
    if(!Unit::SetPosition(x, y, z, orientation, teleport))
        return false;
...
    const float old_x = GetPositionX();
    const float old_y = GetPositionY();
...

        // group update
        if (GetGroup() && (old_x != x || old_y != y))
            SetGroupUpdateFlag(GROUP_UPDATE_FLAG_POSITION);

Более чем уверен, что m_position... у него выставляется в первой же строчке. И взятые потом old_x, old_y всегда будут равны x, y.

[Forgotten]

копипастить внимательно нужно.. Unit::SetPosition возвращает false если юнит не был перемещен, во всех остальных случаях -- true
другими словами, сравнение координат вообще смысла не имеет

[Softer]

Vinolentus, т.е. ты имеешь ввиду что скачала вызывается Unit::SetPosition изменяя координаты юнита, а потом координаты считываются здесь

    const float old_x = GetPositionX();
    const float old_y = GetPositionY();

и в итоге всегда old_{x,y} равны {x,y}?

[Vinolentus]

копипастить внимательно нужно

Читать код внимательно нужно. Тут даже думать не требуется x_x

bool Unit::SetPosition(float x, float y, float z, float orientation, bool teleport)
{
...
        if (GetTypeId() == TYPEID_PLAYER)
            GetMap()->PlayerRelocation((Player*)this, x, y, z, orientation);
...
}
void
Map::PlayerRelocation(Player *player, float x, float y, float z, float orientation)
{
...
    player->Relocate(x, y, z, orientation);
...
}
void WorldObject::Relocate(float x, float y, float z, float orientation)
{
    m_positionX = x;
    m_positionY = y;
    m_positionZ = z;
    m_orientation = orientation;
...
}

[zergtmn]

rsa криво сделал мердж.

Должно быть

bool Player::SetPosition(float x, float y, float z, float orientation, bool teleport)
{
    if (!Unit::SetPosition(x, y, z, orientation, teleport))
        return false;

    // group update
    if (GetGroup())
        SetGroupUpdateFlag(GROUP_UPDATE_FLAG_POSITION);

    if (GetTrader() && !IsWithinDistInMap(GetTrader(), INTERACTION_DISTANCE))
        GetSession()->SendCancelTrade();   // will close both side trade windows

    // code block for underwater state update
    UpdateUnderwaterState(GetMap(), x, y, z);
    CheckAreaExploreAndOutdoor();

    return true;
}

[Forgotten]

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

Читать код внимательно нужно. Тут даже думать не требуется x_x

товарисч Vinolentus, моя реплика относилась к коду rsa, посмотрите оригинальный код trinity
убавьте вашу агрессию

[Softer]

zergtmn, это откуда такой кусок? Или ты просто пропустил часть функции?
В чистом ядре написано почти то-же самое что и у Rsa, за исключением Unit::SetPosition...

товарисч Vinolentus, моя реплика относилась к коду rsa, посмотрите оригинальный код trinity
убавьте вашу агрессию

При чем тут Тринити?

[zergtmn]

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

zergtmn, это откуда такой кусок? Или ты просто пропустил часть функции?
В чистом ядре написано почти то-же самое что и у Rsa, за исключением Unit::SetPosition...

Часть кода вынесена в Unit::SetPosition, в Player::SetPosition это дублировать не нужно.

[Softer]

Понял, спасибо, буду ковырять

[rsa]

Всем спасибо за отыскание проблемы. Подцепленной при тестировании патча на улучшение обработки видимости объектов от Silverice