264447 - Export into vCalendar format has illegal newlines

Status: RESOLVED FIXED

(Calendar :: Sunbird Only, defect)

Description

[gekacheka]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7.3) Gecko/20040913 Firefox/0.10.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a4) Gecko/20041001 Mozilla Sunbird/0.2a

As reported https://bugzilla.mozilla.org/show_bug.cgi?id=231724#c9.
Gives problems for devices which can only import vCalendar 1.0 files.

Problem is iCalendar (v2.0) [rfc2445sec4.1] says lines can be broken anywhere
to comply with 75-octet line length, but vCalendar (v1.0) [vCalendar1.0sec
2.1.3] says lines can only be broken at places where there may be linear white
space.

Another difference is long text values:  iCalendar uses \n to encode newlines in
text values, vcalendar requires quoted-printable =0D=0A or base64 encoding.

Reproducible: Always
Steps to Reproduce:
1. Select events to export
2. control-O (Export Selected Events)
3. Select type vCalendar files (last type)
4. Save

Actual Results:  
BEGIN:VCALENDAR
VERSION
 :2.0
PRODID
 :-//Mozilla.org/NONSGML Mozilla Calendar V1.0//EN
METHOD
 :PUBLISH
BEGIN:VEVENT
UID
 :7fc113ee-1dce-11d9-96e9-aca8b7518fea
SUMMARY
 :DummyTitle
LOCATION
 :DummyLocation
STATUS
 :TENTATIVE
...

Expected Results:  
BEGIN:VCALENDAR
VERSION:1.0
PRODID:-//Mozilla.org/NONSGML Mozilla Calendar V1.0//EN
METHOD:PUBLISH
BEGIN:VEVENT
UID:7fc113ee-1dce-11d9-96e9-aca8b7518fea
SUMMARY:DummyTitle
LOCATION:DummyLocation
STATUS:TENTATIVE
...

Comment 1

[gekacheka]

Attachment: importExport.js patch: convert iCal to vCal newlines

saveEventsToFile:  call patchICalStringForVCal if vcalendar is selected.

patchICalStringForVCal: new, 
    - converts iCal to vCal newlines 
       * None before ':'.
       * if needed for \n in value, add ENCODING=QUOTED-PRINTABLE, use =0D=0A
       * rebreak continuation lines.
    - replace \, with , in summary, description, etc. text values
    - version:2.0 becomes version:1.0.
    
  breakIntoContinuationLines: used by patchICalStringForVCal
  endsWith: used by patchICalStringForVCal

Comment 2

[gekacheka]

Attachment: Input test file with events

Comment 3

[gekacheka]

Attachment: Output from exporting events in Input test file

Comment 4

[Mostafa Hosseini]

Comment on attachment 162149 [details] [diff] [review]
importExport.js patch: convert iCal to vCal newlines

Checked in. Thanks

Comment 5

[Reed Loden [:reed]]

The bugspam monkeys have been set free and are feeding on Calendar :: Sunbird Only. Be afraid for your sanity!