Terra theme uses UTF-8 in "Playback starting" message

[malcolm.parsons@…]

Changeset ​http://cvs.mythtv.org/trac/changeset/22606 changed "..." to "…" in the "Playback starting ..." message.

Mythbuntu's build of 0.22-fixes displays this as "Playback starting …".

I expect this changeset also broke all the translations of this message.

[stuartm]

It was changed at the request of a translator who suggested that the elipsis character was preferable to dot-dot-dot. Yes Terra uses UTF-8, this isn't the problem, it's perfectly correct (character set is correctly declared at the start of the document). I'm happy to change it back, but I be even happier to get to the bottom of why it's incorrectly displayed, QT bug perhaps?

[malcolm.parsons@…]

mythuitext.cpp calls qPrintable() on the value string:

if (element.attribute("lang","").isEmpty()) {

m_Message = qApp->translate("ThemeUI",

qPrintable(getFirstText(element)));

} else if ...

[stuartm]

Nice catch, we don't want to use the value from translate() when it's untranslated.

m_Message = qApp->translate("ThemeUI", qPrintable(getFirstText(element)));
QString tmp = getFirstText(element);
if (m_Message == qPrintable(tmp))
    m_Message = tmp;

is a little messy, but the only solution I can see right now.

[stuartm]

Well obviously moving up the assignment to tmp up a line avoids an additional call to getFirstText() ...

[Malcolm Parsons <malcolm.parsons@…>]

Would this work?

m_Message = qApp->translate("ThemeUI", getFirstText(element).toUtf8(), null, QCoreApplication::UnicodeUTF8);

[stuartm]

Malcom that fix would force themes to use UTF-8 or a compatible encoding. I don't expect themes to use anything else, but there is no reason to restrict them either, we should be able to accept any encoding so long as the xml charset declaration is correct. I'll go with my suggested fix in this case.

[stuartm]

On a second reading I realise I was wrong, Malcolm's proposed fix wouldn't restrict the original charset since the conversion to unicode would have already happened by that point, however I'll still go with the other fix since it would avoid two unnecessary character encoding changes on the string in most cases.

[stuartm]

(In [23970]) Fix the display of non-Latin1 strings in the UI when no translation was available. After some thought (coffee) I went with the fix suggested by Malcolm Parsons as it was the most logical and neatest solution. I will backport once I'm certain that it doesn't break anything. Fixes #7815

[malcolm.parsons@…]

I'm using 0.23-fixes, the Playback starting message is still displayed incorrectly.

See also ticket 8581.

[stuartm]

Closing as a duplicate of #8581, this was fixed for linux, ongoing problems seem to be specific to OSX.

[malcolm.parsons@…]

I am on Linux.

[stuartm]

(In [25145]) Backport [23970] to 0.23-fixes. Fix the display of non-Latin1 strings in the UI when no translation was available. Fixes #7815