Bug 9114 – A non-repeatable alarm becomes repeatable upon SC restart

Bug 9114 - A non-repeatable alarm becomes repeatable upon SC restart

Summary:

A non-repeatable alarm becomes repeatable upon SC restart

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	7.2
Platform:	PC Windows XP

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

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-08-11 15:24 UTC by Philip Meyer
Modified:	2009-09-08 09:28 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
Fix (885 bytes, patch) 2008-08-12 01:34 UTC, Max Spicer	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 Philip Meyer 2008-08-11 15:24:16 UTC

In the web UI:

1. Configured an alarm to fire once (Repeat Alarm=No)
2. Waited for the alarm to go off.  Turned off the alarm.
3. Refreshed the WebUI player alarm settings page.  The alarm was now "Off", and Repeat Alarm was still set to "No".
4. Restarted SqueezeCenter.
5. Revisited the player alarm settings page in the WebUI.  The alarm was still "Off", but Repeat Alarm had changed to "Yes".

Comment 1 Blackketter Dean 2008-08-11 16:39:12 UTC

Max: is this a dup?

Comment 2 Philip Meyer 2008-08-11 17:17:38 UTC

Not a dup - from a forum thread, Max asked me to raise the bug if I could recreate it with his latest checkin.

Comment 3 Max Spicer 2008-08-12 01:24:35 UTC

Confirmed.  Alarm remains disabled but its repeat state is set back to on.  Good catch!  Not only had I previously forgotten to save the repeat state (bug 9075), it turns out I've also forgotten to load it.  Trivial fix, which I'll get in later today.  Do I still need permission to commit the fix after Dean targeted this to 7.2?  Will hold off until I hear back.

Comment 4 Max Spicer 2008-08-12 01:34:03 UTC

Created attachment 3782 [details]
Fix

The following (trivial) patch fixes the problem.

Comment 5 Blackketter Dean 2008-08-12 06:46:15 UTC

please commit!

Thx!

Comment 6 Max Spicer 2008-08-12 07:21:42 UTC

Fixed in change 22539.

Comment 7 James Richardson 2008-08-27 11:20:25 UTC

Verified fixed in
SqueezeCenter 7.2-22900

Comment 8 James Richardson 2008-12-15 12:34:50 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 9 Chris Owens 2009-07-31 10:27:06 UTC

Reduce number of active targets for SC