update_city_activity() mishandling gold surplus

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

Details

update_city_activity() does not use gold surplus, but tries to recalculate it itself:

    /* Update the treasury. */
    pplayer->economic.gold += pcity->prod[O_GOLD];
    pplayer->economic.gold -= city_total_impr_gold_upkeep(pcity);
    pplayer->economic.gold -= city_total_unit_gold_upkeep(pcity);

This breaks Surplus_Waste -effects, and was also mentioned in https://www.hostedredmine.com/issues/892050

Ticket History (3/5 Histories)

2023-07-11 10:13 Updated by: cazfi

• New Ticket "update_city_activity() mishandling gold surplus" created

2023-07-22 04:14 Updated by: cazfi

• Details Updated

2023-07-22 06:14 Updated by: cazfi

• File 0017-Increase-treasury-by-precalculated-city-gold-surplus.patch (File ID: 12931) is attached

2023-07-22 06:30 Updated by: cazfi

• Dono Update from (Nenhum) to cazfi
• Resolução Update from Nenhum to Accepted
• Marco Update from (Nenhum) to 3.0.9 (fechado)

Comentário

Reply To cazfi

This breaks Surplus_Waste -effects

For that reason only main/S3_2 affected. Still, I'd want the fix to S3_1 and even to S3_0 too, to act as a baseline for other fixes.

2023-07-31 05:33 Updated by: cazfi

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

Comentário

Reply To cazfi

This breaks Surplus_Waste -effects

For that reason only main/S3_2 affected. Still, I'd want the fix to S3_1 and even to S3_0 too, to act as a baseline for other fixes.