If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Redesign of header handling

NEW
Unassigned

Status

MailNews Core
Backend
P3
normal
18 years ago
9 years ago

People

(Reporter: BenB, Unassigned)

Tracking

(Blocks: 1 bug, {helpwanted})

Trunk
helpwanted
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
The headers code is wierd. There're at least 5 different definitions for
headers.

Trying to add a header will be not just painful, but be a potential source for
bugs. I'd vote for one or two classes, which (1) store the values and (2) know
the protocol.

Some relevant files:
http://lxr.mozilla.org/seamonkey/source/mailnews/compose/public/nsIMsgCompFields.idl
http://lxr.mozilla.org/seamonkey/source/mailnews/compose/src/nsMsgCompFields.cpp#573
http://lxr.mozilla.org/seamonkey/source/mailnews/mime/public/nsMailHeaders.h
http://lxr.mozilla.org/seamonkey/source/include/libmime.h
http://lxr.mozilla.org/seamonkey/source/mailnews/compose/src/nsMsgCompUtils.cpp#197
(UI files not listet)
(Reporter)

Updated

18 years ago
Status: NEW → ASSIGNED

Comment 1

18 years ago
http://x30.deja.com/viewthread.xp?AN=523516483&search=thread
(Reporter)

Updated

18 years ago
Target Milestone: M14
(Reporter)

Updated

18 years ago
Severity: minor → normal
Priority: P3 → P1
(Reporter)

Updated

18 years ago
Status: ASSIGNED → NEW
Priority: P1 → P3
Whiteboard: [HELP WANTED]
Target Milestone: M14 → M20
(Reporter)

Comment 2

18 years ago
Giving up for now :-(. Maybe later.
(Reporter)

Updated

18 years ago
Summary: Redesign of header handling → [HELP WANTED] Redesign of header handling
(Reporter)

Updated

18 years ago
Assignee: mozilla → nobody
(Reporter)

Updated

18 years ago
Blocks: 12306
(Reporter)

Updated

18 years ago

Updated

18 years ago
Keywords: helpwanted

Updated

18 years ago
Summary: [HELP WANTED] Redesign of header handling → Redesign of header handling
Whiteboard: [HELP WANTED]
Target Milestone: M20
(Reporter)

Comment 3

18 years ago
Created attachment 7547 [details]
Design as DIA 0.40 UML chart
(Reporter)

Comment 4

18 years ago
Created attachment 7548 [details]
Design as PNG
(Reporter)

Comment 5

18 years ago
This attached proposed design is the result of the mentioned discussion on
.mailnews. Not sure, if all this (including this bug) is still valid. Adding
alecf to CC, as he knows the current infrastructure well.

I added descriptions for arrows, because I don't know UML. Please point me to
errors.

Updated

15 years ago
QA Contact: lchiang → nobody

Comment 6

13 years ago
I am beginning a long an arduous process which hopefully will result in a
rewrite of the libmime code in a nice C++ish XPCOMish way (see bug 248846). If
anyone is interested in making such changes, I believe it will be easier when
there is no more legacy code in the obfuscated pseudo-object system which is
hardly interoperable with the rest of Mozilla. For the time being it would be
easier for me if the libmime code were left untouched. If anybody finds it
urgent to work on this bug right now, please coordinate with me. For now, I am
marking this bug dependant on the rewrite bug.
Depends on: 248846
Product: MailNews → Core
QA Contact: nobody → backend
(Assignee)

Updated

9 years ago
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.