mirror/tomahawk
1
0
Fork 0
mirror of https://github.com/tomahawk-player/tomahawk.git synced 2025-07-31 11:20:22 +02:00
Code Issues Releases Wiki Activity

Don't load xspf tracks that have no title or artist, we needs them.

Browse Source 
Show a warning when failing to load tracks from an XSPF.
This commit is contained in:
Leo Franchi
2011-03-13 15:28:08 -04:00
parent 7d13c56081
commit 736e0800de
1 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
src/libtomahawk/utils/xspfloader.cpp
View File

@@ -99,6 +99,7 @@ XSPFLoader::gotBody()
if ( m_title.isEmpty() ) if ( m_title.isEmpty() )
m_title = tr( "New Playlist" ); m_title = tr( "New Playlist" );
bool shownError = false;
for ( unsigned int i = 0; i < tracklist.length(); i++ ) for ( unsigned int i = 0; i < tracklist.length(); i++ )
{ {
QDomNode e = tracklist.at( i ); QDomNode e = tracklist.at( i );
@@ -118,6 +119,14 @@ XSPFLoader::gotBody()
track = n.text(); track = n.text();
} }
} }
if( artist.isEmpty() || track.isEmpty() ) {
if( !shownError ) {
QMessageBox::warning( 0, tr( "Failed to save tracks" ), tr( "Some tracks in the playlist do not contain an artist and a title. They will be ignored." ), QMessageBox::Ok );
shownError = true;
}
continue;
}
plentry_ptr p( new PlaylistEntry ); plentry_ptr p( new PlaylistEntry );
p->setGuid( uuid() ); p->setGuid( uuid() );