Closed Bug 82583 Opened 24 years ago Closed 24 years ago

Can't save a mail attachment when offline.

Categories

(SeaMonkey :: MailNews: Backend, defect, P2)

All
Windows 98
defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.9.1

People

(Reporter: laurel, Assigned: Bienvenu)

Details

(Keywords: dataloss, Whiteboard: [nsbeta1+])

Attachments

(1 file)

Using may24 commercial trunk build I'm not able to successfully save an attachment to a mail message (which has been downloaded for offline use) when offline. Many scenarios to test, this is a simple Text file attachment. I'ved tried various types of attachments, cached/viewed or not. 1. Go to mail window, login to IMAP account. 2. Send yourself a mail message with a small, simple text file attached. Get the message. 3. Download the message for offline use. ( I effected this by having the "simple" offline pref set to make inbox available when offline. ) Make sure the message indicates (currently italicized in thread pane) it has been downloaded for offline use. 4. Go Offline. Select the message. Message is readable offline, attachment shown in the attachment pane/listbox and contents display in the message pane. 5. Select the attachment name in the attachment pane/listbox, right-click and select Save As. Directory/file picker dialog opens, choose a location to file the bug on your local system and confirm/OK. Looks like the process worked. 6. Go to the location you specified and see the attachment was not saved.
Keywords: nsbeta1
Attached patch proposed fixSplinter Review
Navin, can I get a rview for this? The fix is simply to set the flag on the url that we have the msg offline when fetching an imap part. Thanks. Scott, do you think I should push for beta1? It's a trivial fix.
r=naving
yeah, I think we should fix this. It seems like this could lead to dataloss if the user deletes the message because they thought the save went through.
Priority: -- → P2
Whiteboard: [nsbeta1+]
Target Milestone: --- → mozilla0.9.1
sr=mscott
a= asa@mozilla.org for chekin to 0.9.1
Keywords: dataloss
fix checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Functional in 2001053108 commercial linux build Functional in 2001053108 commercial MacOS build Functional in 2001053108 commercial Windows build Standing in for Gary Chan
marking verified based on lrg's comments
Status: RESOLVED → VERIFIED
Did this regress? See bug 86206 and new bug 107314
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: