Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#8530 closed patch (fixed)

Wrong encoding of phrases

Reported by: sandybigboy@… Owned by: sphery
Priority: minor Milestone: 0.23-fixes
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no


In the Manage Recordings->Schedule Recordings->Search Words->Titles (an other options in this menu), the "Title/Phrase?" screen is using a wrong charset when displaying phrases. The phrases are stored UTF-8 encoded in the database backend, but the are displayed ISO-8859-1 or ASCII encoded in the frontend. This is similar to other ticket #7813 In the course of the discussion in this ticket mdean wrote: "Qt-MySQL drivers use QVariant::ByteArray?? for string-type MySQL fields marked with the BINARY attribute (those using a *_bin collation) and QVariant::String for all others. Since QVariant::toString() uses QString::fromAscii() (through QVariant::convert()) when the QVariant's type is QVariant::ByteArray??, we have to use QString::fromUtf8() explicitly to prevent corrupting characters for any columns using a binary collation (or, at the minimum, in any location where the value retrieved from the database is shown to the user).

The following columns are binary (using utf8_bin collation), and may require similar treatment: keyword.phrase oldprogram.oldtitle powerpriority.priorityname recgrouppassword.recgroup storagegroup.dirname "

Attached patch fixes bug with wrong encoding of phrases stored in keyword.phrase field.

Attachments (1)

phrase_encoding_in_proglist.patch (786 bytes) - added by sandybigboy@… 10 years ago.

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by sandybigboy@…

comment:1 Changed 10 years ago by danielk

Owner: changed from Isaac Richards to sphery
Status: newassigned

comment:2 Changed 10 years ago by sphery

Status: assignedaccepted

comment:3 Changed 10 years ago by sphery

(In [25424]) Fix the phrase encoding in proglist.cpp.

This is another case of improper conversion of utf8_bin column data, like the case in #7813. This fixes the issue until I can convert all the utf8_bin columns to utf8_general_ci.

Refs #8530. Thanks to sandybigboy for identifying the issue and submitting the patch.

comment:4 Changed 10 years ago by sphery

Resolution: fixed
Status: acceptedclosed

Fixed in [25427] .

Fix the phrase encoding in proglist.cpp. Backports [25424] from trunk.

comment:5 Changed 10 years ago by stuartm

Milestone: unknown0.23-fixes
Note: See TracTickets for help on using tickets.