Tickets :: Comment on [#12984] Dismissed Alarms coming back / still popping up on Mozilla Lightning

* Your Email Address
* Spam protection	Enter the letters below: . ..__ .__ . .. . \|\ \|\| \\| \\|\/\|\|\ \| \| \\|\|__/\|__/\| \|\| \\|
Comment	> I am using horde webmail 5.1.3 with kronolith 4.1.4. > When using iCal (ICS) remote subscription on Mozilla Thunderbird 24.3 > + Lightning 2.6.4 I keep getting random reset on previous events > alarms status. Then all alarms from about a month back are popping up > again. > > I have investigated into the code and attached a proposed patch. > To me the problem is twofold : > - Lightning make expired alarms pop up (all dismissed alarms popping > up are always on old events) > - Kronolith randomly "garbage collect" old events alarms and then > doesn't keep their status > > Looking at the code it randomly purges the horde_alarm table ("gc" > function) then all informations about old alarms are lost. > When exporting again the iCal in Event class the X-MOZ-LASTACK flag > is not for old events set since we have no more entries for them in > horde_alarm table. > Thus Lightning considers alarms as "fresh" for old events and make > them pop up again... > > So here is a patch attached that adds X-MOZ-LASTACK for all old > alarms even if there is no more entry in horde_alarm table to keep > the dimissed/snooze status. > This seems to be a workaround to me since I do not understand why > expired alarms should me notified ... (Lightning actual behavior) > > Moreover, to help with broken alarm dismissal client implementations > (or read-only access trying to update alarm status info) it may be > helpful to add a config/pref to Kronolith in order to remove expired > alarms for iCal export. Since iCal cannot store the status of alarm > notification (to my knowledge after reading the RFC, that's why > Mozilla seem to have added an extension for this) I don't see any > other mean to get rid of past notifications. > Either way, another subscription URL or better a query string (<ics > url>?no_expired_alarms=1) could make it configurable by the > subscriber for each client.
Attachment
Watch this ticket

Saved Queries