From 7ac798c143b7388ddd4eeedcbcba45a2bb6802ab Mon Sep 17 00:00:00 2001 From: Christian Muehlhaeuser Date: Wed, 14 Nov 2012 03:36:04 +0100 Subject: [PATCH] * Properly implemented FlexibleView's jumpToCurrentTrack. --- src/libtomahawk/playlist/FlexibleView.cpp | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/libtomahawk/playlist/FlexibleView.cpp b/src/libtomahawk/playlist/FlexibleView.cpp index b668398ae..641c6de63 100644 --- a/src/libtomahawk/playlist/FlexibleView.cpp +++ b/src/libtomahawk/playlist/FlexibleView.cpp @@ -268,10 +268,16 @@ FlexibleView::pixmap() const bool FlexibleView::jumpToCurrentTrack() { - m_trackView->jumpToCurrentTrack(); - m_detailedView->jumpToCurrentTrack(); - m_gridView->jumpToCurrentTrack(); - return true; + tDebug() << Q_FUNC_INFO; + + bool b = false; + + // note: the order of comparison is important here, if we'd write "b || foo" then foo will not be executed if b is already true! + b = m_trackView->jumpToCurrentTrack() || b; + b = m_detailedView->jumpToCurrentTrack() || b; + b = m_gridView->jumpToCurrentTrack() || b; + + return b; }