[FEATURE] Save messages

VERIFIED FIXED in M12

Status

MailNews Core
Backend
P3
normal
VERIFIED FIXED
18 years ago
9 years ago

People

(Reporter: scottputterman, Assigned: jefft)

Tracking

Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PR1])

(Reporter)

Description

18 years ago
bug tracking feature in Summary.
(Reporter)

Updated

18 years ago
Blocks: 10791
Summary: [FEATURE] Save messges → [FEATURE] Save messages
Target Milestone: M9

Updated

18 years ago
OS: Windows NT → All
Hardware: PC → All

Updated

18 years ago
Whiteboard: [PR1]

Comment 1

18 years ago
This needs to be fixed for PR1, so I added a note to the Status Whiteboard.

Comment 2

18 years ago
Triage to M11
(Assignee)

Comment 3

18 years ago
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.
(Assignee)

Comment 4

18 years ago
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.

Comment 5

18 years ago
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?
(Assignee)

Comment 6

18 years ago
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.
(Assignee)

Comment 7

18 years ago
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.

Updated

18 years ago
Blocks: 11091

Comment 8

18 years ago
(target milestone is M11 or M12 - add to mail beta tracking bug)
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

18 years ago
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.

Updated

18 years ago
QA Contact: lchiang → esther

Updated

18 years ago
Status: RESOLVED → REOPENED

Comment 10

18 years ago
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?

Updated

18 years ago
Resolution: FIXED → ---

Updated

18 years ago
Target Milestone: M11 → M12

Comment 11

18 years ago
M11 is over, moving to M12 for now since this bug reopened.  phil may want to
change.
(Assignee)

Updated

18 years ago
Status: REOPENED → ASSIGNED
(Assignee)

Updated

18 years ago
Blocks: 1775
(Assignee)

Comment 12

18 years ago
I have the fix in hand waiting for review. The html to txt converter is
addressed in bug 1775.

Comment 13

18 years ago
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).
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 14

18 years ago
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.

Updated

18 years ago
Status: RESOLVED → VERIFIED

Comment 15

18 years ago
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.