160591 - Filepicker for Save All Attachments should read 'DIRECTORY', not 'FILE'.

Status: VERIFIED DUPLICATE of bug 160672

(MailNews Core :: Attachments, defect)

Description

[[e-mail] (gone - use [e-mail] instead)]

Build ID: 8-1-2002 commercial 1.0 branch build.

Summary: Save All Attachments broken on Linux 1.0 branch builds.

Steps to Reproduce:

1.  In a message with multiple attachments, try to do File | Attachments | Save
All, I get the following:

A.  Dialog with 'Save All Attachments' in the window title.
B.  File name textfield (I don't think I should get this, since it should
auto-grep file names from my attachment listing).

2.  If I enter a filename (which again, I shouldn't be doing...), I just get
'File /home/stephend/foo' doesn't exist.

Basically, I think it can be surmised that the Linux filepicker for Save All is
thinking it wants to choose a file, although it's misleading, because it
actually has the 'Save All Attachments' window.title.

Expected Results:

You should just specify a dir and automagically have it save all files into that
dir, with extensions intact.

Actual Results:

See above.

Comment 1

[[e-mail] (gone - use [e-mail] instead)]

nominating, because I think Buffy should have this (at the very least).

Comment 2

[[e-mail] (gone - use [e-mail] instead)]

This is broken on the 2002-08-01-08 Linux builds as well.

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Know both of you guys are extremely busy, but was wondering if you might be able
to help out on this.

Comment 4

[[e-mail] (gone - use [e-mail] instead)]

Oh, and this works fine with one attachment.  So the problem lies somewhere in
http://lxr.mozilla.org/seamonkey/source/mailnews/base/resources/content/msgHdrViewOverlay.js#975
building the attachment array (or reading it later).

Comment 5

[Boris Zbarsky [:bzbarsky]]

I can't seem to reproduce this... (I can repro the crappy UI though -- the text
referring to "file" when it means "directory").

Comment 6

[[e-mail] (gone - use [e-mail] instead)]

Boris, yes, it was the UI - I kept thinking 'Why is this asking me for a
filename?'  Once I actually SELECTED a *directory* (not just moved into it),
this works fine.  So, this bug stands to rename the text to be 'DIRECTORY'.

Comment 7

[Boris Zbarsky [:bzbarsky]]

Ah.  I ended up filing bug 160672 on the issue; should have commented in here
that I did so.  Wanna review the patch?  ;)

Comment 8

[[e-mail] (gone - use [e-mail] instead)]

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

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

verified dup.  boris, now that you're back, could you find time to get bug
160672 in?