Extras as local-ranged requirement targets

• Auswertung: alienvalkyrie
• Verantwortlicher: alienvalkyrie
• Typ: Patches
• Status: Geschlossen
• Komponente: General
• Meilenstein: S3_2 d3f (closed)
• Priorität: 5 - Mittel
• Schweregrad: 5 - Mittel
• Lösung: Gefixt
• Datei: 1

Details

Add support for extras as requirement targets (as already exists for e.g. improvements and unit types), with associated local-ranged requirements (RoadFlag, Extra, ExtraFlag).

This is possible now since the "exists on tile" kind of requirement has been moved to the Tile range with hrm #695469. However, to avoid potential rscompat issues, adding new things to the Local range should not be done until after #43708.

Ticket-Verlauf (3/6 Historien)

2022-02-08 07:52 Aktualisiert von: alienvalkyrie

• New Ticket "Extras as local-ranged requirement targets" created

2022-02-08 08:47 Aktualisiert von: alienvalkyrie

Kommentar

Probably most sensible to delay this until after #43809, to make adding the new requirement target easier.

2022-02-15 03:51 Aktualisiert von: alienvalkyrie

• Verantwortlicher Update from (Keine) to alienvalkyrie

2022-02-15 08:44 Aktualisiert von: alienvalkyrie

• File 0005-Allow-extras-as-targets-for-local-ranged-requirement.patch (File ID: 8539) is attached

2022-02-15 08:44 Aktualisiert von: alienvalkyrie

• Lösung Update from Keine to Accepted

Kommentar

A few notes on this patch:

• The patch depends on #43809, and softly depends on #43708 to avoid potential rscompat confusion.
• No place in the code passes target extras to requirement evaluation yet. This is left to other tickets like #41539 and #41540.
  • This also means the requirement evaluation code is not testable in practice yet
• The default range for Extra, RoadFlag and ExtraFlag requirements is still Tile, not Local, to keep the current behavior

2022-02-19 03:41 Aktualisiert von: alienvalkyrie

• Status Update from Offen to Geschlossen
• Lösung Update from Accepted to Gefixt