#264 SendQuestUpdates does not properly reflect quest flag to client

Closed
opened 3 years ago by image · 1 comments
image commented 3 years ago
There is no content yet.
image commented 3 years ago
Collaborator

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); } } ```
Sign in to join this conversation.
Loading...
Cancel
Save
There is no content yet.