80235 - Cannot save attachments using "File | Attachemnts" if / in filename

Status: VERIFIED DUPLICATE of bug 33567

(SeaMonkey :: MailNews: Message Display, defect)

Description

[E.deWilde]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:0.9) Gecko/20010505
BuildID:    2001050515

Mozilla cannot save  an attachment if there is a / (forward slash) in the
filename of the attachment.

Reproducible: Always
Steps to Reproduce:
1.Try to save such an attachment
2 [details] [diff] [review].
3.

Actual Results:  Trying to save such an attachment doesn't even show the file
attachment explorer window in which you can normally change the name and the
location of the file you want to save 


Expected Results:  Open a explorer window

The Linux version of Mozilla doesn't have this problem.
The attachment with the forward slash in its filename was sent by a Macintosh
computer which allows forward slashes in filenames.

Comment 1

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

this sounds like it will get fix when we land cavin's fix for #63225

cavin, can you try this scenario out on your build with your patch?

Comment 2

[Cavin Song]

My patch on windows works the same as described by the reporter (ie, Save File 
dialog does not even show up). But I think this bug was filed against the Save 
As popup menu from the attachment list (on the upper right corner of the message 
pane or window) which is different from #63225 (the Save Image popup menu from 
the inline image).  Here is what I see on other platforms:

On linux, the name in the save file dialog shows 'foo:bar.gif' and it's saved on 
the disk with the same name. In other words, the filename is converted to 
foo:bar.gif, not the original foo/bar.gif.

On Mac, the name in the save file dialog shows 'foo:bar.gif' but the file is 
actually saved as 'foo-bar.gif' on the disk.  So it looks like we go from '/' 
(original) to ':' (display) and to '-' (actual save).

Comment 3

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

so this is a seperate, but valid bug.

not showing the "Save as" window is bad.

updating summary.

replacing the directory seperator or volume characters (C: on win32) sounds
right to me.

cavin, can you look into it?

Comment 4

[Keyser Sose]

Marking NEW while its figured out.

Comment 5

[Jean-Francois Ducarroz]

This is a dup of 33567. Cavin, if you aggree, please resolve it as dup.

Comment 6

[Cavin Song]

Verify as a dup of bug #33567.

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

Comment 7

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

verified dup.