From 595eec787809258de12b59eb6d5d22b71ed972ad Mon Sep 17 00:00:00 2001 From: Benjamin Sproule Date: Thu, 1 Feb 2018 11:38:25 +0000 Subject: [PATCH] Removed filter for audio/folder OneDrive items as OneDrive doesn't count many audio file types as audio (e.g. flac) (#5977) --- src/internet/skydrive/skydriveservice.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/internet/skydrive/skydriveservice.cpp b/src/internet/skydrive/skydriveservice.cpp index e32d22e25..5ecefd17f 100644 --- a/src/internet/skydrive/skydriveservice.cpp +++ b/src/internet/skydrive/skydriveservice.cpp @@ -125,7 +125,6 @@ void SkydriveService::FetchUserInfoFinished(QNetworkReply* reply) { void SkydriveService::ListFiles(const QString& folder) { QUrl url(QString(kSkydriveBase) + folder + "/files"); - url.addQueryItem("filter", "audio,folders"); QNetworkRequest request(url); AddAuthorizationHeader(&request); @@ -142,7 +141,9 @@ void SkydriveService::ListFilesFinished(QNetworkReply* reply) { QVariantList files = response["data"].toList(); for (const QVariant& f : files) { QVariantMap file = f.toMap(); - if (file["type"].toString() == "audio") { + if (file["type"].toString() == "folder") { + ListFiles(file["id"].toString()); + } else { QString mime_type = GuessMimeTypeForFile(file["name"].toString()); QUrl url; url.setScheme("skydrive"); @@ -161,8 +162,6 @@ void SkydriveService::ListFilesFinished(QNetworkReply* reply) { // Fortunately, just changing the scheme to HTTP works. download_url.setScheme("http"); MaybeAddFileToDatabase(song, mime_type, download_url, QString::null); - } else if (file["type"].toString() == "folder") { - ListFiles(file["id"].toString()); } } }