Summary: | Amarok fails to copy mp3 tracks from Local Collection to iPod in de locale saying MP3 format is not supported | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | Adrian Batzill <linux> |
Component: | Collections/iPod iPhone | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | aacid, linux, matej |
Priority: | NOR | ||
Version: | 2.6.0 | ||
Target Milestone: | 2.7 | ||
Platform: | Ubuntu | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/amarok/fd3ae2019e5fee4400de996daaef472205e35d85 | Version Fixed In: | 2.7 |
Description
Adrian Batzill
2012-08-14 13:20:15 UTC
(In reply to comment #0) > Hey there, > I just tried amarok 2.6 for the improved IPod support. However, it doesn't > work at all for me. Hi Adrian, thanks for your report. > Some of the errors and messages below might be a bit different in the > english version. I'm translating freely from german. > Nano Video (4th Gen.) family uses SysInfoExtended file to generate correct > database checksum > Ipod Name: kos Nanopod (yes, there is a space in there. Could this possibly > cause the issue?) No, even crazy iPod names should be handled just fine. > So everytime I try to copy files (mp3) to the IPod, I get this error (again, > translated freely from german): > The transfer of some Songs failed. > Upon clicking on the "Details" button, it says: > 1 Song could not be copied, because it is not playable - the MP3 format is > not supported. While this seems silly, I think the problem is that the "MP3" is upper-cased here. (if this string is carbon-copy) Can you confirm it by clicking the "Edit Track Details" of the track that fails to copy, what exactly does it's Format show? The thing is that all tracks should return lowercased format, but it seems that we're accidentally translating more that we thought since Albert Astals Cid's commit that fixed another thing ( http://commits.kde.org/amarok/6f6161417c83c3841b2814742cf72ca0443d6edf ) and Germain translation perhaps translates "mp3" to "MP3". Albert, could that be? Adrian, would you be able to build from sources [1]? The patch will be simple fix to FileTypeSupport::toString() to translate only what should be translated, but now it won't unfortunately make it to 2.6.0. And an obvious work-around would be to switch to non-de locale using Help -> Switch Application Language. [1] http://blogs.fsfe.org/myriam/2009/09/compiling-amarok-from-git-locally-full-summary/ Hey there, I can't compile from source right now, sorry. However, your guess was absolutely right! The format is shown as MP3 instead of mp3. Changing the language to english solved the problem. Yes, it can be, sorry my patch created this untought consequence, documenting what we discussed on irc. We can do the quick fix that is going back to only translating the "Other" or the "correct" fix that is investigating all the calls to that function and deciding if it has to be a visible string or not and then creating a new function that returns the non translated string for the cases it is not a visible string. Git commit fd3ae2019e5fee4400de996daaef472205e35d85 by Matěj Laitl. Committed on 15/08/2012 at 01:47. Pushed by laitl into branch 'master'. FileType: don't translate all file types, just the "Other" This fixes a bug where tracks cannot be copied to iPods in de locale because mp3 is apparently translated to MP3 there and IpodCollection compares it case-sensitively against a list of supported file types, which fails. For easier back-portability, this patch includes only the actual fix, second part with misc changes (like ChangeLog and docstrings) will follow. FIXED-IN: 2.7 M +3 -3 shared/FileType.cpp http://commits.kde.org/amarok/fd3ae2019e5fee4400de996daaef472205e35d85 Git commit bf949a8825d954ec29505659c69e14ecbb7e2ce8 by Matěj Laitl. Committed on 15/08/2012 at 01:55. Pushed by laitl into branch 'master'. ChangeLog for previous commit ...this time it is intentionally split. M +1 -0 ChangeLog http://commits.kde.org/amarok/bf949a8825d954ec29505659c69e14ecbb7e2ce8 Git commit 8057ab648444ffcbb8f02df42c0e96b4deb972e4 by Matěj Laitl. Committed on 15/08/2012 at 02:03. Pushed by laitl into branch 'master'. FileType: document pitiful situation wrt i18n, sketch solution Also document other Amarok::FileType methods. CCMAIL: Albert Astals Cid <aacid@kde.org> M +19 -0 shared/FileType.h M +6 -1 src/core/meta/Meta.h http://commits.kde.org/amarok/8057ab648444ffcbb8f02df42c0e96b4deb972e4 |