394634 - Double Quotes in Description not handled correctly when exported to .CSV files

Status: VERIFIED FIXED

(Calendar :: Import and Export, defect)

Description

[eznie0ba]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.6) Gecko/20070725 Firefox/2.0.0.6
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.4pre) Gecko/20070614 Sunbird/0.5

I was trying to export one calendar that I have used for a while now. In several of the events within this calendar, I used double quotes (") to surround some text in the comments.

Upon exporting the calendar to a .csv file, I noticed these double quotes are escaped properly for the opening quote (""), but the closing quote is not escaped, left as a single quote (").



Reproducible: Always

Steps to Reproduce:
1. Open calendar and create an event.
2. Include a "quoted string" in the description for the event.
3. Export the calendar.
Actual Results:  
The resulting value for the Description column will look something like:

....,"Include a ""quoted string" in the description for the event", ...

Expected Results:  
The resulting value for the Description column should look something like:

....,"Include a ""quoted string"" in the description for the event", ...

By not properly escaping the closing quote, import into Excel, OpenOffice, and possibly other products fails.

Comment 1

[eznie0ba]

Is this as simple as making the string replace() call to a global search and replace in the file "C:\Program Files\Mozilla Sunbird\js\calOutlookCSVImportExport.js", function csv_exportToStream(), line 441-444?

        line = line.map(function(v) {
            v = String(v).replace(/"/g,'""');
            return '"'+v+'"';
        })

By adding the 'g' (global) flag to the regex, it appears to escape quoted text properly.

Comment 2

[:Hb]

Attachment: Patch for global substitution in regex

Based on eznie0ba suggestion now replacing all occurrences of douple quotes

Comment 3

[Michiel van Leeuwen (email: mvl+moz@)]

Comment on attachment 304862 [details] [diff] [review]
Patch for global substitution in regex

r=mvl

Comment 4

[Simon Paquet [:sipaq]]

Fix checked into trunk and MOZILLA_1_8_BRANCH.