155476 - Unable to Edit Draft or As new from the Local folder [@ nsXPCWrappedJSClass::GetInterfaceName]

Status: VERIFIED FIXED

(MailNews Core :: Composition, defect)

Description

[marina]

*** observed with 2002-07-02-1.0.0 build ***
this is a serious regression, steps to reproduce:
- set your settings to keep "Drafts" folder on Local;
- compose an email or forward an existing one;
- cancel sending;
- go to Drafts, select the message;
- click on the Edit Drafts button;
- note that the message compose window is blank, if i select a message from the
folder on the Local tree to Edit it as new i'll get an error message (the scree
shot of the error message to follow)

Comment 1

[marina]

Attachment: here is a screen shot of the error message you get when selecting a message from a folder under the Local and trying to Edit it as new

Comment 2

[laurel]

I also see this in july07 branch with win98 and when trying to open a draft via
double-click I crash.

Comment 3

[laurel]

Attachment: talkback 7942268 - open via doubleclick

Comment 4

[scottputterman]

JF, Navin, or Cavin, can you take a look at this?

Comment 5

[Kai Engert [:KaiE:]]

See http://bugzilla.mozilla.org/show_bug.cgi?id=137071#c43 for a first update. I
will comment here when I have analyzed more.

Comment 6

[Kai Engert [:KaiE:]]

I can make the problem go away by backing out the patch from 154734. (Bug 154734
implements a workaround to help bug 137071.)

When forwarding inline a message from "Local Folders", the URL of the original
message is passed to the composition window. The URL looks like:
  mailbox-message://nobody@local%20folders/bla#142395

Because bug 154734 changed the logic to un-escape the URL, it now looks like:
  mailbox-message://nobody@local folders/bla#142395


This causes a problem during the init phase of the opened compose window.
Some methods that are called are: GetMsgDBHdrFromURI, MessageURIToMsgHdr,
DecomposeMailboxURI.
Eventually the logic arrives at nsAuthURLParser::ParseServerInfo which bails
out, because it believes the "local folders" is a host name, and does not allow
the space. It returns an error, which causes the UI error message.


So, while unescaping the URI is a workaround to fix the problems with IMAP as
reported in bug 154734, it breaks working with "Local Folders".

Comment 7

[Dimitrios]

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

Comment 8

[Jean-Francois Ducarroz]

This problem went away with the fix for bug 155671.

Comment 9

[Gregg Meehan]

Tested in XP as well as Win2K, problem is resolved. Drafts can be edited.