Declare song using source

This commit is contained in:
Jonas Kvinge
2019-11-14 00:09:35 +01:00
parent c4f7054ca6
commit 1862e70628
6 changed files with 20 additions and 22 deletions

View File

@@ -350,7 +350,7 @@ SongList CollectionBackend::FindSongsInDirectory(int id) {
SongList ret; SongList ret;
while (q.next()) { while (q.next()) {
Song song; Song song(source_);
song.InitFromQuery(q, true); song.InitFromQuery(q, true);
ret << song; ret << song;
} }
@@ -698,7 +698,7 @@ SongList CollectionBackend::ExecCollectionQuery(CollectionQuery *query) {
SongList ret; SongList ret;
while (query->Next()) { while (query->Next()) {
Song song; Song song(source_);
song.InitFromQuery(*query, true); song.InitFromQuery(*query, true);
ret << song; ret << song;
} }
@@ -772,7 +772,7 @@ SongList CollectionBackend::GetSongsById(const QStringList &ids, QSqlDatabase &d
SongList ret; SongList ret;
while (q.next()) { while (q.next()) {
Song song; Song song(source_);
song.InitFromQuery(q, true); song.InitFromQuery(q, true);
ret << song; ret << song;
} }
@@ -787,7 +787,7 @@ Song CollectionBackend::GetSongByUrl(const QUrl &url, qint64 beginning) {
query.AddWhere("url", url.toString()); query.AddWhere("url", url.toString());
query.AddWhere("beginning", beginning); query.AddWhere("beginning", beginning);
Song song; Song song(source_);
if (ExecQuery(&query) && query.Next()) { if (ExecQuery(&query) && query.Next()) {
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
} }
@@ -804,7 +804,7 @@ SongList CollectionBackend::GetSongsByUrl(const QUrl &url) {
SongList songlist; SongList songlist;
if (ExecQuery(&query)) { if (ExecQuery(&query)) {
while (query.Next()) { while (query.Next()) {
Song song; Song song(source_);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
songlist << song; songlist << song;
} }
@@ -856,7 +856,7 @@ SongList CollectionBackend::GetSongsBySongId(const QStringList &song_ids, QSqlDa
SongList ret; SongList ret;
while (q.next()) { while (q.next()) {
Song song; Song song(source_);
song.InitFromQuery(q, true); song.InitFromQuery(q, true);
ret << song; ret << song;
} }
@@ -880,7 +880,7 @@ SongList CollectionBackend::GetCompilationSongs(const QString &album, const Quer
SongList ret; SongList ret;
while (query.Next()) { while (query.Next()) {
Song song; Song song(source_);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
ret << song; ret << song;
} }
@@ -967,7 +967,7 @@ void CollectionBackend::UpdateCompilations(QSqlQuery &find_song, QSqlQuery &upda
find_song.bindValue(":compilation_detected", int(!compilation_detected)); find_song.bindValue(":compilation_detected", int(!compilation_detected));
find_song.exec(); find_song.exec();
while (find_song.next()) { while (find_song.next()) {
Song song(Song::Source_Collection); Song song(source_);
song.InitFromQuery(find_song, true); song.InitFromQuery(find_song, true);
deleted_songs << song; deleted_songs << song;
song.set_compilation_detected(compilation_detected); song.set_compilation_detected(compilation_detected);
@@ -1102,7 +1102,7 @@ void CollectionBackend::UpdateManualAlbumArt(const QString &artist, const QStrin
SongList deleted_songs; SongList deleted_songs;
while (query.Next()) { while (query.Next()) {
Song song; Song song(source_);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
deleted_songs << song; deleted_songs << song;
} }
@@ -1136,7 +1136,7 @@ void CollectionBackend::UpdateManualAlbumArt(const QString &artist, const QStrin
SongList added_songs; SongList added_songs;
while (query.Next()) { while (query.Next()) {
Song song; Song song(source_);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
added_songs << song; added_songs << song;
} }
@@ -1164,7 +1164,7 @@ void CollectionBackend::ForceCompilation(const QString &album, const QList<QStri
if (!ExecQuery(&query)) return; if (!ExecQuery(&query)) return;
while (query.Next()) { while (query.Next()) {
Song song; Song song(source_);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
deleted_songs << song; deleted_songs << song;
} }
@@ -1187,7 +1187,7 @@ void CollectionBackend::ForceCompilation(const QString &album, const QList<QStri
if (!ExecQuery(&query)) return; if (!ExecQuery(&query)) return;
while (query.Next()) { while (query.Next()) {
Song song; Song song(source_);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
added_songs << song; added_songs << song;
} }

View File

@@ -167,7 +167,7 @@ SongLoader::Result SongLoader::LoadLocalPartial(const QString &filename) {
LoadLocalDirectory(filename); LoadLocalDirectory(filename);
return Success; return Success;
} }
Song song; Song song(Song::Source_LocalFile);
song.InitFromFilePartial(filename); song.InitFromFilePartial(filename);
if (song.is_valid()) { if (song.is_valid()) {
songs_ << song; songs_ << song;
@@ -228,7 +228,7 @@ SongLoader::Result SongLoader::LoadLocal(const QString &filename) {
if (collection_->ExecQuery(&query) && query.Next()) { if (collection_->ExecQuery(&query) && query.Next()) {
// We may have many results when the file has many sections // We may have many results when the file has many sections
do { do {
Song song; Song song(Song::Source_LocalFile);
song.InitFromQuery(query, true); song.InitFromQuery(query, true);
if (song.is_valid()) { if (song.is_valid()) {
@@ -289,7 +289,7 @@ SongLoader::Result SongLoader::LoadLocalAsync(const QString &filename) {
} }
// Assume it's just a normal file // Assume it's just a normal file
Song song; Song song(Song::Source_LocalFile);
song.InitFromFilePartial(filename); song.InitFromFilePartial(filename);
if (song.is_valid()) { if (song.is_valid()) {
songs_ << song; songs_ << song;

View File

@@ -107,10 +107,9 @@ void CddaSongLoader::LoadSongs() {
SongList songs; SongList songs;
for (int track_number = 1; track_number <= num_tracks; track_number++) { for (int track_number = 1; track_number <= num_tracks; track_number++) {
// Init song // Init song
Song song; Song song(Song::Source_CDDA);
song.set_id(track_number); song.set_id(track_number);
song.set_valid(true); song.set_valid(true);
song.set_source(Song::Source_CDDA);
song.set_filetype(Song::FileType_CDDA); song.set_filetype(Song::FileType_CDDA);
song.set_url(GetUrlFromTrack(track_number)); song.set_url(GetUrlFromTrack(track_number));
song.set_title(QString("Track %1").arg(track_number)); song.set_title(QString("Track %1").arg(track_number));
@@ -200,7 +199,7 @@ void CddaSongLoader::AudioCDTagsLoaded(const QString &artist, const QString &alb
if (results.empty()) return; if (results.empty()) return;
int track_number = 1; int track_number = 1;
for (const MusicBrainzClient::Result &ret : results) { for (const MusicBrainzClient::Result &ret : results) {
Song song; Song song(Song::Source_CDDA);
song.set_artist(artist); song.set_artist(artist);
song.set_album(album); song.set_album(album);
song.set_title(ret.title_); song.set_title(ret.title_);
@@ -208,7 +207,6 @@ void CddaSongLoader::AudioCDTagsLoaded(const QString &artist, const QString &alb
song.set_track(track_number); song.set_track(track_number);
song.set_year(ret.year_); song.set_year(ret.year_);
song.set_id(track_number); song.set_id(track_number);
song.set_source(Song::Source_CDDA);
song.set_filetype(Song::FileType_CDDA); song.set_filetype(Song::FileType_CDDA);
song.set_valid(true); song.set_valid(true);
// We need to set url: that's how playlist will find the correct item to update // We need to set url: that's how playlist will find the correct item to update

View File

@@ -876,7 +876,7 @@ void QobuzRequest::SongsReceived(QNetworkReply *reply, const qint64 artist_id_re
QJsonObject json_obj = value.toObject(); QJsonObject json_obj = value.toObject();
++songs_received; ++songs_received;
Song song; Song song(Song::Source_Qobuz);
ParseSong(song, json_obj, artist_id, album_id, album_artist, album, cover_url); ParseSong(song, json_obj, artist_id, album_id, album_artist, album, cover_url);
if (!song.is_valid()) continue; if (!song.is_valid()) continue;
if (song.disc() >= 2) multidisc = true; if (song.disc() >= 2) multidisc = true;

View File

@@ -435,7 +435,7 @@ void SubsonicRequest::AlbumSongsReplyReceived(QNetworkReply *reply, const qint64
QJsonObject json_obj = value.toObject(); QJsonObject json_obj = value.toObject();
++songs_received; ++songs_received;
Song song; Song song(Song::Source_Subsonic);
ParseSong(song, json_obj, artist_id, album_id, album_artist); ParseSong(song, json_obj, artist_id, album_id, album_artist);
if (!song.is_valid()) continue; if (!song.is_valid()) continue;
if (song.disc() >= 2) multidisc = true; if (song.disc() >= 2) multidisc = true;

View File

@@ -834,7 +834,7 @@ void TidalRequest::SongsReceived(QNetworkReply *reply, const qint64 artist_id, c
} }
++songs_received; ++songs_received;
Song song; Song song(Song::Source_Tidal);
ParseSong(song, json_obj, artist_id, album_id, album_artist); ParseSong(song, json_obj, artist_id, album_id, album_artist);
if (!song.is_valid()) continue; if (!song.is_valid()) continue;
if (song.disc() >= 2) multidisc = true; if (song.disc() >= 2) multidisc = true;