Bug 7314 - DateTime screensaver doesn't display alarm bell for alarms on sundays
: DateTime screensaver doesn't display alarm bell for alarms on sundays
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: 7.0
: PC Windows XP
: -- normal (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-24 12:26 UTC by Michael Herger
Modified: 2009-07-31 10:17 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Herger 2008-02-24 12:26:05 UTC
If I set an alarm for Sunday, the alarm bell isn't displayed on Sunday itself. Any other day seems to be working as well as the bell for Sunday's alarm is displayed on Saturday.
Comment 1 Blackketter Dean 2008-02-24 16:54:58 UTC
If the alarm itself works, please punt this to 7.0.1.
Comment 2 Michael Herger 2008-02-24 22:18:11 UTC
Alarm's working fine, the screensaver is broken.
Comment 3 KDF 2008-02-24 23:06:12 UTC
This should do it.  Note: this is taken from the same fixed used for bug2300 in Slim::Buttons::AlarmClock

Index: Slim/Plugin/DateTime/Plugin.pm
===================================================================
--- Slim/Plugin/DateTime/Plugin.pm	(revision 17707)
+++ Slim/Plugin/DateTime/Plugin.pm	(working copy)
@@ -138,7 +138,8 @@
 	my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) = localtime(time);
 
 	# the alarm's days are sunday=7 based - 0 is daily
-
+	$wday = 7 if !$wday;
+	
 	my $alarm = preferences('server')->client($client)->get('alarm');
 
 	my $alarmtime = preferences('server')->client($client)->get('alarmtime');
Comment 4 Michael Herger 2008-02-29 00:23:06 UTC
change 17758 - thanks Kevin!
Comment 5 James Richardson 2008-05-08 14:54:17 UTC
Verified fixed in 7.0.1 - 19325

SB2/3 & Transporter now show Alarm bell on Sunday
Comment 6 James Richardson 2008-05-15 12:28:31 UTC
This bug has recently been fixed in the latest release of SqueezeCenter 7.0.1

Please try that version, if you still see the error, then reopen this bug.

To download this version, please navigate to: http://www.slimdevices.com/su_downloads.html
Comment 7 Chris Owens 2009-07-31 10:17:41 UTC
Reduce number of active targets for SC