jeirmeister/Cockatrice
1
0
Fork 0
mirror of https://github.com/Cockatrice/Cockatrice.git synced 2026-11-13 05:20:00 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

[Server] Rely on the player order event instead of a full state resend

Browse source 
cmdSetPlayerOrder re-sent the whole game state after reseating, so every
client reloaded its deck view mid-lobby for no reason. The new seating reaches
clients through the Event_PlayerOrderChanged that reseatPlayers broadcasts.
This commit is contained in:
Lukas Brübach 2026-11-04 08:58:39 +01:00
parent 16a2e70039
commit 7abffe84d5
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
libcockatrice_network/libcockatrice/network/server/remote/game/server_abstract_participant.cpp
View file

@ -457,8 +457,8 @@ Response::ResponseCode Server_AbstractParticipant::cmdSetPlayerOrder(const Comma
game->reorderPlayerSeats(playerNames); game->reorderPlayerSeats(playerNames);
} }
game->sendGameStateToPlayers(); // Clients learn the new seating from the Event_PlayerOrderChanged that
// reseatPlayers broadcasts; no full game state resend is needed for it.
return Response::RespOk; return Response::RespOk;
} }