Ticket #42102

Rename "spaceship_travel_time" server setting

Eröffnet am: 2021-04-29 02:36 Letztes Update: 2022-01-29 13:19

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

Details

Rename "spaceship_travel_time" server setting to indicate it's percentage multiplier.

Ticket-Verlauf (3/7 Historien)

2021-04-29 02:36 Aktualisiert von: cazfi
  • New Ticket "Rename "spaceship_travel_time" server setting" created
2021-11-10 10:16 Aktualisiert von: cazfi
Kommentar

Now that we have support also for older format (one version back) rulesets, renaming server settings has become more hairy. It's not only the existing, and already robust, system of handling compatibility with old savegames with the old setting name, but now we should consider also old game.ruleset with server settings included.

AFAIK this would be the first time we need to support outdated server setting name from a ruleset, and we would need to write the systems to do it from scratch. As the issue with "spaceship_travel_time" name is not very serious, I'm inclined to postpone this to 3.2, instead of starting to squeeze all of this to 3.1 schedule.

2021-12-12 15:42 Aktualisiert von: cazfi
Kommentar

Reply To cazfi

As the issue with "spaceship_travel_time" name is not very serious, I'm inclined to postpone this to 3.2, instead of starting to squeeze all of this to 3.1 schedule.

Writing the code is not going to take much time, but it would be new code introduced to a stabilizing branch. Will do to master only.

2021-12-12 17:01 Aktualisiert von: cazfi
Kommentar

setcompat module to be used -> #43394

2022-01-15 22:54 Aktualisiert von: cazfi
  • Lösung Update from Keine to Accepted
2022-01-29 13:19 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