Author: bklaas
Date: Thu Jan 20 07:34:00 2011
New Revision: 9291
URL: http://svn.slimdevices.com/jive?rev=9291&view=rev
Log:
Fixed Bug: 15515
Fixed Bug: 16673
Description: use AlarmSnooze's RTC fallback alarm timer state (running or not)
to govern whether an alarm icon is present on the iconbar
This should fix all bugs that involve the iconbar to display the wrong thing
for a upcoming-within-24h alarm
Modified:
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
7.6/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua
Modified:
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
URL:
http://svn.slimdevices.com/jive/7.6/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua?rev=9291&r1=9290&r2=9291&view=diff
==============================================================================
---
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
(original)
+++
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
Thu Jan 20 07:34:00 2011
@@ -678,7 +678,6 @@
function _alarmOff(self, stopStream)
if self.alarmInProgress == 'rtc' then
self:_silenceFallbackAlarm()
- iconbar:setAlarm('OFF')
log:warn('_alarmOff: RTC alarm canceled')
else
if self.localPlayer:isConnected() then
@@ -717,6 +716,9 @@
log:warn('_stopTimer: stopping WOL timer')
self.wakeOnLanTimer:stop()
end
+
+ -- no RTC timer means no alarm, so let's go ahead and remove our alarm
icon
+ iconbar:setAlarm('OFF')
end
@@ -786,6 +788,7 @@
end
self.RTCAlarmTimer:start()
+ iconbar:setAlarm('ON')
end
Modified: 7.6/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua
URL:
http://svn.slimdevices.com/jive/7.6/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua?rev=9291&r1=9290&r2=9291&view=diff
==============================================================================
--- 7.6/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua (original)
+++ 7.6/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua Thu Jan 20
07:34:00 2011
@@ -1151,11 +1151,7 @@
iconbar:setSleep('OFF')
end
- if self:getAlarmState() == 'set' or self:getAlarmState() ==
'snooze' then
- iconbar:setAlarm('ON')
- else
- iconbar:setAlarm('OFF')
- end
+ -- alarm icon in iconbar is set directly via AlarmSnoozeApplet
iconbar:setRepeat(0)
_______________________________________________
Jive-checkins mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/jive-checkins