Opened 9 years ago

Closed 9 years ago

#8665 closed defect (fixed)

TFW/MythBackend will not write non-ASCII filenames

Reported by: ryan.goat@… Owned by: robertm
Priority: minor Milestone: 0.24
Component: MythTV - General Version: Master Head
Severity: medium Keywords: metadata unicode
Cc: Ticket locked: no

Description

I see this when grabbing metadata for Pokémon episodes (notice the accent over the 'e').

The metadata look up with ttvdb.py works fine. Mythfrontend displays the title correctly. Also coverart and fanart are downloaded and saved to the disk (into their respective storage groups). But they have filenames like: "Pok?mon Season 1_coverart.jpg" & "Pok?mon Season 1_fanart.jpg" The 'é' character is not saved correctly. Mythfrontend/mythbackend can not find the files when it tries to display them because it is looking for the special character. The files work fine when I open them manually.

For what it is worth here is how nautilus (gnome file manager) displays these file names: "Pok?mon Season 1_coverart.jpg (invalid encoding)".

The errors in mythfrontend log look like:
2010-07-16 19:32:46.227 MythUIHelper, Error: LoadScaleImage(myth://Fanart@192.168.0.215:6543/Pokémon Season 1_fanart.jpg) failed to load image

Attachments (1)

TFW.diff (479 bytes) - added by robertm 9 years ago.
Allow TFW to write out files with local 8 bit charset.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 9 years ago by robertm

Component: Plugin - MythVideoMythTV - General

Not a MythVideo? Issue, MythBackend/ThreadedFileWriter? will not write out non-ASCII filenames. Patch attached, changing component.

comment:2 Changed 9 years ago by robertm

Summary: metadata image downloads can't handle special unicode characters in titleTFW/MythBackend will not write non-ASCII filenames

Changed 9 years ago by robertm

Attachment: TFW.diff added

Allow TFW to write out files with local 8 bit charset.

comment:3 Changed 9 years ago by robertm

Owner: changed from robertm to cpinkham
Status: newassigned

User reported success with this patch. I've been using it without issue for almost two years.

comment:4 Changed 9 years ago by robertm

Owner: changed from cpinkham to robertm

comment:5 Changed 9 years ago by robertm

Milestone: unknown0.24

comment:6 Changed 9 years ago by robertm

Resolution: fixed
Status: assignedclosed

(In [25628]) Allow ThreadedFileWriter? to write out filenames using the local 8 Bit charset. Fixes issues with artwork filenames being broken or truncated when they contained non-Ascii character sets. Have run this patch for several years without issue. Credit to Mike Dean and Chris Pinkham for helping me figure out what the issue was way back then, scorn to me if it breaks anything. Fixes #8665.

Note: See TracTickets for help on using tickets.