Bug 1597 – Add current selected playlist to alarm clock selection list

Bug 1597 - Add current selected playlist to alarm clock selection list

Summary:

Add current selected playlist to alarm clock selection list

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Playlists
Version:	unspecified
Platform:	All All

Importance:	P2 enhancement (vote)
Target Milestone:	---
Assigned To:	KDF

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2005-05-23 03:02 UTC by Harald Wilhelm
Modified:	2009-09-08 09:20 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
current and random playlist selection (playerUI for now) (2.24 KB, patch) 2005-09-05 12:40 UTC, KDF	Details \| Diff
View All 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 Harald Wilhelm 2005-05-23 03:02:33 UTC

Hi,

if I loop thru the playlist for the alarm clock, the current selection of the
player itself is missing.

The current selection is stored on disk and is named after the players IP. A
lookup in the conf would come up with the real name of the player.

Add this name to the selection list for the playlists.

As a result we can use the collection that we heard last in the morning as well.

Thanks for voting ;-)

Comment 1 KDF 2005-05-23 07:12:52 UTC

already a feature in the plugin version.

Comment 2 Gerd Boerrigter 2005-09-04 11:42:06 UTC

Now that there are weekday alarms and random mix is part of slimserver, it would 
be cool, if 'current' and 'random' would appear in the selection list for the 
alarm playlist.

Comment 3 KDF 2005-09-04 14:36:57 UTC

Dean, any thoughts?

Comment 4 KDF 2005-09-04 15:22:23 UTC

fyi.  in the current setup, if you use the web UI to se the playlist, there is a
blank option.  If you use this, then the server will fallback to using whatever
tracks are already set as the current playlist.  This feature has always existed
for the case of having no playlist setting, but i've just quickly patched it to
handle the blank setting.

Further UI changes would have to be approved by dean.

Comment 5 Blackketter Dean 2005-09-05 07:44:22 UTC

Sure, make it "Current Playlist", "Random Artist", "Random Albums" or "Random Songs"

Comment 6 KDF 2005-09-05 10:21:03 UTC

okies, I'll have a go at it for 6.2

Comment 7 KDF 2005-09-05 12:40:22 UTC

Created attachment 800 [details]
current and random playlist selection (playerUI for now)

I haven't decided how to handle the special playlists in the web setup just
yet.  Feedback on this would be useful before going into that part in Setup.pm

Comment 8 Blackketter Dean 2005-09-05 17:21:02 UTC

Looks good to me.  One concern: if somebody disables the random play plugin (I can't imagine why?) 
this might cause a crash, no?

Comment 9 KDF 2005-09-05 17:45:19 UTC

The patch checks that random isn't disabled when presenting the options.  It also checks before calling 
playRandom (or did I attach the wrong version of the patch again?)

Comment 10 Blackketter Dean 2005-09-05 17:51:28 UTC

My bad, I missed that.  Go for it!

Comment 11 KDF 2005-09-05 19:18:21 UTC

committed to trunk at change 4173, leaving open to deal with web UI.

Comment 12 KDF 2005-09-05 23:02:33 UTC

web UI committed at change 4175.
we may want to eventually move the 'specialPlaylists' to a more central part of
the code and maybe even give them a url format to be useful everywhere that any
playlist might come up.

Comment 13 KDF 2005-09-06 10:45:07 UTC

present in Spet 6 build of 6.2  re-open or create a new bug if issues come up as
a result, or remain unfixed.