Tickets :: Comment on [#6183] Kronolith gives premature notifications

* Your Email Address
* Spam protection	Enter the letters below: __ . , __..___.__ / ` \./ (__ [__ [__) \__. \| .__)[___[__)
Comment	> I think I found the problem: > > I suppose alarm_snooze should not be set to '1970-01-01 00:00:00'? > For my alarms, this is always true. > > (See eric's horde_alarms sample below.) > > > > I tracked the alarm notification to the select statement in > lib/Horde/Alarm/sql.php, function _list. > > One of the conditions is: > > alarm_snooze <= $time->rfc3339DateTime() > > > > If alarm_snooze is set to 1970, this is always true. > > > > eg.: > > SELECT alarm_id, alarm_uid, alarm_start, alarm_end, alarm_methods, > alarm_params, alarm_title, alarm_text, alarm_snooze, alarm_internal > FROM horde_alarms WHERE alarm_dismissed = 0 AND ((alarm_snooze IS > NULL AND alarm_start <= '2008-03-21T10:10:07') OR alarm_snooze <= > '2008-03-21T10:10:07') AND (alarm_end IS NULL OR alarm_end >= > '2008-03-21T10:10:07') AND (alarm_uid = '' OR alarm_uid = 'myuser') > ORDER BY alarm_start, alarm_end; > > > > > > I am using a MySQL database. In the horde SQL scripts, alarm_snooze > is set to DATETIME. > > If you insert something into horde_alarms without setting > alarm_snooze explicitely, it is set to the start of unix time (1970). > > > > Attaching a patch.
Attachment
Watch this ticket