City size change not sent after nuke

• 报告人: kvilhaugsvik
• 属主: kvilhaugsvik
• 类型: Bugs
• 状态: 关闭
• 组件: Server
• 里程碑: 2.6.4 (关闭)
• 优先: 5 - Medium
• 严重性: 5 - Medium
• 处理结果: Fixed
• 文件: 1

Details

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

任务单历史 (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

• 处理结果 Update from 无 to Accepted
• 里程碑 Update from (无) to 2.6.4 (关闭)

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

• 状态 Update from 开启 to 关闭
• 处理结果 Update from Accepted to Fixed

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

评论

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

评论

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

评论

Opened -> #41860

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

评论

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.