38288 - Send fails and compose window does not close.

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[fenella]

Linux (2000-05-05-08 M16)
1. Launch Messenger using -mail option
2. Click on New Msg to compose a message and send it to myself
3. Click Send (or File|Send)

Actual result: No response. compose window does not close. Mail is not sent. 

Expected result: It should send the message and close the Compose window.

Have not tested Win32 and Mac yet.

Comment 1

[lchiang]

mscott is not here. reassign to sspitzer for now.

Comment 2

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

I see this on win32 also, here's what's in the console:

JavaScript error:
 line 0: uncaught exception: [Exception... "Component returned failure code: 0x8
000ffff (NS_ERROR_UNEXPECTED) [nsIPref.GetBoolPref]"  nsresult: "0x8000ffff (NS_
ERROR_UNEXPECTED)"  location: "JS frame :: chrome://messenger/content/messengerc
ompose/MsgComposeCommands.js :: GenericSendMessage :: line 515"  data: no]

Comment 3

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

fix in hand.  here's the problem:

we access mail.SpellCheckBefore send, but we never define a default value for 
the pref in mailnews.js

testing fix now, and then I'll check in.

Comment 4

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

fix checked in.

Comment 5

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

adding ducarroz and kin to the cc list.

ducarroz, when you read a pref from JS, you have to either:

1) specify a default value 

or

2) wrap it with try and catch

if the pref has no default, and the user hasn't specified a value, you'll get a 
JS exception.

Comment 6

[fenella]

win32 (2000-05-05-09 M16)
Win32 also has this problem.

Comment 7

[fenella]

Mac (2000-05-05-10 M16) Mozilla and commercial
it has problem too.

Comment 8

[laurel]

Marking verified based on all platform mail smoketest results for 05/11
commercial builds.