Author: agrundman
Date: Wed Sep 21 08:07:26 2011
New Revision: 9539

URL: http://svn.slimdevices.com/jive?rev=9539&view=rev
Log:
 r79780@andy-imac (orig r9537):  fmueller | 2011-09-21 05:00:20 -0400
 Bug: 16100 
 Description: Fallback alarm - fix to make it sound from the speaker even 
though headphones are plugged in  
 
 r79781@andy-imac (orig r9538):  fmueller | 2011-09-21 05:02:38 -0400
 Bug: 16100 
 Description: Fallback alarm - fix to make it sound from the speaker even 
though headphones are plugged in  
 

Modified:
    7.7/trunk/   (props changed)
    
7.7/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
    7.7/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua
    
7.7/trunk/squeezeplay/src/squeezeplay_baby/share/applets/SqueezeboxBaby/SqueezeboxBabyApplet.lua

Propchange: 7.7/trunk/
------------------------------------------------------------------------------
--- svk:merge (original)
+++ svk:merge Wed Sep 21 08:07:26 2011
@@ -15,7 +15,7 @@
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.4/trunk:8423
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.5/private-branches/7.5.3-rc:9282
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.5/trunk:9408
-bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.6/trunk:9533
+bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.6/trunk:9538
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/branches/7.0:2013
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/branches/SN:1083
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/branches/scrolling:1378

Modified: 
7.7/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
URL: 
http://svn.slimdevices.com/jive/7.7/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua?rev=9539&r1=9538&r2=9539&view=diff
==============================================================================
--- 
7.7/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
 (original)
+++ 
7.7/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
 Wed Sep 21 08:07:26 2011
@@ -446,6 +446,9 @@
        self.localPlayer:volumeLocal(self.alarmVolume)
        self.localPlayer:stop(true)
        self.alarmInProgress = 'rtc'
+       -- Bug 16100 - Sound comes from headphones although Radio is set to use 
speaker
+       -- We need to set a valid alarm state for the fallback case
+       self.localPlayer:setAlarmState("active_fallback")
        self.localPlayer:playFileInLoop(self.alarmTone)
        self.fadeInTimer = Timer(200, 
                        function ()
@@ -498,7 +501,9 @@
 
        log:warn('openAlarmWindow()', caller, ' ', 
self.localPlayer:isConnected())
 
-       self:_alarmThroughSpeakers()
+       -- Bug 16100 - Sound comes from headphones although Radio is set to use 
speaker
+       -- Calling _alarmThroughSpeakers() here is too early for the fallback 
alarm
+       --self:_alarmThroughSpeakers()
 
        -- if UI is controlling a different player, switch to the local player
        -- if notify_playerLoaded needs invocation prior to player change 
taking effect then refire openAlarmWindow() at that time
@@ -557,6 +562,10 @@
                log:error('openAlarmWindow: unknown caller')
        end
        
+       -- Bug 16100 - Sound comes from headphones although Radio is set to use 
speaker
+       -- Calling _alarmThroughSpeakers() here works for regular, fallback 
alarm and snooze
+       self:_alarmThroughSpeakers()
+
        if self.alarmWindow then
                return
        end

Modified: 7.7/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua
URL: 
http://svn.slimdevices.com/jive/7.7/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua?rev=9539&r1=9538&r2=9539&view=diff
==============================================================================
--- 7.7/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua (original)
+++ 7.7/trunk/squeezeplay/src/squeezeplay/share/jive/slim/Player.lua Wed Sep 21 
08:07:26 2011
@@ -1538,6 +1538,13 @@
        return self.alarmState
 end
 
+
+-- Bug 16100 - Sound comes from headphones although Radio is set to use speaker
+-- We need to be able to set the alarm state for fallback alarms
+function setAlarmState(self, state)
+       self.alarmState = state
+end
+
 -- getPlayMode returns nil|stop|play|pause
 --
 function getPlayMode(self)

Modified: 
7.7/trunk/squeezeplay/src/squeezeplay_baby/share/applets/SqueezeboxBaby/SqueezeboxBabyApplet.lua
URL: 
http://svn.slimdevices.com/jive/7.7/trunk/squeezeplay/src/squeezeplay_baby/share/applets/SqueezeboxBaby/SqueezeboxBabyApplet.lua?rev=9539&r1=9538&r2=9539&view=diff
==============================================================================
--- 
7.7/trunk/squeezeplay/src/squeezeplay_baby/share/applets/SqueezeboxBaby/SqueezeboxBabyApplet.lua
 (original)
+++ 
7.7/trunk/squeezeplay/src/squeezeplay_baby/share/applets/SqueezeboxBaby/SqueezeboxBabyApplet.lua
 Wed Sep 21 08:07:26 2011
@@ -989,8 +989,8 @@
 
        self.powerState = state
 
-       -- Bug 16100, only setEndpoint if alarm is not active
-       if self.player and (self.player:getAlarmState() == 'active' or 
self.player:getAlarmState() == 'snooze') then
+       -- Bug 16100, only setEndpoint if alarm is not in state: active, snooze 
or active_fallback
+       if self.player and (self.player:getAlarmState() == 'active' or 
self.player:getAlarmState() == 'snooze' or self.player:getAlarmState() == 
'active_fallback') then
                -- leave audio coming out speaker when alarm is active or in 
snooze (alarm forces output out speaker)
                log:info('Alarm either in progress or snooze, do not call 
_setEndpoint()')
        else

_______________________________________________
Jive-checkins mailing list
Jive-checkins@lists.slimdevices.com
http://lists.slimdevices.com/mailman/listinfo/jive-checkins

Reply via email to