HomeWildfire Games

Don't error out on complex requirements without tooltip.

Description

Don't error out on complex requirements without tooltip.

Since it is not nice to make the game unusable (citation needed) we'll emit a warning, which is less not nice.
Reported by: @Langbart

Differential revision: https://code.wildfiregames.com/D4930
Reviewed by: @Langbart
Fixes #6724

Event Timeline

Does this mean we need to be adding a <RequirementsTooltip> entry to a bunch of units for a27? I am seeing this warning all the time. It would help this effort a ton if the warning could print the template name missing the RequirementsTooltip, so that we know immediately which template to fix with the requirements tooltip.
I can't tell which template to change. The technology has the same requirementsTooltip as all the other city phase upgrades, and the warning occurs whether or not I have a Tooltip added to the unit involved.

I am seeing this warning all the time.

Define "all the time"? Since I don't get it al all. ^^

I am seeing this warning all the time.

Define "all the time"? Since I don't get it al all. ^^

Any new unit that requires some technology to unlock. Ex. https://code.wildfiregames.com/D5114 and https://code.wildfiregames.com/D5109.
Onagers, Legionaries, and Centurions all produce this warning even though they all have a tooltip.
I am also confused how the athenian marines do not cause this warning even though they have no tooltip, and are locked behind a unique tech.

Without looking at the specific templates, you may need to replace="" or remove (-) the previous tech requirement.

You'd need a Tooltip inside the Requirements entry for specifying the requirements.
Example:

<Requirements>
  <Tooltip>Unlocked by researching Marian Reforms or by reaching the City Phase.</Tooltip>
</Requirements>