BPM/Tempo field

Alf B Lervaag alfborge at stud.ntnu.no
Wed May 31 23:25:50 UTC 2006


Alf B Lervaag wrote:
> Just noticed a bug in my patch.  Forgot to update the getValueType
> method, so the smartplaylisteditor didn't work properly.

While we're at it, I just fixed another bug that broke dynamic playlists
from smartplaylists.  Sorry for the noise.  Didn't test smart playlists
and dynamic playlists properly before I posted the original patch.

Updated the patch at bugs.kde.org as well.  Now everything seems to work
for me, so if anyone could please test it I'd be a very happy user.

--- amarok-old/src/playlist.cpp 2006-06-01 01:13:50.000000000 +0200
+++ amarok/src/playlist.cpp     2006-06-01 01:09:43.000000000 +0200
@@ -745,7 +745,7 @@

         if ( !sp->query().isEmpty() ) {
             //We have to filter all the un-needed results from query(
             //sql )
-            for (uint x=12; x < queryResult.count() ; x += 14)
+            for (uint x=13; x < queryResult.count() ; x += 15)
                 items << queryResult[x];
         } else {
             items = queryResult;

-- 
Alf



More information about the Amarok mailing list