From 61aaefd65ca80fea246a7bc0461f5b095c59f885 Mon Sep 17 00:00:00 2001
From: Arnaud Bienner <arnaud.bienner@gmail.com>
Date: Mon, 26 Dec 2011 18:15:35 +0100
Subject: [PATCH] Magnatune's URL are already encoded: we shouldn't use QUrl
 constructor directly because some characters (like ') will be encoded twice
 (%27 then %2527). Fixes issue 2510

---
 src/internet/magnatuneservice.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/internet/magnatuneservice.cpp b/src/internet/magnatuneservice.cpp
index 7893743e2..7f492193e 100644
--- a/src/internet/magnatuneservice.cpp
+++ b/src/internet/magnatuneservice.cpp
@@ -221,7 +221,9 @@ Song MagnatuneService::ReadTrack(QXmlStreamReader& reader) {
       if (name == "cover_small")     song.set_art_automatic(value);
       if (name == "albumsku")        song.set_comment(value);
       if (name == "url") {
-        QUrl url(value);
+        QUrl url;
+        // Magnatune's URLs are already encoded
+        url.setEncodedUrl(value.toLocal8Bit());
         url.setScheme("magnatune");
         song.set_url(url);
       }