118916 - file picker does not respect SetDisplayDirectory() when opening in "pick folder" mode

Status: RESOLVED DUPLICATE of bug 89325

(SeaMonkey :: Bookmarks & History, defect)

Description

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

file picker does not respect SetDisplayDirectory() when opening in "pick folder"
 mode

here's the mailnews code for "saving all attachments", which wants you to pick a
folder.

   filePicker->Init(
        nsnull, 
        GetString(NS_LITERAL_STRING("Save All Attachments").get()),
        nsIFilePicker::modeGetFolder
        );

    rv = GetLastSaveDirectory(getter_AddRefs(lastSaveDir));
    if (NS_SUCCEEDED(rv) && lastSaveDir) {
      filePicker->SetDisplayDirectory(lastSaveDir);
    }
    
    filePicker->Show(&dialogResult);
    if (dialogResult == nsIFilePicker::returnCancel)
        goto done;

the call to SetDisplayDirectory() doesn't work.  so even though we are trying to
persist the last directory the user chose, it doesn't work.

this blocks bug #108542

Comment 1

[Bill Law]

Seth,  do you see the same problem on all platforms?  There are different
implementations of nsIFilePicker so we may have to fix this is three different
places.

I can definitely take care of the Win32 implementation and can manage the Linux
(.js) one, too.  The Mac is a different story, but most likely this isn't broken
there (I hope).

I'm setting the target milestone to match bug 108542.

Comment 2

[Mark Hammond [:markh] [:mhammond]]

Almost certainly a dupe of but 89325 - and that bug even has a patch ;)

Comment 3

[Markus Gerstel]

Sorry

Comment 4

[Mark Hammond [:markh] [:mhammond]]

Marking as dupe of 89325, which has a patch.  All we need now is to work out the
process for moving a confirmed bug with patch into the build, within 18 months
of being reported and a fix made.

Yep - 18 months!  It really doesn't encourage community participation :(

*** This bug has been marked as a duplicate of 89325 ***