#264 SendQuestUpdates does not properly reflect quest flag to client

Stängd
öppnade 3 år sedan av image · 1 kommentarer
image kommenterad 3 år sedan
Det finns inget innehåll än.
image kommenterad 3 år sedan
Deltagare

Fixed code replaces the else statement to only call SendAllSpawnsForVisChange instead of iterating entire spawn list and doing some sanity checks that didn't work to update spawns correctly.

void ZoneServer::SendQuestUpdates(Client* client, Spawn* spawn){
    if(!client)
        return;

    if(spawn){
        if(client->GetPlayer()->WasSentSpawn(spawn->GetID()) && !client->GetPlayer()->WasSpawnRemoved(spawn))
            SendSpawnChanges(spawn, client, false, true);
    }
    else{
        client->GetCurrentZone()->SendAllSpawnsForVisChange(client);
    }
}
Fixed code replaces the else statement to only call SendAllSpawnsForVisChange instead of iterating entire spawn list and doing some sanity checks that didn't work to update spawns correctly. ``` void ZoneServer::SendQuestUpdates(Client* client, Spawn* spawn){ if(!client) return; if(spawn){ if(client->GetPlayer()->WasSentSpawn(spawn->GetID()) && !client->GetPlayer()->WasSpawnRemoved(spawn)) SendSpawnChanges(spawn, client, false, true); } else{ client->GetCurrentZone()->SendAllSpawnsForVisChange(client); } } ```
Logga in för att delta i denna konversation.
Laddar...
Avbryt
Spara
Det finns inget innehåll än.