Tíquete #41796

NoVeteran in effects but not in use

: 2021-03-17 03:21 Última Atualização: 2023-09-02 17:35

Relator:
Dono:
Tipo:
Estado:
Fechado
Componente:
Marcos:
Prioridade:
5 - Medium
Gravidade:
5 - Medium
Resolução:
Fixed
Arquivo:
3

Details

The rulesets civ2civ3, classic, experimental, sanndbox and webperimental mention the NoVeteran flag in effects but no unit has the flag.

Ticket History (3/12 Histories)

2021-03-17 03:21 Updated by: kvilhaugsvik
  • New Ticket "NoVeteran in effects but not in use" created
2021-03-20 04:28 Updated by: ihnatus
Comentário

Might be, it's a good preparation for the case they ever appear, and we should just omit such requirements when we load the ruleset?

2021-03-20 05:53 Updated by: kvilhaugsvik
Comentário

Reply To ihnatus

we should just omit such requirements when we load the ruleset?

No point omitting them. A !present requirement for a flag not in use is always fulfilled.

2021-03-20 06:14 Updated by: cazfi
Comentário

Also, we should not omit any rules from the ruleset at loading time - that would leave them out also when saving rules back from ruledit or ruleup. We even have to store some meta-rules that have any effect only at ruleset loading time.

2021-03-20 19:42 Updated by: ihnatus
Comentário

Reply To kvilhaugsvik

Reply To ihnatus

we should just omit such requirements when we load the ruleset?

No point omitting them. A !present requirement for a flag not in use is always fulfilled.

Just for a bit of productivity. But per Cazfi, maybe not worth the work.

2023-05-12 13:18 Updated by: cazfi
Comentário

Reply To kvilhaugsvik

The rulesets civ2civ3, classic, experimental, sanndbox and webperimental mention the NoVeteran flag in effects but no unit has the flag.

See also #48028 (affects main -> 3.3+)

2023-06-26 20:56 Updated by: cazfi
  • Marco Update from (Nenhum) to 3.0.9 (fechado)
  • Componente Update from (Nenhum) to Rulesets
2023-08-27 13:03 Updated by: cazfi
  • Dono Update from (Nenhum) to cazfi
  • Resolução Update from Nenhum to Accepted
Comentário

Reply To cazfi

See also #48028 (affects main -> 3.3+)

Thus main is not affected. Patches for S3_0 - S3_2 attached.

2023-09-02 17:35 Updated by: cazfi
  • Estado Update from Aberto to Fechado
  • Resolução Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Login