Author Topic: wxSmith non i18N strings with international characters are not correctly managed  (Read 6796 times)

Offline Bug Killer

  • Multiple posting newcomer
  • *
  • Posts: 17
    • Railroad modelling hardware and software
In nightly build 13186
In wxSmith/wxscodinglang.cpp
In wxString WxString(wxsCodingLang Lang, const wxString& Source, bool WithTranslation)

strings with international characters are not considered alphabetic. So, string prefix is always _( whichever is the prefix chosen in wxSmith settings for non i18N strings.

To fix comment out :

                if (DoTranslation)
                {
                    // Check if translation is really needed. For now, just check if it contains alphabetic chars
                    if (std::any_of(Source.begin(), Source.end(), [] (wxUniChar c) {return wxIsalpha(c);}))
                    {
                        Prefix = "_(";
                        Postfix = ")";
                    }
                }
« Last Edit: February 02, 2023, 01:00:04 pm by Bug Killer »

Offline Miguel Gimenez

  • Developer
  • Lives here!
  • *****
  • Posts: 1584
Strings with international characters ARE alphabetic (wxIsalpha() returns true for them).
If you comment out that code no string from wxSmith will be translatable.

If this bothers you then disable global I18n support in wxSmith settings or disable local I18n support in the affected wxSmith window.


Offline Bug Killer

  • Multiple posting newcomer
  • *
  • Posts: 17
    • Railroad modelling hardware and software
Thanks for the explanation. C::B 20.03 behaviour was different.

Is there a way to disable internationalization for all dialog, panels and frame ?

Why strings with international characters are not displayed when the prefix is _( ?

Offline Miguel Gimenez

  • Developer
  • Lives here!
  • *****
  • Posts: 1584
Quote
Is there a way to disable internationalization for all dialog, panels and frame ?
As said before, disable global I18n support in wxSmith settings or disable local I18n support in the affected wxSmith window.

Quote
Why strings with international characters are not displayed when the prefix is _( ?
I do not know, probably wxWidgets' GetTranslatedString() does not accept non-ASCII strings.

Offline sodev

  • Regular
  • ***
  • Posts: 497
GetTranslatedString() works with any narrow string, it does not work with wide strings, but in case of error it simply returns the input value and not empty string.

On windows, two reasons come into my mind, but both apply to narrow strings, so not only the _() macro would be affected. Either the compiler uses a wrong encoding to read the source file and interprets the strings as garbage or the execution charset of the application cannot represent the characters. wxWidgets always assumes narrow strings use the execution charset and converts them to its internal unicode representation, if that fails, it produces an empty string.

Offline Bug Killer

  • Multiple posting newcomer
  • *
  • Posts: 17
    • Railroad modelling hardware and software
Whichever is the value of the global i18N settings prefix is always _( because of a mistake in wxscodinglang.cpp.

Line : const bool DoTranslation = WithTranslation && (cfg->ReadBool("/useI18N"), true);

Should be : const bool DoTranslation = WithTranslation && cfg->ReadBool("/useI18N", true);

Now everything goes well.

Offline Miguel Gimenez

  • Developer
  • Lives here!
  • *****
  • Posts: 1584
Fixed in trunk, thank you.