Bug 359 - UTF8 tags in FLAC files not displaying properly
: UTF8 tags in FLAC files not displaying properly
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Tagging
: 5.x or older
: PC RedHat Linux
: P2 normal (vote)
: ---
Assigned To: Blackketter Dean
http://lists.slimdevices.com/archives...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-06-08 16:50 UTC by Michael Brouwer
Modified: 2008-12-18 11:50 UTC (History)
0 users

See Also:
Category: ---


Attachments
Patch to Slim/Formats/FLAM.pm (1.21 KB, patch)
2004-06-12 11:27 UTC, Michael Brouwer
Details | Diff
New patch that doesn't depend on perl 5.8 (1.19 KB, patch)
2004-06-25 23:43 UTC, Michael Brouwer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Brouwer 2004-06-08 16:50:14 UTC
I'm running slimserver-2004_05_26-1 on linux (redhat 9) and I've noticed that if I have FLAC files with 
native tags, strings with UTF8 in them don't display correctly in the browser nor on the squeezebox 
display.

For example "Bj�rk" doesn't display right the output of
    metaflac --export-vc-to=- 01* | hexdump -C
00000000  41 52 54 49 53 54 3d 42  6a c3 b6 72 6b 0a 54 49  |ARTIST=Bj..rk.TI|

shows the UTF sequence C3 B6 for the \"o character, which seems to be in agreement with what my 
Mac thinks it should be.

In the browser and on the squeezebox it displays as: Björk.
Comment 1 Michael Brouwer 2004-06-12 11:27:21 UTC
Created attachment 46 [details]
Patch to Slim/Formats/FLAM.pm

Here is a patch for Slim/Formats/FLAC.pm I cooked up that makes FLAC tags with
non ascii chars in them display properly.  A better fix would be keep tags in
utf8 format as long as possible though out slimsever, but that should probably
wait until post 5.2.  I checked and I believe this patch should also work on
Mac OS 10.2 (Jaguar).  So far I've tested it on redhat 9

Note that although CPAN::MP3::Info can return tags in utf8 format it currently
doesn't so id3 tags with non latin1 encodings don't work right currently. 
That's a separate bug though.
Comment 2 Michael Brouwer 2004-06-25 23:43:32 UTC
Created attachment 53 [details]
New patch that doesn't depend on perl 5.8

Been running with this for several days now and it works like a charm
Comment 3 Chris Owens 2006-06-16 14:40:21 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.
Comment 4 Chris Owens 2008-12-18 11:50:57 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.