Author: bklaas
Date: Tue Jun  1 12:47:14 2010
New Revision: 8838

URL: http://svn.slimdevices.com/jive?rev=8838&view=rev
Log:
 r39...@daddymac-520 (orig r8836):  agrundman | 2010-06-01 11:31:10 -0500
 Change the color of the spectrum analyzer cap to one that doesn't have the MSB 
bit set, as this causes expensive alpha-blending for an unknown reason on fab4 
but not desktop
 r39...@daddymac-520 (orig r8837):  bklaas | 2010-06-01 14:44:37 -0500
 Fixed Bug: 16230
 Description: initialize self.localPlayer for local players that call 
notify_playerCurrent()
 if self.localPlayer is not available when openAlarmWindow() is called, try to 
get it then as well
 

Modified:
    7.6/trunk/   (props changed)
    
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/AlarmSnooze/AlarmSnoozeApplet.lua
    
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/WQVGAlargeSkin/WQVGAlargeSkinApplet.lua
    
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/WQVGAsmallSkin/WQVGAsmallSkinApplet.lua

Propchange: 7.6/trunk/
------------------------------------------------------------------------------
--- svk:merge (original)
+++ svk:merge Tue Jun  1 12:47:14 2010
@@ -13,7 +13,7 @@
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.4/private-branches/fab4-skin:4552
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.4/private-branches/new-alsa:6567
 bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.4/trunk:8423
-bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.5/trunk:8826
+bbe22326-0783-4b3a-ac2b-7ab96b24c8d9:/7.5/trunk:8837
 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.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=8838&r1=8837&r2=8838&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
 Tue Jun  1 12:47:14 2010
@@ -282,6 +282,9 @@
 
 function notify_playerCurrent(self, player)
        log:info("notify_playerCurrent(", player, ")")
+       if player:isLocal() then
+               self.localPlayer = player
+       end
 end
 
 
@@ -408,6 +411,17 @@
 end
 
 function openAlarmWindow(self, caller)
+
+       -- Bug 16230: if self.localPlayer hasn't been set, try to here as it's 
essential to the alarm working
+       if not self.localPlayer then
+               log:warn('openAlarmWindow: called without self.localPlayer 
being set. try to acquire it now')
+               self.localPlayer = Player:getLocalPlayer()
+               -- if that returns nil (very unlikely), throw an error and 
return
+               if not self.localPlayer then
+                       log:error('openAlarmWindow: without self.localPlayer 
the alarm cannot sound')
+                       return
+               end
+       end
 
        log:warn('openAlarmWindow()', caller, ' ', 
self.localPlayer:isConnected())
 
@@ -462,10 +476,6 @@
                        log:warn('openAlarmWindow: fallback alarm activation')
                else
                        log:warn('openAlarmWindow: fallback alarm snooze or 
explicit audio cycle')                      
-               end
-               if not self.localPlayer then
-                       log:warn('openAlarmWindow: cannot play an alarm without 
a player')
-                       return
                end
 
                self:soundFallbackAlarm()

Modified: 
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/WQVGAlargeSkin/WQVGAlargeSkinApplet.lua
URL: 
http://svn.slimdevices.com/jive/7.6/trunk/squeezeplay/src/squeezeplay/share/applets/WQVGAlargeSkin/WQVGAlargeSkinApplet.lua?rev=8838&r1=8837&r2=8838&view=diff
==============================================================================
    (empty)

Modified: 
7.6/trunk/squeezeplay/src/squeezeplay/share/applets/WQVGAsmallSkin/WQVGAsmallSkinApplet.lua
URL: 
http://svn.slimdevices.com/jive/7.6/trunk/squeezeplay/src/squeezeplay/share/applets/WQVGAsmallSkin/WQVGAsmallSkinApplet.lua?rev=8838&r1=8837&r2=8838&view=diff
==============================================================================
    (empty)

_______________________________________________
Jive-checkins mailing list
[email protected]
http://lists.slimdevices.com/mailman/listinfo/jive-checkins

Reply via email to