Bug 513 - Minor playlist textual bug
: Minor playlist textual bug
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Playlists
: 5.x or older
: All All
: P2 normal (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-08-25 12:02 UTC by Chris Harris
Modified: 2008-12-18 11:53 UTC (History)
0 users

See Also:
Category: ---


Attachments
reindex if playlist is shorter than current index (209 bytes, patch)
2004-08-26 13:27 UTC, KDF
Details | Diff
another option (1007 bytes, patch)
2004-08-26 22:06 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Harris 2004-08-25 12:02:52 UTC
If you create a playlist whilst the server is stopped you can hit 'Now Playing'
and browse through the list.

If you select, say, song (3 of 8) and hit 'add' it will delete the song you're
currently viewing from the playlist and show you the new third song.

As you continue to delete the songs it goes:
(3 of 7)
(3 of 6)
(3 of 5)
(3 of 4)
(3 of 3)
(3 of 2)
(3 of 1)

You can see the problem!
Comment 1 KDF 2004-08-26 13:27:55 UTC
Created attachment 111 [details]
reindex if playlist is shorter than current index

This checks against the playlist length and will reindex the song if it has
become a position beyond the song count.  

note: when I tested this, deleting a song from teh playlist would then return
to the 'current' song even if stopped, not the new song 3.
Comment 2 KDF 2004-08-26 14:00:36 UTC
the attached patch should also partly solve bug505, which is nearly a dupe of this.
Comment 3 KDF 2004-08-26 22:06:26 UTC
Created attachment 112 [details]
another option

doing the reindex inside browseplaylistindex can cause an odd loop when
entering playlist mode.  doing the reindex as part of the lines should solve
the problem without causing another warning.
Comment 4 Blackketter Dean 2004-09-04 08:29:21 UTC
Looks good.  Go for it.
Comment 5 Chris Owens 2006-06-16 14:40:30 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.
Comment 6 Chris Owens 2008-12-18 11:53:08 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.