CQUI-Org / cqui

Civilization 6 - Chao's Quick UI. Reduce clicks and manage your empire faster!

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Report Screen: Ressource-Label is missing

JHCD opened this issue · comments

The label for bonus ressources isn't working...

Report Screen -> Ressources
rs_ressource_label_missing

It works for me:
image

Could it be a that the text is to long in German? Or Firaxis have managed to remove the text for that LOC.

No, there is a short word in the localisation:

<Replace Tag="LOC_HUD_REPORTS_BONUSES" Language="de_DE">
  <Text>{1: plural 1?Bonus; 2?Bonusse; other?Bonusse;}</Text>
</Replace>

There is a linguistic differentiation between "singular/plural" in the string.
So I think the string isn't called correctly.

That is probably the reason. The string is called just as it is in Vanilla, so it is an oversight by Firaxis. It is just added with String="LOC_HUD_REPORTS_BONUSES" on the xml element, I think the text needs to be fetched using Locale.Lookup().

Hmm unless it is handled differently for Label vs CheckBox or the style could have an impact I guess.

It's not the size, it's not Locale.Lookup...
Oh, Germans... ;)

where is the style defined? It might be that.

OK, got it... it's the stupid pluralization of the string... I replaced the string for a single word (all we need, you know?) and it works fine. I will assume that the gerries (I love you) are the only ones with this problem and update the urs txt file, feel free to thinker with more languages if you want.

That doesn't work. Civ skipped this entry as duplicated key.
Otherwise you would kill the correct single use on other UI elements.

It worked fine on my setup.

I will play some turn with it...
Can't believe, that it is isolated for the mod only.

In my opinion we shouldn't change official text strings, because we don't know where they are used either.
It's more overhead, but when necessary we should create our own key.

Very funny... the issue is already in the original game...
Never missed that label ;)

[843875.858] ERROR: Missing argument 1. ({1: plural 1?Bonus; 2?Bonusse; other?Bonusse;})
[843875.858] ERROR: Failed to evaluate parameter 1. {1: plural 1?Bonus; 2?Bonusse; other?Bonusse;}

Surpriiiisee!!!

Firaxis. The amount of bugs they introduced with this new "patch" is appalling... I know they have fallen low since the Soren times, but this low...?

Only option then is to transform the loc into a custom one, in honor of our Deutsches Volke... will do.

In this case yes, but there a many of this plurals constructs in different language files.
So I think there must be a solution and they have only forgotten it in this xml.

Done. You can merge at will now.