Opened 18 years ago
Closed 17 years ago
#4372 closed defect (fixed)
UPnP and UTF-8 fixes
| Reported by: | otto at kolsi dot fi | Owned by: | stuartm |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.21.1 |
| Component: | upnp | Version: | head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Mythbackend/UPnP returns recording, music and video information without correct UTF-8 conversions in place. Attached patch fixes couple of places so that e.g. song and recording names are returned correctly.
With this patch in place there still are following places that gets corrupted if there are non-ascii characters:
- when browsing music By Album, the album name is still corrupted
- when browsing recordings By Title, when title is in container/directory name it is corrupted but when that directory is opened, same name is okay in filename
- when browsing recordings By Group the group name / directory name is still corrupted
These are all cases where corruption occurs in container/directory name. This should be easy to fix by someone with more knowledge of the code.
Attachments (3)
Change History (17)
by , 18 years ago
| Attachment: | upnp-utf8.patch added |
|---|
comment:1 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → accepted |
comment:2 by , 18 years ago
comment:3 by , 18 years ago
| Status: | accepted → started |
|---|
comment:4 by , 18 years ago
| Milestone: | unknown → 0.21 |
|---|
comment:5 by , 18 years ago
| Type: | patch → defect |
|---|
by , 18 years ago
| Attachment: | upnp_utf8_fixes.diff added |
|---|
comment:6 by , 18 years ago
| Status: | started → infoneeded |
|---|
Attached a patch with additional utf8 fixes for music and video (some whitespace fixes slipped in too). Could you test the patch and let me know if anything remains to be fixed and whether it breaks anything that used to work?
comment:7 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | infoneeded → closed |
comment:8 by , 18 years ago
comment:9 by , 18 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → new |
There are still following UTF-8 problems:
When recordings are browsed, if recording "directory/folder" contains non-ascii characters it gets corrupted. At the same time all the program titles are fine so this only affects containers/folders/directories (don't know what is the correct term). Problem occurs in several places, when browsing By Title and By Group.
Exactly same happens when browsing music, directory names (album names) are corrupted but song titles are fine.
With Videos, the same issue seems a bit worse. I have only one UPnP client (Nokia N95) and that refuses to open (shows an error) directory that contains non-ascii characters. Since Video-directories are directly read from the filesystem it is possible that I have non-ascii characters in wrong format in video directory names.
comment:10 by , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:11 by , 18 years ago
| Owner: | removed |
|---|---|
| Status: | assigned → new |
by , 18 years ago
| Attachment: | upnp-utf8-v2.patch added |
|---|
comment:12 by , 18 years ago
I've attached upnp-utf8-v2.patch which fixes the UTF-8 problem in Music and Recordings (corruption of album titles and recording names). Patch is against 0.21-fixes.
Videos still have some problems but that might be related to filesystem / environment issues as mentioned earlier in this ticket.
comment:13 by , 17 years ago
| Owner: | set to |
|---|---|
| Status: | new → accepted |
comment:14 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
(In [15241]) UTF8 conversion fixes for the upnp code, patch by Otto Kolsi.
Refs #4372