Closed
Bug 96314
Opened 24 years ago
Closed 24 years ago
unable to save msg unless given .html or .txt extension
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.5
People
(Reporter: bugzilla, Assigned: sspitzer)
References
Details
(Keywords: dataloss, platform-parity, regression, Whiteboard: [PDT])
Attachments
(1 file, 1 obsolete file)
904 bytes,
patch
|
Bienvenu
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
i'm unable to save a mail message unless i provide an .html or .txt extension to
the filename. found using 2001.08.21.08-comm bits on linux --however, i've seen
it on 2001.08.15.14, so not sure how long this has been around.
1. while viewing a mail message in the 3pane window, either bring up the context
menu and select Save As, or select File > Save As > File from the main menu.
2. in the resulting file picker, either keep the default name ["mail"], or
change it to something like mail-blah.eml.
3. click the Save button. [side note: if that button is greyed-out and is
labelled as Open, just click in the File Name field. that's bug 94154.]
4. check the dir in which you tried to save the file.
result: the file isn't saved.
notes:
* this seems mailnews-specific. not a problem when saving files in the browser.
* .eml appears to be the default extension for mail messages, yes? that's the
displayed type in the file picker. odd that you cannot save with that extension.
* workarounds: if i append .html or .txt to the filename, i can save the file.
or, copying the msg as a new local folder also works [kinda kludgy].
Reporter | ||
Comment 1•24 years ago
|
||
this is a regression. using 2001.08.09.10-comm bits, i'm able to save a msg
without appending .html or .txt.
Keywords: regression
Reporter | ||
Updated•24 years ago
|
Keywords: mozilla0.9.4
Updated•24 years ago
|
QA Contact: esther → sheelar
Reporter | ||
Comment 2•24 years ago
|
||
still a problem using 2001.08.31.08-comm bits on linux. unable to repro on mac
2001.08.31.10-comm.
bug 97863, involving saving as template, also seems linux-only. related? [then
again, i currently don't have access to test this on win32.]
Comment 3•24 years ago
|
||
Seth, this really sucks. Most users won't realize that they need to put an
extention on it and may save messages and delete them from mail. Can you please
look into it. I suspect that PDT will want this for emojo and I definitely want
it for 0.9.5. Thanks.
Keywords: dataloss
Comment 5•24 years ago
|
||
An existing message or received message can be saved on the below mentioned
platforms.
using 2001-10-01-05 commercial build on win98 and MAC OS 9.1 and X
This is a linux only problem. Saving a message without extension worked on
win32 and mac.
Keywords: pp
Comment 6•24 years ago
|
||
on linux i was able to save a message if i added a .txt extention but it
did not save the entire message. a bunch of headers went missing.
Assignee | ||
Comment 9•24 years ago
|
||
.html and .txt take different code paths then .eml, .foobar or no extension at all.
also, I think this is an imap vs. local issue. (I think it should work for
local/pop, but not for imap).
testing a fix (fingers crossed), I've also got a minor change so that if you
save as "mail" (no extension), it saves it as mail.eml (since the file is really
an .eml)
Assignee | ||
Comment 10•24 years ago
|
||
punting until after 0.9.4 is done.
Assignee | ||
Comment 11•24 years ago
|
||
this will have to wait until 0.9.4 is done, but can someone investigate this?
does it fail on linux the message you are trying to save is local or news? (not
imap).
does this fail, when reading a message in your imap inbox, does
File | Save As | Template work if the template folder is local? if the
template folder is imap?
I bet more than just saving message as a file (on linux) is broken.
I bet one of those scenarios is also busted.
Reporter | ||
Comment 12•24 years ago
|
||
tested seth's scenarios above --mind you, the account i'm using is an imap account.
a. if the msg is in a *local* folder, i can save without specifying .txt or .html
b. if the msg is in a newsgroup, i can save without specifying .txt or .html
c. trying to save a message as a template --when the Templates folder is imap--
fails if it's via the thread pane or standalone. that's bug 97863.
d. trying to save a message as a template --when the Templates folder is local--
also fails --get "unable to save the message. please check your file name and
try again later".
Comment 13•24 years ago
|
||
From today's branch build: 2001-10-03-05 linux. Some more scenarios that work
and don't work if it helps.
-If you compose a new message and save as template to your local template
folder, you can save the message.
-Saving as template an existing message to the template folder from an imap
account is the problem whether it is saved to imap template folder or local
template folder.
POP account-I just tried saving an existing message from 3 pane and stand alone
without extension and the message is saved.
Saving newly composed message as well as an existing message to local template
folder and pop template folder worked on linux for pop account.
Reporter | ||
Comment 14•24 years ago
|
||
forgot to mention: the tests in my 2001-10-03 13:59 comments were done using
2001.10.03.08-trunk comm bits.
Assignee | ||
Comment 15•24 years ago
|
||
found the problem.
we are setting up the file stream and then delete the file spec.
here comes the patch. simple patch, might be good for the branch.
Assignee | ||
Comment 16•24 years ago
|
||
Assignee | ||
Comment 17•24 years ago
|
||
Assignee | ||
Comment 18•24 years ago
|
||
this fixes the problem, but I'm noticing some line ending issues with the file
we send.
but luckily, the .eml file we do save does get display properly when you load
it in mozilla (even with the mixed line endings).
I'll log a seperate bug for that.
Assignee | ||
Comment 19•24 years ago
|
||
s/send/save
Assignee | ||
Updated•24 years ago
|
Attachment #51977 -
Attachment is obsolete: true
Updated•24 years ago
|
Attachment #51975 -
Flags: superreview+
Assignee | ||
Comment 20•24 years ago
|
||
I think this fix would be good for the branch, as it fixes this bug and "save
as template" on linux.
mscott, can you re-assess?
Keywords: nsbranch-
Comment 21•24 years ago
|
||
Comment on attachment 51975 [details] [diff] [review]
patch
sr=bienvenu
Comment 22•24 years ago
|
||
r=naving
Comment 23•24 years ago
|
||
Comment on attachment 51975 [details] [diff] [review]
patch
a=asa (on behalf of drivers) for checkin to 0.9.5
Attachment #51975 -
Flags: approval+
Assignee | ||
Comment 24•24 years ago
|
||
fixed on the trunk for 0.9.5
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla0.9.5
Reporter | ||
Comment 25•24 years ago
|
||
tested using 2001.10.04.08-trunk comm bits on linux. looks good!
* save as file from an imap inbox works [gave a filename w/o an extension].
* save as file from a local folder works [gave a filename w/o an extension].
* save as file from a newsgroup works [gave a filename w/o an extension].
* save as template from an imap inbox works when Templates is an imap folder.
* save as template from an imap inbox works when Templates is a local folder.
Comment 26•24 years ago
|
||
thanks for QA'ing this bug so quickly sairuh. I think I'm going to take it to
PDT today since we have a QA'ed fix in hand already.
Comment 27•24 years ago
|
||
This was resolved | fixed, but it has not been checked into the trunk. Reopen.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: PDT → [PDT]
Comment 28•24 years ago
|
||
pls check this into the branch - PDT+
Comment 29•24 years ago
|
||
Jaime meant to say it hasn't been checked into the 094 branch yet. It is fixed
on the trunk.
Assignee | ||
Comment 30•24 years ago
|
||
fixed on branch and trunk.
Status: REOPENED → RESOLVED
Closed: 24 years ago → 24 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 31•24 years ago
|
||
vrfy fixed on the branch with 2001.10.05.04-0.9.4 linux bits. ran the same tests
as in my 2001-10-04 14:21 comments.
Status: RESOLVED → VERIFIED
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•