Bug 40773

Summary:	russian mp3 tag info is in cp1251 while locale codec is KOI8-R
Product:	[Unmaintained] kio	Reporter:	stephan
Component:	general	Assignee:	Multimedia Developers <kde-multimedia>
Status:	RESOLVED DUPLICATE
Severity:	wishlist
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description stephan 2002-04-11 20:35:25 UTC

(*** This bug was imported into bugs.kde.org ***)

Package:           noatun
Version:           KDE 3.0.0 
Severity:          wishlist
Installed from:    Compiled From Sources
Compiler:          gcc 3.0.4
OS:                Linux
OS/Compiler notes: Not Specified

i think this patch to kdemultimedia-3.0/kfile-plugins/mp3/kfile_mp3.cpp  will solve the problem

34a35
> #include <qtextcodec.h>
178a180185
>
>         QString loc = QTextCodec::locale();
> 
>         if (loc == "ru")
>         {
>             QTextCodec *codec = QTextCodec::codecForName("CP 1251"); // get the codec for CP 1251
180184c187200
<         appendItem(id3group "Title" QString::fromLocal8Bit(mp3.id3.title));
<         appendItem(id3group "Artist" QString::fromLocal8Bit(mp3.id3.artist));
<         appendItem(id3group "Album" QString::fromLocal8Bit(mp3.id3.album));
<         appendItem(id3group "Date" QString::fromLocal8Bit(mp3.id3.year));
<         appendItem(id3group "Comment" QString::fromLocal8Bit(mp3.id3.comment));
---
>             appendItem(id3group "Title" codec->toUnicode(mp3.id3.title));
>             appendItem(id3group "Artist" codec->toUnicode(mp3.id3.artist));
>             appendItem(id3group "Album" codec->toUnicode(mp3.id3.album));
>             appendItem(id3group "Date" codec->toUnicode(mp3.id3.year));
>             appendItem(id3group "Comment" codec->toUnicode(mp3.id3.comment));
>         }
>         else
>         {
>             appendItem(id3group "Title" QString::fromLocal8Bit(mp3.id3.title));
>             appendItem(id3group "Artist" QString::fromLocal8Bit(mp3.id3.artist));
>             appendItem(id3group "Album" QString::fromLocal8Bit(mp3.id3.album));
>             appendItem(id3group "Date" QString::fromLocal8Bit(mp3.id3.year));
>             appendItem(id3group "Comment" QString::fromLocal8Bit(mp3.id3.comment));
>         }
257261c273292
<     strncpy(mp3.id3.title  info["id3v1.1"]["Title"]  .value().toString().local8Bit() 31);
<     strncpy(mp3.id3.artist info["id3v1.1"]["Artist"] .value().toString().local8Bit() 31);
<     strncpy(mp3.id3.album  info["id3v1.1"]["Album"]  .value().toString().local8Bit() 31);
<     strncpy(mp3.id3.year   info["id3v1.1"]["Date"]   .value().toString().local8Bit()  5);
<     strncpy(mp3.id3.commentinfo["id3v1.1"]["Comment"].value().toString().local8Bit() 29);
---
>     QString loc = QTextCodec::locale();
> 
>     if (loc == "ru")
>     {
>         QTextCodec *codec = QTextCodec::codecForName("CP 1251"); // get the codec for CP 1251
> 
>         strncpy(mp3.id3.title  codec->fromUnicode(info["id3v1.1"]["Title"]  .value().toString()) 31);
>         strncpy(mp3.id3.artist codec->fromUnicode(info["id3v1.1"]["Artist"] .value().toString()) 31);
>         strncpy(mp3.id3.album  codec->fromUnicode(info["id3v1.1"]["Album"]  .value().toString()) 31);
>         strncpy(mp3.id3.year   codec->fromUnicode(info["id3v1.1"]["Date"]   .value().toString())  5);
>         strncpy(mp3.id3.commentcodec->fromUnicode(info["id3v1.1"]["Comment"].value().toString()) 29);
>     }
>     else
>     {
>         strncpy(mp3.id3.title  info["id3v1.1"]["Title"]  .value().toString().local8Bit() 31);
>         strncpy(mp3.id3.artist info["id3v1.1"]["Artist"] .value().toString().local8Bit() 31);
>         strncpy(mp3.id3.album  info["id3v1.1"]["Album"]  .value().toString().local8Bit() 31);
>         strncpy(mp3.id3.year   info["id3v1.1"]["Date"]   .value().toString().local8Bit()  5);
>         strncpy(mp3.id3.commentinfo["id3v1.1"]["Comment"].value().toString().local8Bit() 29);
>     }


(Submitted via bugs.kde.org)

Comment 1 Charles Samuels 2002-09-20 20:45:50 UTC

Correct me if I'm wrong, but aren't ID3 tags supposed to be 
Latin-1? 
 
Perhaps try Vorbis, as they encode to utf8...

Comment 2 Scott Wheeler 2003-12-11 02:32:01 UTC

Yes, ID3v1 is limited to ISO-8859-1.  However I've got a work around for this explained in the duplicate bug report.  Please read the note that I wrote there...

*** This bug has been marked as a duplicate of 48604 ***