Generate sensible names for covers returned from Musicbrainz.
This commit is contained in:
parent
4d9e102bbd
commit
e5760815a1
|
@ -59,6 +59,8 @@ bool MusicbrainzCoverProvider::StartSearch(
|
||||||
SLOT(ReleaseSearchFinished(QNetworkReply*, int)),
|
SLOT(ReleaseSearchFinished(QNetworkReply*, int)),
|
||||||
reply,
|
reply,
|
||||||
id);
|
id);
|
||||||
|
|
||||||
|
cover_names_[id] = QString("%1 - %2").arg(artist, album);
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -99,12 +101,13 @@ void MusicbrainzCoverProvider::ImageCheckFinished(int id) {
|
||||||
replies.constBegin(), replies.constEnd(),
|
replies.constBegin(), replies.constEnd(),
|
||||||
boost::bind(&QNetworkReply::isFinished, _1));
|
boost::bind(&QNetworkReply::isFinished, _1));
|
||||||
if (finished_count == replies.size()) {
|
if (finished_count == replies.size()) {
|
||||||
|
QString cover_name = cover_names_.take(id);
|
||||||
QList<CoverSearchResult> results;
|
QList<CoverSearchResult> results;
|
||||||
foreach (QNetworkReply* reply, replies) {
|
foreach (QNetworkReply* reply, replies) {
|
||||||
reply->deleteLater();
|
reply->deleteLater();
|
||||||
if (reply->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt() < 400) {
|
if (reply->attribute(QNetworkRequest::HttpStatusCodeAttribute).toInt() < 400) {
|
||||||
CoverSearchResult result;
|
CoverSearchResult result;
|
||||||
result.description = "foobar";
|
result.description = cover_name;
|
||||||
result.image_url = reply->url();
|
result.image_url = reply->url();
|
||||||
results.append(result);
|
results.append(result);
|
||||||
}
|
}
|
||||||
|
|
|
@ -41,6 +41,7 @@ class MusicbrainzCoverProvider : public CoverProvider {
|
||||||
private:
|
private:
|
||||||
QNetworkAccessManager* network_;
|
QNetworkAccessManager* network_;
|
||||||
QMultiMap<int, QNetworkReply*> image_checks_;
|
QMultiMap<int, QNetworkReply*> image_checks_;
|
||||||
|
QMap<int, QString> cover_names_;
|
||||||
};
|
};
|
||||||
|
|
||||||
#endif // MUSICBRAINZCOVERPROVIDER_H
|
#endif // MUSICBRAINZCOVERPROVIDER_H
|
||||||
|
|
Loading…
Reference in New Issue