Closed Bug 361836 Opened 13 years ago Closed 13 years ago

Calendar csv export does not include alarms

Categories

(Calendar :: Import and Export, defect)

Lightning 0.3
x86
Windows XP
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ds332, Assigned: jminta)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0
Build Identifier: Lightning 0.3 on Thunderbird 1.5.0.8

When I export my calendar to a csv file from Lightning, there is a column for Reminder On/Off, but no data exports to that column, regardless of the reminder/alarm status set for the event in Lightning.

Reproducible: Always

Steps to Reproduce:
1. Create an event in Lightning
2. Export as csv file
3. Reminder column will be blank

Actual Results:  
File exported (although time data ignored my timezone as mentioned in a separate bug report), but no alarm data (regardless of whether or not one was set) exported to the Reminder, Reminder Date, or Reminder Time columns in the csv file.

Expected Results:  
Should put "True" or "False" in Reminder column and time and date (if TRUE for Reminder) in Reminder Date and Reminder Time columns.
Version: unspecified → Lightning 0.3
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) 
Gecko/2006120104
Calendar/0.4a1

Reproducible: Always

Steps followed to confirm the bug:
1. Created an event in Lightning.
2. Exported the calendar as a csv file.
3. Opened the csv file with a text editor.

Result:
The Reminder on/off, Reminder Date, Reminder Time columns were blank in the exported csv file.
Status: UNCONFIRMED → NEW
Ever confirmed: true
This patch fleshes out the stubbed alarm support in the csv importer and exporter.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #247878 - Flags: second-review?(mvl)
Attachment #247878 - Flags: first-review?(lilmatt)
Comment on attachment 247878 [details] [diff] [review]
add import/export for alarms

r=lilmatt
Attachment #247878 - Flags: first-review?(lilmatt) → first-review+
Comment on attachment 247878 [details] [diff] [review]
add import/export for alarms

r2=mvl
Attachment #247878 - Flags: second-review?(mvl) → second-review+
Whiteboard: [needs checkin]
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [needs checkin]
You need to log in before you can comment on or make changes to this bug.