Do not set cover for Grooveshark songs if we don't have one.
This wrongly gives the impression that these songs have a cover
This commit is contained in:
parent
45c172f3fb
commit
3338174a1f
|
@ -1452,10 +1452,12 @@ Song GroovesharkService::ExtractSong(const QVariantMap& result_song) {
|
||||||
QString artist_name = result_song["ArtistName"].toString();
|
QString artist_name = result_song["ArtistName"].toString();
|
||||||
int album_id = result_song["AlbumID"].toInt();
|
int album_id = result_song["AlbumID"].toInt();
|
||||||
QString album_name = result_song["AlbumName"].toString();
|
QString album_name = result_song["AlbumName"].toString();
|
||||||
QString cover = result_song["CoverArtFilename"].toString();
|
|
||||||
qint64 duration = result_song["EstimateDuration"].toInt() * kNsecPerSec;
|
qint64 duration = result_song["EstimateDuration"].toInt() * kNsecPerSec;
|
||||||
song.Init(song_name, artist_name, album_name, duration);
|
song.Init(song_name, artist_name, album_name, duration);
|
||||||
song.set_art_automatic(QString(kUrlCover) + cover);
|
QVariant cover = result_song["CoverArtFilename"];
|
||||||
|
if (cover.isValid()) {
|
||||||
|
song.set_art_automatic(QString(kUrlCover) + cover.toString());
|
||||||
|
}
|
||||||
QVariant track_number = result_song["TrackNum"];
|
QVariant track_number = result_song["TrackNum"];
|
||||||
if (track_number.isValid()) {
|
if (track_number.isValid()) {
|
||||||
song.set_track(track_number.toInt());
|
song.set_track(track_number.toInt());
|
||||||
|
|
Loading…
Reference in New Issue