Bugzilla – Bug 12760
Feature Request: Alarm Timeout behavior
Last modified: 2009-12-21 14:25:57 UTC
Currently, when an alarm activates then goes into timeout, the player stays in the ON state. This enhancement is to make a change so that when the alarm times out, it then goes into Idle mode.
Is this from a customer, Julius?
Adding comment for customer: As an enhancement, I suggest that users be given settings with the option to switch Squeezebox to power off or pause after the alarm cycle times out. I assume most people power it off manually and some may use the sleep mode to let the unit time out and power off. The enhancement would have the same result. A new setting would enable the unit to power off instead of going into idle or pause.
Max: Does this sound reasonable and easy to do?
Doing this would negate the need for Bug 9569, correct? I'm all for one of these fixes getting into 7.4
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Created attachment 5820 [details] Return to previous power state This patch stores the previous power state and restores it when the alarm is stopped. This works in much the same way as the pre-alarm volume is saved and restored. If a pref is absolutely required for this, it's easy to add a per-alarm pref to go along with it. Seems sensible just to leave it as automatic since that is what is done with volume.
Created attachment 5829 [details] fix a typo
Pat, I don't have a clear feeling about whether you think a patch like this should be reviewed and committed at this time. It is for an alarm-related bug, which would be good to have fixed for ba... I mean Squeezebox Radio.
Ben: can you review the attached patch? maybe implement for 7.4.1
Finally getting back to 7.4.x after clearing my 7.5.0 P1 list. This seems simple and so I'll make it a P1.
== Auto-comment from SVN commit #29665 to the slim repo by bklaas == == https://svn.slimdevices.com/slim?view=revision&revision=29665 == Fixed Bug: 12760 Fixed Bug: 9569 Desription: Patch from KDF (better late than never). Grab power condition before alarm so we can return later on a timeout.