Ticket #42989

Default range for "Good" req should be a possible one

Date d'ouverture: 2021-10-07 04:43 Dernière mise à jour: 2021-10-30 21:18

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

Détails

Brief code analysis of common/requirements.c shows a problem in req_from_str() function. We usually don't actually use default ranges, but if we have them, let them be possible. VUT_GOOD can not have local range, default one should be RANGE_CITY.

Ticket History (3/7 Histories)

2021-10-07 04:43 Updated by: ihnatus
  • New Ticket "Default range for "Good" req should be a possible one" created
2021-10-08 02:20 Updated by: cazfi
Commentaire

Less than 36h to beta3 release, it's not even theoretically possible that a fix would get submitted and pass review period before it (in practice we've not pushed anything in since Monday)

2021-10-13 21:27 Updated by: cazfi
Commentaire

We define default ranges there, but apparently it's not possible for a ruleset author to rely on it. When one leaves range out from a requirement list:

1: In secfile_lookup_str() [../../../src/utility/registry_ini.c:2177]: secfile '../../src/data/civ2civ3/effects.ruleset' in section 'NULL': "effect_tile_bonus_good.reqs0.range" entry doesn't exist.

2021-10-14 07:27 Updated by: cazfi
  • Résolution Update from Aucun to Accepted
2021-10-30 21:18 Updated by: cazfi
  • État Update from Ouvert to Atteints
  • Propriétaire Update from (Aucun) to cazfi
  • Résolution Update from Accepted to Fixed

Modifier

Please login to add comment to this ticket » Connexion