Closed Bug 10801 Opened 25 years ago Closed 25 years ago

[FEATURE] Save messages

Categories

(MailNews Core :: Backend, defect, P3)

defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: scottputterman, Assigned: jefft)

References

Details

(Whiteboard: [PR1])

bug tracking feature in Summary.
Blocks: 10791
Summary: [FEATURE] Save messges → [FEATURE] Save messages
Target Milestone: M9
OS: Windows NT → All
Hardware: PC → All
Whiteboard: [PR1]
This needs to be fixed for PR1, so I added a note to the Status Whiteboard.
Triage to M11
Save message as *.eml file now (9-3-99) works for Pop3. Waiting for SaveMessageToDisk to be implemented (mscott) for Nntp & Imap. Save message as *.hml & *.txt requires to hook up with mime and some other converter. Save message as Templates may use the copy service to do the trick.
Save message as template from pop3 to pop3 template folder is working. Pop3 messages to the imap templates folder isn't working yet due to the unwanted dummy header ans x-mozilla-status headers. mscott, when you implement SaveMessageToDisk for NNTP and IMAP message service could you make sure they generate additional dummy envelope header and x-mozilla-status headers.
Hey Jeff, can this stuff use the copy service or does it need the message to be in a temp file. With my performance changes, it is really easy to get a message from pop or news in the form of a stream where you are a stream listener. I'm still doing save message to a temp file but would it be faster if you can stream the message directly to the appropriate folder?
SaveMessageToDisk is meant to save the raw message to the disk (*.eml file). I am afraid that we cannot avoid it. For saving messages as *.html or *.txt files we will be able to utilize the stream listener. We could pipelining through the stream converter and then save the final result to a local file. I am using copy service CopyFileMessage to save message as template. It requires using a temp file. Is it possible for you to add an additional boolean argument to thr SaveMessageToDisk() to indicate whether we want to generate dummy envelope and x-mozilla-status headers. We needs to have those headers when messages are saved to a pop3 templates folder. And not those headers when messages are saved to an Imap4 templates folder.
Save message as raw rfc822 mail message and html document are working correctly on Pop3, News, and Imap4. Save message as plain text also coded. As long as akkana finishes her text/html => text/plain converter, the code will just work. Save message as template, is working for messages within the same server. Cross the server boundary isn't working yet. It a tedious job to figure out whether a dummy header should be generated/stripped out when messages were copied to the destination Templates folder. 4.0 servers prohibits a message contains dummy berkerley header being append to the server. Local mail folder won't work if a message was copied from the imap server which does not contain dummy berkerley envelope header plus the X-Mozilla-Status flags.
Blocks: 11091
(target milestone is M11 or M12 - add to mail beta tracking bug)
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
I have fixed the save message as template problem. The only not working now is save message as plain text which akkana will have a fix in the future. The works for Save Messages are done for the mail/news team.
QA Contact: lchiang → esther
Status: RESOLVED → REOPENED
Using win98 build 1999111909 m12, and 1999112008 on mac and linux, the menu item File|Save As|Template is not saving a message selected from my IMAP Inbox and saved as Template to my IMAP Templates folder. I read through this bug and based on the last comment (10/07/99 8:33) this should be working for IMAP too. Jeff let me know if I'm not testing this correctly. Note: My Template folder is set up correctly because I can Save As Template a newly created message. Also, Save as Text is not working, per comments in this bug this now belongs to Akkana, should I write a new bug for her?
Resolution: FIXED → ---
Target Milestone: M11 → M12
M11 is over, moving to M12 for now since this bug reopened. phil may want to change.
Status: REOPENED → ASSIGNED
Blocks: 1775
I have the fix in hand waiting for review. The html to txt converter is addressed in bug 1775.
Why does this require a stream converter? When you save, aren't you saving from a document, so you can just use the nsIDocumentEncoder service? That seems much more efficient than using whatever means you're currently using to save to html (which means saving to XIF, parsing the XIF then converting to html) then taking that html and parsing it again, saving it to XIF then parsing the XIF again and saving to plaintext (which is what will happen if you save to html then use a stream converter to convert).
Status: ASSIGNED → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
No, we are not saving it into html first then convert it into a text document. We are trying to use the stream converter's magic without going through any temporary file. For example, from A to D, we don't do A->B, B->C, then C->D ourself. We leave it to the nsIStreamConverterService to do the work. We only tell the stream converter service what's the src and the destination format we are interested. Stream converter service will then based on it's registry to invoke a converter or a series of converters to do the convertion. The fix for save as template has been checked in. nsIMsssenger.idl, widgetglue.js and nsMessenger.cpp modified.
Status: RESOLVED → VERIFIED
Using builds 1999112409m12 on windows and linux and 19991124 on mac most of this is fixed. Save as Template working on Win & Mac linux has a new bug for not working correctly (20063) and text not working on all is a new bug (20064) this is verified for the most part.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.