Closed
Bug 284381
Opened 20 years ago
Closed 13 years ago
open/view/save attachments of .eml file in browser
Categories
(MailNews Core :: MIME, defect)
MailNews Core
MIME
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 707631
People
(Reporter: nian.liu, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [patchlove][only affects seamonkey])
Attachments
(1 file, 6 obsolete files)
(deleted),
message/rfc822
|
Details |
1.open eml file in browser
2.for the attachment, there is only a table shows it is there. However, there is
no way to access it.
Reporter | ||
Comment 1•20 years ago
|
||
Reporter | ||
Comment 2•20 years ago
|
||
Reporter | ||
Comment 3•20 years ago
|
||
file mimeeobj.cpp is missed in last patch
Attachment #175999 -
Attachment is obsolete: true
Reporter | ||
Comment 4•20 years ago
|
||
Boris,
The reason why I didn't reuse the code in nsMultiMixedConv.cpp are
1)For nsStreamConverter, nsPartChannel is much easier than in nsMultiMixedConv.
An adapter pattern is enough.
2)To reuse the code in nsMultiMixedConv, local class nsPartChannel must be
refactored to a public class which means add new files, change make files, and
so on. I'm not sure whether this is acceptable in community.
Pls. advice.
Comment 5•20 years ago
|
||
There is nothing wrong with adding new files and changing makefiles.
If there is really no code to be shared with nsMultiMixedConv, then I can see
not sharing the code. Use a different class name, though, otherwise static
builds that link in both classes will break.
Reporter | ||
Comment 6•20 years ago
|
||
In fact, I think nsPartChannel in nsMultiMixedConv can use part of code in
nsStreamConver. And I want to file new bug to refactor after this bug is fixed.
Reporter | ||
Comment 7•20 years ago
|
||
rename class name in case of static link conflict according to Boris's
suggestion
Attachment #176001 -
Attachment is obsolete: true
Hmm ... I don't see a difference to your Bug 270115. I guess the other could be
duped against this one.
Blocks: 269826
Summary: open attachment of eml file in browser → open attachment of .eml file in browser
See Bug 264167 too.
Comment 10•20 years ago
|
||
Nian, the code you're adding is in mailnews, while nsPartChannel is in necko.
Mailnews can depend on necko for proper functioning, but not the other way around.
Reporter | ||
Comment 11•20 years ago
|
||
*** Bug 270115 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 12•20 years ago
|
||
Boris, I see. So after this is done, I'll fix another bug to refactor which
makes the reuse part kept in necko.
Reporter | ||
Comment 13•20 years ago
|
||
refer 264167 and make link part in table
Attachment #176017 -
Attachment is obsolete: true
Comment 14•20 years ago
|
||
Why not just do it right the first time instead of filing a followup bug to do
it right?
Reporter | ||
Comment 15•20 years ago
|
||
I just wnat to concentrate on issues one by one. However, I'll try to make a
patch including refactor part. Before that, Can you have a look on patchv2?
Comment 16•20 years ago
|
||
I'm really not the best person to review mailnews code... I don't know much
about it.
Updated•20 years ago
|
OS: Linux → All
Hardware: PC → All
Reporter | ||
Comment 17•20 years ago
|
||
*** Bug 264167 has been marked as a duplicate of this bug. ***
Reporter | ||
Updated•20 years ago
|
Attachment #176096 -
Flags: review?(bienvenu)
Summary: open attachment of .eml file in browser → open/view/save attachments of .eml file in browser
Comment 18•20 years ago
|
||
Would this patch fixing bug 270772 and bug 174692?
Comment 19•20 years ago
|
||
Comment on attachment 176096 [details] [diff] [review]
patchv2
Jean-Francois should review this, since it's mime.
Attachment #176096 -
Flags: superreview?(bienvenu)
Attachment #176096 -
Flags: review?(ducarroz)
Attachment #176096 -
Flags: review?(bienvenu)
Comment 20•20 years ago
|
||
If all the nsIRequest methods forward to mChannel, why not just use the macro we
have for the purpose?
What's with the random printf you're checking into the tree?
Reporter | ||
Comment 21•20 years ago
|
||
for comment#18,
no, image display uses different module
Reporter | ||
Comment 22•20 years ago
|
||
(In reply to comment #20)
> If all the nsIRequest methods forward to mChannel, why not just use the macro we
> have for the purpose?
Pls. show me any example of it
>
> What's with the random printf you're checking into the tree?
>
Sorry, debug code and my mistake
>
Comment 23•20 years ago
|
||
> Pls. show me any example of it
http://lxr.mozilla.org/seamonkey/source/modules/libpr0n/decoders/icon/gtk/nsIconChannel.h#54
All idl interfaces automatically define a like macro (just like they define
NS_DECL_*).
Reporter | ||
Comment 24•20 years ago
|
||
modified according Boris's suggestion in comment#23
BTW, Boris, it's really fantastic! Can I find any backgroud knowledge of that
design?
Comment 25•20 years ago
|
||
Maybe in the xpidl documentation?
Comment 26•20 years ago
|
||
Sorry for the delay! I am really busy right now, I'll try to review it next week.
Comment 27•20 years ago
|
||
Comment on attachment 176543 [details] [diff] [review]
patchv2.1
Patch looks good. R=ducarroz
Attachment #176543 -
Flags: superreview?(bienvenu)
Attachment #176543 -
Flags: review+
Comment 28•20 years ago
|
||
I presume patchv2 can be marked as obsolete?
Reporter | ||
Comment 29•20 years ago
|
||
Comment on attachment 176096 [details] [diff] [review]
patchv2
as your wish^_^
Attachment #176096 -
Attachment is obsolete: true
Reporter | ||
Comment 30•20 years ago
|
||
David, can you help me sr this patch or is there someone else you recommend to
sr it?
Comment 31•20 years ago
|
||
I tried this patch against seamonkey - it applied and built, but what effect
should I have seen when I opened a .eml file in the browser? I didn't see the
attachments displayed in such a way that I could do anything with them...
Reporter | ||
Comment 32•20 years ago
|
||
There should be a link for each attachment. With the link, you can open/save the
attachment.
Comment 33•20 years ago
|
||
with my trunk seamonkey build, and this patch, I don't even see a table or link
for the attachments - they just display inline as hex or text attachments.
Reporter | ||
Comment 34•20 years ago
|
||
not sure what made that. with my src(seamonkey downloaded May 20th) and this
patch , test case works fine.
Comment 35•20 years ago
|
||
(In reply to comment #34)
> not sure what made that. with my src(seamonkey downloaded May 20th) and this
> patch , test case works fine.
Worksforme?
Comment 36•20 years ago
|
||
"with this patch". The patch is not checked in. Please test things yourself
before saying "worksforme", ok?
Comment 37•20 years ago
|
||
(In reply to comment #36)
> "with this patch". The patch is not checked in. Please test things yourself
> before saying "worksforme", ok?
Sorry. I read that as if it were checked in.
Updated•19 years ago
|
Flags: blocking-aviary2.0?
Reporter | ||
Comment 38•19 years ago
|
||
David,
Can you sr this patch or attach your local pathc of related bugs?
Comment 39•19 years ago
|
||
Comment on attachment 176543 [details] [diff] [review]
patchv2.1
I tried this patch on some other .eml files, and it did work - I don't remember
what file it didn't work on before, but it did work on your test case, in a
way...except that since thunderbird is now registered as the handler for .eml
files, opening the zip file in your test case says that it's going to open with
winzip but clicking on the link ends up just launching thunderbird on the .eml
file. I'm not sure if that behaviour is acceptable to seamonkey.
+ *aReturn = mChannel;
+ NS_IF_ADDREF(*aReturn);
this can be NS_IF_ADDREF(*aReturn = mChannel);
+ if (filenameField) {
+ mContentDisposition.Append("inline; filename=");
+ mContentDisposition.Append(filenameField);
+ }
the prevailing braces style in this file is
if (a)
{
...
}
not,
if (a) {
...
}
can you fix those? There are several instances of this.
I'm going to try this patch in Thunderbird to try to make sure it doesn't break
anything (since this is shared code). If you address my nits, and I verify that
it doesn't break thunderbird, then I'll sr+ this. Sorry for the long delay!
Attachment #176543 -
Flags: superreview?(bienvenu) → superreview-
Comment 40•19 years ago
|
||
this fixes the opening of attachments from saved local messages - the mailbox
uri in the case of saved local messages is not useful, so we use the url, which
should always be useful. I'll look into the account central issue next.
Assignee: nian.liu → bienvenu
Status: NEW → ASSIGNED
Attachment #193466 -
Flags: superreview?(mscott)
Comment 41•19 years ago
|
||
Comment on attachment 193466 [details] [diff] [review]
fix opening of attachments from saved messages
wrong bug, sorry
Attachment #193466 -
Attachment is obsolete: true
Attachment #193466 -
Flags: superreview?(mscott)
Reporter | ||
Comment 42•19 years ago
|
||
(In reply to comment #39)
> (From update of attachment 176543 [details] [diff] [review] [edit])
> I tried this patch on some other .eml files, and it did work - I don't remember
> what file it didn't work on before, but it did work on your test case, in a
> way...except that since thunderbird is now registered as the handler for .eml
> files, opening the zip file in your test case says that it's going to open with
> winzip but clicking on the link ends up just launching thunderbird on the .eml
> file. I'm not sure if that behaviour is acceptable to seamonkey.
>
seems open take the eml file instead of the attachment. also wrong with linux.
save is OK. I'll investigate it.
Reporter | ||
Comment 43•19 years ago
|
||
works for remote file. doesn't work for local file. related to bug 73757
Reporter | ||
Comment 44•19 years ago
|
||
Comment on attachment 176543 [details] [diff] [review]
patchv2.1
David, pls. re-sr the patch since 309241 is checked in and should resolve the
local file issue
Attachment #176543 -
Flags: superreview- → superreview?(bienvenu)
Comment 45•19 years ago
|
||
Comment on attachment 176096 [details] [diff] [review]
patchv2
Cancelling review request on obsolete patch.
Attachment #176096 -
Flags: superreview?(bienvenu)
Attachment #176096 -
Flags: review?(ducarroz)
Updated•16 years ago
|
QA Contact: mime
Assignee | ||
Updated•16 years ago
|
Product: Core → MailNews Core
Updated•16 years ago
|
Attachment #176543 -
Attachment is obsolete: true
Attachment #176543 -
Flags: superreview?(bienvenu)
Comment 47•16 years ago
|
||
Comment on attachment 176543 [details] [diff] [review]
patchv2.1
$ patch -p0 --dry-run < ~/Desktop/patch21
patching file emitters/src/nsMimeHtmlEmitter.cpp
Hunk #1 succeeded at 472 (offset 13 lines).
patching file src/nsStreamConverter.h
Hunk #1 FAILED at 101.
1 out of 1 hunk FAILED -- saving rejects to file src/nsStreamConverter.h.rej
patching file src/nsStreamConverter.cpp
Hunk #1 FAILED at 69.
Hunk #2 FAILED at 621.
Hunk #3 FAILED at 1128.
3 out of 3 hunks FAILED -- saving rejects to file src/nsStreamConverter.cpp.rej
Already bit-rotted, cancelling sr request. Can an updated patch please be provided?
Comment 48•16 years ago
|
||
As part of patchlove, bienvenu, is this 3.5 year old patch still relevant?
Updated•16 years ago
|
Flags: wanted-thunderbird3?
Comment 49•16 years ago
|
||
I could not get it to work at the time; since the problem, if not the solution, is specific to SeaMonkey, a SeaMonkey person might be better suited to un-bitrotting the patch and checking if it fixes the problem.
Updated•16 years ago
|
Flags: wanted-thunderbird3?
Updated•16 years ago
|
Flags: wanted-thunderbird3?
Comment 50•16 years ago
|
||
wanted‑thunderbird3- as this wouldn't affect thunderbird in any user visible way.
Flags: wanted-thunderbird3? → wanted-thunderbird3-
Comment 51•16 years ago
|
||
I don't see this anymore in:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090419 Lightning/1.0pre Shredder/3.1a1pre ID:20090419051533
Mozilla/5.0 (Windows; U; Windows NT 6.0; el; rv:1.9.1b3pre) Gecko/20090223 Thunderbird/3.0b2 ID:20090223175111
or:
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre) Gecko/20090227 Lightning/1.0pre Shredder/3.0b2
Fixed then?
Comment 52•16 years ago
|
||
Klonos, this seems to be a SeaMonkey issue (see comment #49 ), please test in SeaMonkey to be sure.
Comment 53•16 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b5pre) Gecko/20090425 SeaMonkey/2.0b1pre
Bug is still there in current SeaMonkey trunk build on WinXP.
To make it clear: The bug shows only in the *browser* part. If you open an eml file in the mail part it works (in Thunderbird too). This bug is like the others in comment 18 SeaMonkey browser specific, but the responsible code is in MailNews (which is shared between TB & SM). Therefore the product of these bugs (MailNews Core, not SeaMonkey).
Comment 54•16 years ago
|
||
Ok then, I thought we needed to set this to SeaMonkey only. Now that OstGote clarified that it is, but responsible code is in MailNews I'm shutting up ;)
PS (to comment #52): I am using tb and fx at the moment + trying to setup some extra VMs to test things more and in non-production environment. As soon as I have them ready I'll try to install latest SeaMonkey as well and test things. Till then, others can confirm/test on sm.
Updated•16 years ago
|
Whiteboard: [patchlove] → [patchlove][only affects seamonkey]
Comment 55•13 years ago
|
||
Opening .eml files in the browser is not intended behaviour.
Assignee: dbienvenu → nobody
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•