Bug 106360

Summary: alarm.py: fails alarm for next day
Product: [Applications] amarok Reporter: Adeodato Simó <dato>
Component: Tools/Script ManagerAssignee: Amarok Developers <amarok-bugs-dist>
Status: RESOLVED INTENTIONAL    
Severity: normal CC: ana
Priority: NOR    
Version: 1.2.3   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Adeodato Simó 2005-05-27 02:20:57 UTC
Hello, I got this report at the Debian BTS (http://bugs.debian.org/309856, 
reported by Cruz Fernandez <cruz@chuchuwa.com.ar>): 
 
-----------------------------8<---------------------------------------- 
The alarm.py script (in /usr/share/apps/amarok/scripts/alarm/ ) doesn't 
work for a time of the next day, because it asumes the time is of the 
current day. 
 
Possible solution, add the seconds in a day to secondsleft: 
 
114,116d113 
<             if secondsleft < 0: 
<               secondsleft += 60 * 60 * 24 
< 
 
-----------------------------8<---------------------------------------- 
 
Nevertheless, I see in the page of the script 
(http://www.kde-apps.org/content/show.php?content=23160) that there's a more 
recent version than the one that amarok ships. At a first glance, it seems 
that it fixes Cruz's problem, so updatating it globally would be an option 
too. 
 
Thanks.
Comment 1 Greg Meyer 2006-03-01 03:20:57 UTC
Alarm.py has been removed from amaroK proper so I am closing this bug.