City size change not sent after nuke

• Relator: kvilhaugsvik
• Dono: kvilhaugsvik
• Tipo: Bugs
• Estado: Fechado
• Componente: Server
• Marcos: 2.6.4 (fechado)
• Prioridade: 5 - Medium
• Gravidade: 5 - Medium
• Resolução: Fixed
• Arquivo: 1

Details

Update city information to all players that can see the tile of the nuked city.

Ticket History (3/8 Histories)

2021-03-19 05:28 Updated by: kvilhaugsvik

• New Ticket "City size change not sent after nuke" created

2021-03-19 05:29 Updated by: kvilhaugsvik

• File 0001-Send-city-size-change-during-nuke.patch (File ID: 6294) is attached

2021-03-19 05:33 Updated by: kvilhaugsvik

• Resolução Update from Nenhum to Accepted
• Marco Update from (Nenhum) to 2.6.4 (fechado)

2021-03-21 21:56 Updated by: kvilhaugsvik

• Estado Update from Aberto to Fechado
• Resolução Update from Accepted to Fixed

2021-03-26 15:09 Updated by: cazfi

Comentário

So this change is about sending *less* data? It used to do full sync_cities() that would send all affected cities. Now it sends just one city.

Just wondering about inclusion of this to S2_6 (updating NEWS-2.6.4 at the moment).

2021-03-26 15:29 Updated by: cazfi

Comentário

Ok, so you want to send the city also to *other* players than the owner. Yet it's a regression that other cities are not sent to their owners (they can be affected *at least* by fallout, maybe also by the worker rearrangements of the nuked city). We should send do both.

2021-03-26 15:31 Updated by: cazfi

Comentário

Opened -> #41860

2021-03-26 17:54 Updated by: kvilhaugsvik

Comentário

Reply To cazfi

Yet it's a regression that other cities are not sent to their owners (they can be affected *at least* by fallout, maybe also by the worker rearrangements of the nuked city). We should send do both.

You are right. Thank you for spotting and fixing the regression.