Fix bug 474275 - Alarm don't store on storage calendar (regression). r=philipp
authorPhilipp Kewisch <mozilla@kewis.ch>
Tue, 20 Jan 2009 13:48:50 +0100
changeset 1701 3858b26cdd918533df9e951deaf3ceb52fab9fe3
parent 1700 1a22abcfbc7c5e70b0242ea0f31b1f91f07e72c8
child 1702 5667e365df4be5d690dca281bb8bc934750fea6c
push id1365
push usermozilla@kewis.ch
push dateWed, 21 Jan 2009 11:19:12 +0000
treeherdercomm-central@3858b26cdd91 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersphilipp
bugs474275
Fix bug 474275 - Alarm don't store on storage calendar (regression). r=philipp
calendar/providers/storage/calStorageCalendar.js
--- a/calendar/providers/storage/calStorageCalendar.js
+++ b/calendar/providers/storage/calStorageCalendar.js
@@ -1770,16 +1770,17 @@ calStorageCalendar.prototype = {
             duration.normalize();
 
             // TODO ALARMSUPPORT for now just use the one relative alarm. Will
             // change when supporting multiple alarms.
             item.clearAlarms();
             let alarm = cal.createAlarm();
             alarm.related = row.alarm_related + 1;
             alarm.offset = duration;
+            item.addAlarm(alarm);
         }
         if (row.alarm_last_ack) {
             // alarm acks are always in utc
             item.alarmLastAck = newDateTime(row.alarm_last_ack, "UTC");
         }
 
         if (row.recurrence_id)
             item.recurrenceId = newDateTime(row.recurrence_id, row.recurrence_id_tz);