Bug 9672 – Alarm playlist param for updates should be 'url' not 'playlisturl'

Bug 9672 - Alarm playlist param for updates should be 'url' not 'playlisturl'

Summary:

Alarm playlist param for updates should be 'url' not 'playlisturl'

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	CLI
Version:	7.2
Platform:	All All

Importance:	-- enhancement (vote)
Target Milestone:	7.x
Assigned To:	Max Spicer

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-10-08 01:09 UTC by Max Spicer
Modified:	2009-07-31 10:30 UTC (History)
CC List:	1 user (show)

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 Max Spicer 2008-10-08 01:09:43 UTC

When you query alarms using the cli 'alarms' command, the alarm playlist is returned via the 'url' parameter.  However, when you update the alarm playlist using the 'alarm cmd:update ...' command, the playlist has to be set using the 'playlisturl' parameter.  I don't know why it's this way, but we should also support updating playlists via the 'url' parameter if we can.

Comment 1 Max Spicer 2008-10-08 01:43:33 UTC

Committed to 7.3 as change 23459.  'url' is now accepted for updates but 'playlisturl' is also supported.  'url' wins over 'playlisturl' in case both are specified.

I haven't tested this change.  Barry, could you have a go?

Comment 2 Barry Gordon 2008-10-08 08:40:52 UTC

I will test this as soon as as I install 7.3 and report back.  Thanks for the compatability! but I have the code in my client to have version based command issuance, so for 7.3 I will make it url and for and 7.2.1 I will make it playlisturl I no longer handle alarm clocks pre 7.2.1. this will handle deprecation of playlisturl if that ever happens

Comment 3 James Richardson 2008-10-10 15:26:15 UTC

Targeting Enhancement bugs

Comment 4 Max Spicer 2008-10-11 01:39:06 UTC

This is already in 7.3 and just waiting confirmation that it works.

Comment 5 Michael Herger 2008-11-04 03:01:56 UTC

Let's assume this fixed. QA will then verify it. Thanks Max!

Comment 6 James Richardson 2008-12-15 12:08:10 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Comment 7 Chris Owens 2009-07-31 10:30:47 UTC

Reduce number of active targets for SC