Go to: http://www.zdnetasia.com/news/dailynews/story/0,2000010021,20072958-1,00.htm and select File -> Send Page This will generate 4 attachments in the compose window!

This issue can be reproduced in the branch build. win32 commercial seamonkey build 2000-092711-mn6 installed on P500 Win98 linux commercial seamonkey build 2000-092710-mn6 installed on P200 RedHat 6.2 macos commercial seamonkey build 2000-092711-mn6 installed on G3/400 OS 9.04 Note: when you try to send the message, seamonkey reports that it could not send the attachment and ask if you would like to send the message anyway.

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

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

nominating rtm. Fix should be simple

but at this point, I don't think it will be approved!

I'm willing to see your simple fix in the form of a reviewed patch :-) This could be important because entire services have been built up around sending these types of email. If this is broken, a lot of people are going to see it as they forward these messages to friends and such.

It's a little bit worst than expected. Even if you add the web page url manually (using File/Attach Web Page), you cannot send the message because of the comma in the url. Also, I discovered that with Classic theme, you cannot remove an attachment!! I'll file a new bug for that.

Same similar problem appends when you try to send a file with a comma in the name. You get a error message asking you if you want to send the message without the attachment.

update summary

I have a solution for this problem but now I hit another problem on Mac: I cannot send an attachment if the file name part if more than 32 characters, this is the case with the URL attached to this email once I have replaced the "," with "%2C". The worst in with this new problem is that we failed to show an error. We should really fix this mess for RTM

r=scottip

Per Alecf suggestion, I tuned up a little be the error reporting: Index: nsMsgCompose.cpp =================================================================== RCS file: /cvsroot/mozilla/mailnews/compose/src/nsMsgCompose.cpp,v retrieving revision 1.203.2.2 diff -b -u -2 -r1.203.2.2 nsMsgCompose.cpp --- nsMsgCompose.cpp 2000/10/04 19:54:06 1.203.2.2 +++ nsMsgCompose.cpp 2000/10/12 21:02:54 @@ -718,5 +718,6 @@ ShowWindow(PR_TRUE); if (rv != NS_ERROR_BUT_DONT_SHOW_ALERT) - nsMsgDisplayMessageByID(prompt, rv); + if (NS_FAILED(nsMsgDisplayMessageByID(prompt, rv))) + nsMsgDisplayMessageByID(prompt, NS_ERROR_SEND_FAILED); } @@ -822,5 +823,6 @@ if (m_window) m_window->GetPrompter(getter_AddRefs(prompt)); - nsMsgDisplayMessageByID(prompt, rv); + if (NS_FAILED(nsMsgDisplayMessageByID(prompt, rv))) + nsMsgDisplayMessageByID(prompt, NS_ERROR_SEND_FAILED); } }

sr=alecf, marking rtm+

rtm++

pdt marked rtm++ so I'm changing the status whiteboard.

Fixed and checked in both Branch & Trunk

fixed in 2000101608mn6, vtrunking...

Verified Fixed on trunk builds Sending test page works as expected linux 101808 RedHat 6.2 win32 101804 NT 4 mac 101804 Mac OS9 Setting bug to Verified and removing vtrunk keyword