Closed Bug 13574 Opened 25 years ago Closed 25 years ago

Reply to a News or a IMAP message use old quoting

Categories

(MailNews Core :: Composition, defect, P3)

x86
Windows NT
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: jefft)

References

Details

(Whiteboard: Need to verify with M12 build.)

new quoting should be use when reply to a News or a IMAP message. Until then we won't be able to reply correctly.
Blocks: 6931
Blocks: 3745
Blocks: 7573
Assignee: rhp → jefft
Jeff, can you look at this and see what it would take to use the same quoting code for news/IMAP that we do for local mail?
Status: NEW → ASSIGNED
What's the different between old quoting and new quoting? It seems working fine with me when I tried to replay a news message.
Target Milestone: M11
M11 ...
The bug is kind of fixed but this is not what we really want. The current implementation still using the temp file for quoting. We should use stream converter instead.
Should we reassign this bug to mscott?
No, I want to handle it since I need to know how stream conver works in general. This will be my step stone toward Save/Open attachments.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
We now use stream coverter service instead of temp file for reply quoting. Thanks to mscott for handling imap implementation of enabling nsIStreamListener for message sink. nsIMsgQuote.idl, nsMsgQuote.h, nsMsgQuote.cpp, nsImapProtocol.cpp, nsImapUrl.cpp, nsImapService.cpp, nsImapUtils.cpp, nsNewsUtils.cpp, nsLocalUtils.cpp, nsStreamConverter.cpp modified.
QA Contact: lchiang → pmock
changing QA assigned to pmock@netscape.com
Per mscott, to verify this bug I need to check under IMAP and POP that quoting bar/delimiter appears correctly. This includes using a mail html and plain text editor. Jeff, if there are more cases, please let me know. thanks.
nope. Just try to reply to a complex HTML message and check everything is fine.
Verified on win32, macos, and linux for IMAP and POP rich text quoting. The builds that I used were: ftp://sweetlou/products/client/seamonkey/windows/32bit/x86/1999-11-09-11-M11/bac kup/install.exe ftp://sweetlou/products/client/seamonkey/macos/8.x/ppc/1999-11-08-08-M11/netscap e5-mac-M11.sea.bin ftp://sweetlou/products/client/seamonkey/unix/linux_glibc/2.2/x86/1999-11-09-11- M11/netscape-i686-pc-linux-gnu.tar.gz Need to verify quoting under News in order finish verifying this bug.
Depends on: 17983
Using the 1999-11-09-m11 build for win32, MacOS, and Linux, html quoting works under News. Note: plain text quoting (for news and mail)is broken and there is a separate bug on this. I'm making this bug 17983
oops. I meant to say this bug depends on bug 17983
Whiteboard: Need to verify with M12 build.
Update: bug 17983 has been resolved fixed but the fix is not in M11. That bug is target fix for M12.
Forgot to add... I tried the M11 commercial build for win32 and linux. ftp://sweetlou/products/client/seamonkey/windows/32bit/x86/1999-11-12-18-M11/bac kup/install.exe ftp://sweetlou/products/client/seamonkey/unix/linux_glibc/2.2/x86/1999-11-12-18- M11/netscape-i686-pc-linux-gnu.tar.gz
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.