#891 AnyNonLord treated as AnyNonLordOrDemilord

[Anonymous]

Conditional is checking !lord && !demi in RecruitingSubTree.getPossibleRecruiters( hex )

if (!anyNonLord.keySet().isEmpty())
{
for (CreatureType ct : creatureTypes.getCreatureTypes())
{
if (!ct.isLord() && !ct.isDemiLord())
{
possibleRecruiters.add(ct);
}
}
}

[Anonymous]

Can 3 warlocks muster a guardian? The AnyNonLord should still be fixed, but maybe the terrain definition should say AnyNonLordOrDemiLord.

[Craig Lish]

A warlock is not a creature, it is a demilord, so the game works right in the wrong way :o)

[Clemens Katzer]

All right, in principle doesn't look that difficult to fix the variant XML data accordingly.

But we have to keep the compatibility issues in mind, what happens when we install new version to server, but players use older Colossus versions which contain old/different variant definitions. That might or might not go well, depending on the case. I've created a bug report about that issue:

https://sourceforge.net/tracker/?func=detail&aid=3590291&group_id=1939&atid=101939

I guess I will try to fix that first, and make a very minor bugfix release asap to make it possible for users to have/use a client release that is in that sense insensitive to "server has different variant XML data".