82583 - Can't save a mail attachment when offline.

Status: VERIFIED FIXED

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

Description

[laurel]

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.

Comment 1

[David :Bienvenu]

Attachment: proposed fix

Comment 2

[David :Bienvenu]

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.

Comment 3

[Navin Gupta]

r=naving 

Comment 4

[scottputterman]

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.

Comment 5

[Scott MacGregor]

sr=mscott

Comment 6

[Asa Dotzler [:asa]]

a= [e-mail] for chekin to 0.9.1

Comment 7

[David :Bienvenu]

fix checked in.

Comment 8

[lrg]

Functional in 2001053108 commercial linux build
Functional in 2001053108 commercial MacOS build
Functional in 2001053108 commercial Windows build

Standing in for Gary Chan

Comment 9

[grylchan]

marking verified based on lrg's comments

Comment 10

[R.K.Aa.]

Did this regress? See bug 86206 and new bug 107314