324633 - Error importing Outlook 2002 CSV calendar

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[quark.green]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20051104 Mozilla Sunbird/0.3a1

I get the following error when importing a CSV calendar file into Sunbird that I exported from Outlook 2002:

Unable to read from file:(path and filename)
[Exception... "Cannot modify properties of a WrappedNative" nsresult: "0x80570034 (NS_ERROR_XPC_CANT_MODIFY_PROP_ON_WN)" location: "JS frame :: file:J:/Portable/sunbird/components/calOutlookCSVImportExport.js :: csv_importFromStream :: line 269" data: no

Reproducible: Always

Steps to Reproduce:
1. Export outlook 2002 calendar as csv format
2. Import calendar in Sunbird using type "Comma Separated Values (from Outlook)"
3.

Actual Results:  
No data is imported.  I get the following error:

Unable to read from file:(path and filename)
[Exception... "Cannot modify properties of a WrappedNative" nsresult: "0x80570034 (NS_ERROR_XPC_CANT_MODIFY_PROP_ON_WN)" location: "JS frame :: file:J:/Portable/sunbird/components/calOutlookCSVImportExport.js :: csv_importFromStream :: line 269" data: no

Expected Results:  
Import the calendar data and display it.

Comment 1

[Joey Minta]

Confirmed.  isAll
Day was removed awhile ago.

Comment 2

[Joey Minta]

Attachment: use isDate instead of isAllDay

isAllDay was removed awhile ago.  Set isDate on the starting date instead.

Comment 3

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

Comment on attachment 210522 [details] [diff] [review]
use isDate instead of isAllDay

this part looks ok, but there seem to be other spots:
http://lxr.mozilla.org/seamonkey/search?string=isAllDay

Comment 4

[Joey Minta]

Attachment: use isDate instead of isAllDay v2

Version 2 should have all the other uses of isAllDay.  The remaining calls in the lxr link were in obsolete code (or were just variable names, not calls to the obsolete method).

Comment 5

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

Comment on attachment 210535 [details] [diff] [review]
use isDate instead of isAllDay v2

r=mvl

Comment 6

[Joey Minta]

Patch checked in.

Comment 7

[Simon Paquet [:sipaq]]

*** Bug 326102 has been marked as a duplicate of this bug. ***