Page MenuHomeWildfire Games

Translate XmppClient::TLSErrorToString
ClosedPublic

Authored by elexis on Sep 9 2019, 4:07 PM.

Details

Summary

These XMPP Certificate error strings were added untranslated in rP21901 during string freeze last release and should become translated now.

Test Plan

Install pology and run the source/tools/l10n/updateTemplates.py script. Notice that the new strings (should) show up in the transifex pology files.
If you fail to install pology because there are no repositories available, compare this piece of code against the other translated strings, and in case this is committed, check the next autobuild committed pology files to verify.

To test that the strings are actually displayed,
change in default.cfg:

server = "lobby2.wildfiregames.com" ; Address of lobby server

and in XmppClient::onDisconnect:

CreateGUIMessage("system", "disconnected", std::time(nullptr), "reason", TLSErrorToString(gloox::CertWrongPeer));

for instance.

The string should show up in engine.pot, for example:

#: lobby/XmppClient.cpp:1102
msgid "Player already logged in"
msgstr ""

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

elexis created this revision.Sep 9 2019, 4:07 PM
Stan awarded a token.Sep 9 2019, 4:12 PM
Vulcan added a comment.Sep 9 2019, 4:16 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/110/display/redirect

elexis added inline comments.Sep 9 2019, 4:17 PM
source/lobby/XmppClient.cpp
1099 ↗(On Diff #9680)

Notice that this cannot become a static const (member or local) because the target language can change during calls.

elexis edited the summary of this revision. (Show Details)Sep 9 2019, 4:18 PM
elexis updated the Trac tickets for this revision.
Vulcan added a comment.Sep 9 2019, 4:19 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/619/display/redirect

elexis edited the test plan for this revision. (Show Details)Sep 9 2019, 4:23 PM
elexis edited the test plan for this revision. (Show Details)Sep 9 2019, 4:32 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 9 2019, 4:34 PM
This revision was automatically updated to reflect the committed changes.

If you fail to install pology because there are no repositories available, compare this piece of code against the other translated strings, and in case this is committed, check the next autobuild committed pology files to verify.

Check:
https://code.wildfiregames.com/rP22896#change-yysGWEA7AmCF