Ticket #43759

Metamessage resetted on ruleset load

Date d'ouverture: 2022-02-01 21:44 Dernière mise à jour: 2022-03-25 00:45

Rapporteur:
Propriétaire:
Type:
État:
Atteints
Composant:
Priorité:
5 - moyen
Sévérité:
5 - moyen
Résolution:
Fixed
Fichier:
4

Détails

As metamessage was turned to a server setting in freeciv-3.0, it now gets resetted to default (empty value) when a new ruleset is loaded. That's even when it has been set with the (supposedly backward compatible) dedicated 'metamessage' command, and not via 'set' command interface.

Ticket History (3/13 Histories)

2022-02-01 21:44 Updated by: cazfi
  • New Ticket "Metamessage reseted on ruleset load" created
2022-02-01 21:48 Updated by: cazfi
Commentaire

Arguably resetting *any* server operation related settings just when the rules should change is a bug.

2022-02-01 21:48 Updated by: cazfi
  • Summary Updated
2022-02-01 22:51 Updated by: cazfi
Commentaire

Attached is a S3_0 patch that I've applied to cazfi.net server as a quick fix. It might be a sensible starting point.

2022-02-03 23:51 Updated by: cazfi
2022-03-13 06:03 Updated by: cazfi
  • Propriétaire Update from (Aucun) to cazfi
  • Résolution Update from Aucun to Accepted
Commentaire

Attached is the earlier patch with minimal adjustments. I have a feeling that the change is a bit big for resolving a simple problem, but no better solution has been suggested and I consider this one of the most important issues to fix in 3.0.1.

2022-03-13 23:09 Updated by: cazfi
  • Résolution Update from Accepted to Aucun
Commentaire

Patch going to master should follow new CodingStyle compound literals recommendation.

2022-03-14 08:23 Updated by: cazfi
  • Résolution Update from Aucun to Accepted
Commentaire

- New master version with compound literals used

2022-03-25 00:45 Updated by: cazfi
  • État Update from Ouvert to Atteints
  • Résolution Update from Accepted to Fixed

Modifier

Please login to add comment to this ticket » Connexion