Closed
Bug 14045
Opened 25 years ago
Closed 25 years ago
[DOGFOOD, REGRESSION] Quoting for for imap and pop not working today...
Categories
(MailNews Core :: Composition, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
M11
People
(Reporter: mscott, Assigned: sfraser_bugs)
References
Details
This appears to be an editor problem. Note: this is different from Bug #13891
which just effected replying to local mail and which Jean-Francois already
fixed.
If you try to reply to a message in todays build, the editor instance never
shows the message body.
I've traced it through and our code is assembling the message in a temp file and
handing a file url off to the editor instance telling it to load this url in the
editor instance.
But editor never ends up loading the file.
Jean-Francois, I'm pretty sure this is a editor and/or netwerk bug (from
warren's changes yesterday??). Whose are main contact over there for re-assiging
this bug?
Comment 1•25 years ago
|
||
I was just looking at this problem. What appends is that we (msgCompose) tell
ender to load a local url (temp/nscomp.html). Then we receive the callback
(NotifyDocumentCreated) from ender telling us that the document has been
created. At this point we delete the temp file but I get later the folowing
error on the output window:
Error: Can't load: file:///C|/TEMP/nscomp.html (8000ffff)
I don't know why ender tell us it's done when it isn't!
Updated•25 years ago
|
Assignee: ducarroz → buster
Summary: [DOGFOOD] Quoting for for imap and pop not working today... → [DOGFOOD, REGRESSION] Quoting for for imap and pop not working today...
Target Milestone: M11
Comment 2•25 years ago
|
||
I reassign this bug to buster.
Comment 3•25 years ago
|
||
Cc'ing warren. He checked in something that somehow delayed message loading in
the editor to some time later than when we get the OnEndDocumentLoad
notification. This is causing other problems (e.g. asserts on startup) and I
wonder if it also might be the root of this bug as well?
When you think you have this fixed, pleaes test 14073 as well, which I think is
a duplicate.
Assignee | ||
Updated•25 years ago
|
Assignee: buster → sfraser
Assignee | ||
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•25 years ago
|
||
I checked in a change to nsEditorShell which should fix this. The old double-
reload with charset bug was back. We now handle this by checking the status value
on the first OnEndDocumentLoad call, which is NS_BINDING_ABORTED. For this
call, we don't try to create the editor.
Assignee | ||
Comment 7•25 years ago
|
||
Fixed.
Reporter | ||
Comment 8•25 years ago
|
||
I pulled Simon's fix and quoting does indeed work again! Thanks simon.
changing QA assigned to pmock@netscape.com
Comment 10•25 years ago
|
||
Verified on IMAP for win32, macos, and linux using the following builds:
ftp://sweetlou/products/client/seamonkey/windows/32bit/x86/1999-11-12-18-M11/bac
kup/install.exe
ftp://sweetlou/products/client/seamonkey/macos/8.x/ppc/1999-11-11-11-M11/netscap
e5-mac-M11.sea.bin
ftp://sweetlou/products/client/seamonkey/unix/linux_glibc/2.2/x86/1999-11-12-18-
M11/netscape-i686-pc-linux-gnu.tar.gz
I verified quoting is working on POP and IMAP using rich text editor. Plain
text quoting is not working in these build. There is a separate bug on this
issue bug 13574.
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•