If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

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

VERIFIED FIXED

Status

Calendar
Import and Export
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: Stefan Sitter, Assigned: Joey Minta)

Tracking

({regression})

Trunk
x86
Windows 2000
regression

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.46 KB, patch
Matthew (lilmatt) Willis
: first-review+
Joey Minta
: second-review+
Details | Diff | Splinter Review
(Reporter)

Description

11 years ago
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
(Assignee)

Comment 1

11 years ago
Created attachment 233631 [details] [diff] [review]
extra arg

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+
(Reporter)

Comment 3

11 years ago
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.
(Assignee)

Comment 4

11 years ago
Created attachment 233667 [details] [diff] [review]
also publishing

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+
(Reporter)

Updated

11 years ago
Summary: Calendar export fails with 'Unable to write to file' message → Calendar export/publish fails with 'Unable to write to file' message
(Assignee)

Comment 6

11 years ago
Comment on attachment 233667 [details] [diff] [review]
also publishing

r=dmose
Attachment #233667 - Flags: second-review?(dmose) → second-review+
(Assignee)

Comment 7

11 years ago
patch checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED

Updated

11 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.