diff --git a/src/libtomahawk/database/DatabaseCommand_DeleteDynamicPlaylist.cpp b/src/libtomahawk/database/DatabaseCommand_DeleteDynamicPlaylist.cpp index 354202c72..d2c99bcbc 100644 --- a/src/libtomahawk/database/DatabaseCommand_DeleteDynamicPlaylist.cpp +++ b/src/libtomahawk/database/DatabaseCommand_DeleteDynamicPlaylist.cpp @@ -50,18 +50,14 @@ DatabaseCommand_DeleteDynamicPlaylist::exec( DatabaseImpl* lib ) void DatabaseCommand_DeleteDynamicPlaylist::postCommitHook() { - qDebug() << Q_FUNC_INFO << "Reporting:" << m_playlistguid; + tDebug( LOGVERBOSE ) << Q_FUNC_INFO << "Reporting:" << m_playlistguid; if ( !source() || !source()->dbCollection() ) { Q_ASSERT( false ); return; } - // we arent sure which it is, but it can't be more than one. so try both - dynplaylist_ptr playlist = source()->dbCollection()->autoPlaylist( m_playlistguid ); - if ( !playlist ) - playlist = source()->dbCollection()->station( m_playlistguid ); - + dynplaylist_ptr playlist = DynamicPlaylist::get( m_playlistguid ); if ( playlist ) { playlist->reportDeleted( playlist ); diff --git a/src/libtomahawk/database/DatabaseCommand_RenamePlaylist.cpp b/src/libtomahawk/database/DatabaseCommand_RenamePlaylist.cpp index a5985001d..35050d725 100644 --- a/src/libtomahawk/database/DatabaseCommand_RenamePlaylist.cpp +++ b/src/libtomahawk/database/DatabaseCommand_RenamePlaylist.cpp @@ -58,18 +58,14 @@ DatabaseCommand_RenamePlaylist::exec( DatabaseImpl* lib ) void DatabaseCommand_RenamePlaylist::postCommitHook() { - playlist_ptr playlist = source()->dbCollection()->playlist( m_playlistguid ); - // fallback, check for auto and stations too - if( playlist.isNull() ) - playlist = source()->dbCollection()->autoPlaylist( m_playlistguid ); - if( playlist.isNull() ) - playlist = source()->dbCollection()->station( m_playlistguid ); - + playlist_ptr playlist = Playlist::get( m_playlistguid ); Q_ASSERT( !playlist.isNull() ); + if ( !playlist ) + return; - qDebug() << "Renaming old playlist" << playlist->title() << "to" << m_playlistTitle << m_playlistguid; + tDebug() << "Renaming playlist" << playlist->title() << "to" << m_playlistTitle << m_playlistguid; playlist->setTitle( m_playlistTitle ); - if( source()->isLocal() ) + if ( source()->isLocal() ) Servent::instance()->triggerDBSync(); } diff --git a/src/libtomahawk/widgets/RecentPlaylistsModel.cpp b/src/libtomahawk/widgets/RecentPlaylistsModel.cpp index d8d6120fc..1589510fa 100644 --- a/src/libtomahawk/widgets/RecentPlaylistsModel.cpp +++ b/src/libtomahawk/widgets/RecentPlaylistsModel.cpp @@ -95,16 +95,7 @@ RecentPlaylistsModel::playlistsLoaded( const QListget( plPair.first ); - if ( !source ) - continue; - - playlist_ptr pl = source->dbCollection()->playlist( plPair.second ); - if ( !pl ) - pl = source->dbCollection()->autoPlaylist( plPair.second ); - if ( !pl ) - pl = source->dbCollection()->station( plPair.second ); - + const playlist_ptr& pl = Playlist::get( plPair.second ); if ( !pl ) { tDebug() << "ERROR: Found a playlist that is not associated with any source:" << plPair.first << plPair.second; diff --git a/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.cpp b/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.cpp index c6b442219..df0ffc51d 100644 --- a/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.cpp +++ b/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.cpp @@ -227,18 +227,10 @@ RecentlyPlayedPlaylistsModel::rowCount( const QModelIndex& ) const void -RecentlyPlayedPlaylistsModel::plAdded( const QString& plguid, int sId ) +RecentlyPlayedPlaylistsModel::plAdded( const QString& plguid, int sourceId ) { - source_ptr source = SourceList::instance()->get( sId ); - if ( !source ) - return; - - playlist_ptr pl = source->dbCollection()->playlist( plguid ); - if ( !pl ) - pl = source->dbCollection()->autoPlaylist( plguid ); - if ( !pl ) - pl = source->dbCollection()->station( plguid ); - + Q_UNUSED( sourceId ); + const playlist_ptr& pl = Playlist::get( plguid ); if ( !pl ) return; diff --git a/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.h b/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.h index a37bbdc35..e719b7998 100644 --- a/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.h +++ b/src/libtomahawk/widgets/RecentlyPlayedPlaylistsModel.h @@ -54,7 +54,7 @@ private slots: void onPlaylistsRemoved( QList ); void loadFromSettings(); - void plAdded( const QString& plid, int sid ); + void plAdded( const QString& plguid, int sourceId ); void playlistRevisionLoaded(); private: