Closed Bug 348651 Opened 18 years ago Closed 18 years ago

Calendar export/publish fails with 'Unable to write to file' message

Categories

(Calendar :: Import and Export, defect)

Product:
Calendar
Component:
Import and Export
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: ssitter, Assigned: jminta)

Details

(Keywords: regression)

Attachments

(1 file, 1 obsolete file)

also publishing
2.46 KB, patch
mattwillis
: first-review+
jminta
: second-review+
Details | Diff | Splinter Review 
Calendar export fails with 'Unable to write to file' message.

Steps to Reproduce:
1. Switch to Calendars pane
2. Select 'Export Calendar...' from context menu
3. Select format, name and location. Press Ok.

Actual Results:
A dialog is shown with title "Error getting calendar" and message "Unable to write to file:<location>". The target file is created but is empty.

Works in sunbird-0.3a2+.en-US.win32-2006-08-14-09-trunk
Fails in sunbird-0.3a2+.en-US.win32-2006-08-14-12-trunk
Attached patch extra arg (obsolete) — — Splinter Review 
I suck.  It's throwing "not enough args."  Regression from the print-title bug that I checked in inside that window.
Assignee: nobody → jminta
Status: NEW → ASSIGNED
Attachment #233631 - Flags: second-review?(dmose)
Attachment #233631 - Flags: first-review?(mattwillis)
Comment on attachment 233631 [details] [diff] [review]
extra arg

r1=lilmatt
Attachment #233631 - Flags: first-review?(mattwillis) → first-review+
Error: [Exception... "Not enough arguments"  nsresult: "0x80570001 (NS_ERROR_XPC_NOT_ENOUGH_ARGS)"  location: "JS frame :: chrome://calendar/content/publish.js :: publishItemArray :: line 197"  data: no]

Please check if there are more calls to exportToStream() affected.
Attached patch also publishing — — Splinter Review 
Also includes the changes to publish.js.  According to http://lxr.mozilla.org/mozilla/search?string=exporttostream this should be it.
Attachment #233631 - Attachment is obsolete: true
Attachment #233667 - Flags: second-review?(dmose)
Attachment #233667 - Flags: first-review?(mattwillis)
Attachment #233631 - Flags: second-review?(dmose)
Comment on attachment 233667 [details] [diff] [review]
also publishing

r1=lilmatt
Attachment #233667 - Flags: first-review?(mattwillis) → first-review+
Summary: Calendar export fails with 'Unable to write to file' message → Calendar export/publish fails with 'Unable to write to file' message
Comment on attachment 233667 [details] [diff] [review]
also publishing

r=dmose
Attachment #233667 - Flags: second-review?(dmose) → second-review+
patch checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: