Ticket #42253

Governor Presets Shown as "custom" (3.1.90.3-dev 132f869f05)

Eröffnet am: 2021-05-13 04:57 Letztes Update: 2022-02-09 22:43

Auswertung:
Verantwortlicher:
Typ:
Status:
Geschlossen
Komponente:
Meilenstein:
Priorität:
5 - Mittel
Schweregrad:
5 - Mittel
Lösung:
Gefixt
Datei:
1

Details

This applies to qt and gtk guis. It happens if you upgrade your config from S3_[01]or if you start without any ~/.freeciv/.

So let's start with no config. Start a new game. Build a city. In the city dialog, create a new governor preset. Save it. Set the city to run that new governor preset. Go to the Cities page. Observe that this page claims the city is running with the "custom" governor.

The fact that qt and gtk behave the same, means to me that the server is doing stuff that I would have imagined was being done in the client. Ok, I guess that's why the guis are so consistent.

Ticket-Verlauf (3/7 Historien)

2021-05-13 04:57 Aktualisiert von: chippo
  • New Ticket "Governor Presets Shown as "custom" (3.1.90.3-dev 132f869f05)" created
2021-05-20 07:05 Aktualisiert von: cazfi
Kommentar

Reply To chippo

The fact that qt and gtk behave the same, means to me that the server is doing stuff that I would have imagined was being done in the client. Ok, I guess that's why the guis are so consistent.

Server should have nothing to do with client governor presets. Sounds more like it's in client-common code.

2022-02-01 13:45 Aktualisiert von: cazfi
Kommentar

After some investigation (but not full) I suspect the 'max_growth' field of the the cma parameter is not part of the city attribute block that's stored in the savegame. As a result, the city's cma settings loaded from the savegame are not exactly the same as the preset.

2022-02-01 13:59 Aktualisiert von: cazfi
Kommentar

Ok, handling of 'max_growth' is lacking in multiple ways (will open separate ticket about the one identified earlier). The particular problem in this ticket is caused by having wrong value on the preset side, after loading it from the client settings file. It's not saved there at all. We could start saving it, but as there's no gui to adjust it, more sane approach at this point is probably to just set it to the default value.

2022-02-01 14:08 Aktualisiert von: cazfi
  • Lösung Update from Keine to Accepted
  • Meilenstein Update from (Keine) to 3.1.0
  • Komponente Update from (Keine) to Client common
Kommentar

The 'max_growth' field was added in S3_1. Arguably the general solution of this fix could go to S3_0 too.

2022-02-09 22:43 Aktualisiert von: cazfi
  • Status Update from Offen to Geschlossen
  • Verantwortlicher Update from (Keine) to cazfi
  • Lösung Update from Accepted to Gefixt

Bearbeiten

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Anmelden