108542 - Save All... attachments should remember the last saved folder name

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P2)

Description

[Roy Yokoyama]

1) in mail/new, highlight on a mail with attachment
2 [details] [diff] [review]) right mouse button click on attachment and select "Save All.." context menu
3) Select a folder to save the attachments and click "OK"
4) repeat the step #2 again
5) mail/news should highlighted(retained) the last saved foldername.

Comment 1

[Jason Bennett]

Confirm that this happens with "Save As" as well.

Comment 2

[Dimitrios]

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

Comment 3

[Eugene Savitsky]

Dup of 115148? Since all save dialog files to remember last used directory.

Comment 4

[(not reading, please use [e-mail] instead)]

accepting, I'm going to fix all these related bugs at once.

Comment 5

[(not reading, please use [e-mail] instead)]

I've got a fix for part of this, but the fix for save all isn't working.  the 
file picker code isn't respecting when I call SetDisplayDirectory() before I do 
show.

Comment 6

[(not reading, please use [e-mail] instead)]

Attachment: fix for this, and the compose attach scenario

Comment 7

[(not reading, please use [e-mail] instead)]

I've landed my piece of code for this, r/sr=bienvenu.

these scenarios work:
saving a single attachment
saving a message as a file

but the "Save All Attachments" scenario doesn't work, as there is a folder
picker problem.

I'll keep this open until I fix that issue.

Comment 8

[(not reading, please use [e-mail] instead)]

moving "Save Attachment & Save Message As File" to another bug, since they are
fixed.

only "Save All..." is broken, because of bug #118916

Comment 9

[(not reading, please use [e-mail] instead)]

moving out to 1.0.1

Comment 10

[scottputterman]

If 118916 gets fixed we'll take any changes, if any, that we need for this.

Comment 11

[odi]

This happens also when saving multiple images (from different windows) to disk.
This is very annoying.

Comment 12

[R.K.Aa.]

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

Comment 13

[Boris 'pi' Piwinger]

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

Comment 14

[Croco Dil]

On my 2004041809 this seems to work ?!

Comment 15

[OstGote!]

This was already fixed and is WFM. The pref messenger.save.dir is used for that.

see bug 115148 comment 43