ProditorMagnus / Ageless-for-1-14

Continues https://github.com/ProditorMagnus/Ageless-for-1-11
8 stars 2 forks source link

[unit_type][variation] without a variation_id #30

Closed stevecotton closed 1 year ago

stevecotton commented 2 years ago

With Ageless 4.28.0, Wesnoth 1.16 prints the warning error config: Skipping duplicate unit variation ID: '' several times when starting a multiplayer game. Having adding a bit more debugging, the unit_types involved are:

ProditorMagnus commented 2 years ago

There has been something unreliable about dalefolk variations, I had to remove them from orocia lists.

ProditorMagnus commented 2 years ago

variation_id: The id of this variation. Defaults to variation_name.

Looks like this warning should be ignored.

stevecotton commented 2 years ago

I did a test, and corrected that Wiki page you quoted. :(

In debug mode, create a AE_mag_Walking_Corpse. There's an edge case with the first variation that has no id: creating a unit with debug mode will default to the variation with variation_name=wyvern, which is the first one with no id, instead of creating the base unit.

Hovering over that unit, the debug commands:

I've just noticed that this is an EoMa unit, and the missing ids have been added in the EoMa version.

In Wesnoth's codebase, 724dd0f9 is supposed to fall back to the variant_name, but this is only in code for displaying the help.

sevu commented 1 year ago

(changes the graphics but not the stats)

I came along that recently too … what I got from it (when using WML instead of debug mode) that using the [transform_unit] tag – and transforming to itself instead of some other unit – will change the stats afterwards as well.

https://github.com/Robertdebrus/ANLEra/blob/master/units/dunefolk/Civilian.cfg#L110-L143