mirror/tomahawk
1
0
Fork 0
mirror of https://github.com/tomahawk-player/tomahawk.git synced 2025-02-22 19:05:05 +01:00
Code Issues Releases Wiki Activity

Don't repeat scriptcollection queries when an album has no tracks.

Browse Source 
This should never happen with sane data.
This commit is contained in:
Teo Mrnjavac 2013-04-04 12:44:52 +02:00
parent 3f48d1a3ad
commit a47e1043fc
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/libtomahawk/AlbumPlaylistInterface.cpp
View File

@ -43,6 +43,7 @@ AlbumPlaylistInterface::AlbumPlaylistInterface( Tomahawk::Album* album, Tomahawk
, m_mode( mode )
, m_collection( collection )
, m_album( QPointer< Tomahawk::Album >( album ) )
, m_lastQueryTimestamp( 0 )
{
}
@ -108,7 +109,7 @@ AlbumPlaylistInterface::setCurrentTrack( unsigned int albumpos )
QList< Tomahawk::query_ptr >
AlbumPlaylistInterface::tracks() const
{
if ( m_queries.isEmpty() && m_album )
if ( m_queries.isEmpty() && m_album && QDateTime::currentMSecsSinceEpoch() - m_lastQueryTimestamp > 10000 /*10s*/ )
{
if ( ( m_mode == Mixed || m_mode == InfoSystemMode ) && !m_infoSystemLoaded )
{
@ -131,6 +132,8 @@ AlbumPlaylistInterface::tracks() const
connect( Tomahawk::InfoSystem::InfoSystem::instance(),
SIGNAL( finished( QString ) ),
SLOT( infoSystemFinished( QString ) ) );
const_cast< int& >( m_lastQueryTimestamp ) = QDateTime::currentMSecsSinceEpoch();
}
else if ( m_mode == DatabaseMode && !m_databaseLoaded )
{
@ -153,6 +156,7 @@ AlbumPlaylistInterface::tracks() const
cmd->enqueue();
}
const_cast< int& >( m_lastQueryTimestamp ) = QDateTime::currentMSecsSinceEpoch();
}
}

2
src/libtomahawk/AlbumPlaylistInterface.h
View File

@ -82,6 +82,8 @@ private:
Tomahawk::collection_ptr m_collection;
QPointer< Tomahawk::Album > m_album;
int m_lastQueryTimestamp; //To prevent infinite loops on empty responses. This should not happen, but let's make sure.
};
}; // ns