This both silences a wmllint warning about special notes and ensures
that the special notes displayed in the unit's description in-game are
consistent with the two variations for this unit type (not that many
people will ever get to see the pre-reveal variation's anyway).
No translatable string changes.
Silences a newly introduced wmllint warning.
No effective changes since the engine sees these as synonymous when only
one of the attributes is present.
Silences a newly introduced wmllint warning.
No effective changes since the engine sees these as synonymous when only
one of the attributes is present.
Silences a newly introduced wmllint warning.
No effective changes since the engine sees these as synonymous when only
one of the attributes is present.
This is somewhat incomplete, but already covers most core functionality
and helped uncover an important bug in the BFS implementation (see
commit 1c927c8407).
Most notably, normalize_path() isn't tested yet, as well as corner cases
with empty strings and get_wml_location().
wesnoth.get_locations includes border hexes, so they need to be
excluded specifically. Previously this could cause error messages and
the CA to be blacklisted.
The problem is due to a dog adjacent to a sheep being excluded from the
dog_move CA (so that it can act further later on in case the sheep
moves again). It is fixed by adding another CA at the end that simply
takes all moves away from all dogs once all other CAs are done.
Hexes occupied by allied units were previously not excluded properly by
the AI, which could lead to error messages and the candidate action to
be blacklisted.
While chapter 3 only skipped one track, leading to an off by one error,
the track of chapter 4 had to be fixed and the SAURIANS track for chapter
5 had to be made completely anew.
[skip ci]
Needs to be reverted and replaced with real support for variation_id +
variation_name in wmllint (as well as the legacy
variation_name-as-variation_id behavior).