110502 - "Forward-inline" / "edit as new" does not bring up the compose window

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect)

Description

[Sheela Ravindran]

Build: 2001-11-16-05 win98, linux


Forwarding a message inline broken on all machines. When you select a message 
and click on forward button with prefs set to forward inline the compose window 
never comes up. But forwarding as attachment seems to be working on win98

This problem is same on imap as well as pop accounts. 

Result: The compose window never comes up.  
Expected: compose window should open with the message and should be able to 
forward inline.

Comment 1

[Sheela Ravindran]

Saving messages to the draft and template is working. I checked this on the imap 
account.  But cannot edit as draft when you click on the edit draft button.  
Cannot make changes to the template either.  So these two seems to be partly 
broken.  

Comment 2

[Darin Fisher]

this is likely related to the standard url changes.

Comment 3

[Sheela Ravindran]

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

Comment 4

[Håkan Waara]

Here is some console output I get when tryinf to forward:

WARNING: malformed uri, file C:\mozilla\moz\mozilla\mozilla\netwerk\base\src\nsU
RLParsers.cpp, line 157

###!!! ASSERTION: NS_ENSURE_TRUE(NS_SUCCEEDED(rv)) failed: '(!((rv) & 0x80000000
))', file C:\mozilla\moz\mozilla\mozilla\mailnews\base\util\nsMsgUtils.cpp, 
line 90

Unable to set the output stream for the mime parser...
could be failure to create internal libmime data
JavaScript error:
 line 0: uncaught exception: [Exception... "Component returned failure code: 0x8
000ffff (NS_ERROR_UNEXPECTED) [nsIMsgComposeService.OpenComposeWindow]"  nsresul
t: "0x8000ffff (NS_ERROR_UNEXPECTED)"  location: "JS frame :: chrome://messenger
/content/mailCommands.js :: ComposeMessage :: line 209"  data: no]

Comment 5

[Francisco León]

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

Comment 6

[Francisco León]

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

Comment 7

[Jean-Francois Ducarroz]

This is caused by bug 110508

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

Comment 8

[Francisco León]

sspitzer says this is not a dupe

Comment 9

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

not a dup.

fix in hand.

Comment 10

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

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

Comment 11

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

Attachment: fix,

Comment 12

[Bradley Baetz (:bbaetz)]

Comment on attachment 58311 [details] [diff] [review]
fix,

r=bbaetz

Comment 13

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

fixed.

this has r=bbaetz, sr=bienvenu

Comment 14

[R.K.Aa.]

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

Comment 15

[David :Bienvenu]

I don't think '-' is a valid scheme character either.

        if (!(nsCRT::IsAsciiAlpha(*scheme) ||
              nsCRT::IsAsciiDigit(*scheme) ||
'-' isn't alpha, so things will still fail.

Comment 16

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

'-' is valid.  see nsURLParsers.cpp, IsValidScheme().

for the conversation about '_' and '-' see, start here.

http://bugzilla.mozilla.org/show_bug.cgi?id=110508#c10

Comment 17

[David :Bienvenu]

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

Comment 18

[Sheela Ravindran]

verified using 2001-11-27-06 build on win98, linux, mac os x