[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 123142] Add tempo-field to metadata
From:       Alf B "Lervåg" <alfborge () gmail ! com>
Date:       2006-05-03 19:31:33
Message-ID: 20060503193133.10628.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=123142         
alfborge gmail com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #15714|0                           |1
        is obsolete|                            |



------- Additional Comments From alfborge gmail com  2006-05-03 21:31 -------
Created an attachment (id=15895)
 --> (http://bugs.kde.org/attachment.cgi?id=15895&action=view)
New patch that works with smart playlists

> - dragging smart playlists to the current playlist

Fixed.

> - transferring smart playlists to media devices

Not tested, but should work.

> (This is because many queries don't specify the fields from the tags table
they are interested in, but they just assume a certain number and order for
them.)
>  In general, watch out for the magic number '12' in the source code!

First part of the paranthesis doesn't seem to be true anymore, but the latter
was correct.  The magic number was '11' though, but it's 12 now. :)

> In addition, smart & dynamic playlists should support matches for BPM, too.

Smart playlists support matches for BPM now.  Don't understand what has to be
done with dynamic playlists to support matches for BPM.  Anyways, let me know
if there is anything else that needs to be added.

Finally, please look at the changes in the dcop and decide wether the calls to
bundle.save() are redundant or not.
[prev in list] [next in list] [prev in thread] [next in thread]