Bug 16947 – Playlists are missing "Delete Playlist" and "Add to Favorites" items

Bug 16947 - Playlists are missing "Delete Playlist" and "Add to Favorites" items

Summary:

Playlists are missing "Delete Playlist" and "Add to Favorites" items

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Playlists
Version:	7.6.0
Platform:	Macintosh Debian Linux

Importance:	P3 normal (vote)
Target Milestone:	7.6.0
Assigned To:	Ben Klaas

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2011-02-11 02:44 UTC by Michael Herger
Modified:	2011-04-29 16:44 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Michael Herger 2011-02-11 02:44:05 UTC

Ben - your change in response to bug 14360 broke this. The count is NOT up to date at this point. Thus while the query does return all the data, the count would be wrong. Eg. take a playlist with five items: it would return seven, but the count would still be 5.

Reverting your change 31807 does fix this issue...

Comment 1 SVN Bot 2011-02-14 13:04:26 UTC

 == Auto-comment from SVN commit #31908 to the slim repo by bklaas ==
 == http://svn.slimdevices.com/slim?view=revision&revision=31908 ==

Fixed Bug: 16947
Bug: 14360
Description: increment $totalCount by two for all chunks in response
this allows for the special items delete playlist & save to favorites to exist at the end of the list, while not screwing up the SP menu rendering with conflicting counts between the last chunk and all other chunks

Comment 2 Paul Chandler 2011-04-29 16:44:02 UTC

I can now get the context menu for a playlist and it includes 'Save To Favorites' and Delete Playlist

7.6.0 r32376