playlist: do not move both tree and non-tree items + restrict move action
authorJakob Leben <jakob.leben@gmail.com>
Sun, 9 Aug 2009 22:04:59 +0000 (00:04 +0200)
committerRémi Denis-Courmont <remi@remlab.net>
Thu, 13 Aug 2009 15:04:39 +0000 (18:04 +0300)
commita2d9bd0a079e288e89224e39ae85e3b7c3923963
treeeaed9d55a16acde73b98a888dddea3481d30ca58
parent953d13ba71b1e2df067e3f10b48b3f456fedcc13
playlist: do not move both tree and non-tree items + restrict move action

Items are stored twice, once in a tree structure and once in a one-level list.
This patch removes attempt of the playlist_TreeMove() function to move both tree and
one-level instances of an item.

Firstly the reason is that function was not effective in this attempt.
And secondly, the attempt itself doesn't make sense in some cases: you can always
map moves within the tree to moves within one-level, but you can't always do it
the other way around, it is in most cases ambiguous.

Moreover, this patch restricts moves in the tree to within the present parent of the
item being moved. Again, in most cases, moving an item out of its parent or into another
parent is meaningless.

Signed-off-by: Rémi Denis-Courmont <remi@remlab.net>
src/playlist/item.c