Closed
Bug 466067
Opened 16 years ago
Closed 15 years ago
gloda should convert messages with only HTML bodies to text for full-text indexing
Categories
(MailNews Core :: Database, defect, P2)
MailNews Core
Database
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b2
People
(Reporter: asuth, Assigned: asuth)
Details
(Whiteboard: [no l10n impact][target frozen])
Attachments
(1 file)
13.97 KB,
patch
|
Bienvenu
:
review+
|
Details | Diff | Splinter Review |
If a message only has a text/html and no text/plain, it won't get full-text indexed. We should use the html to text conversion support available to create a text representation. Marking blocking because people will probably be sad and confused if they can't search their HTML emails.
Flags: blocking-thunderbird3?
Assignee | ||
Updated•16 years ago
|
Priority: -- → P2
Updated•15 years ago
|
Flags: blocking-thunderbird3? → blocking-thunderbird3+
Updated•15 years ago
|
Whiteboard: [no l10n impact]
Updated•15 years ago
|
Whiteboard: [no l10n impact] → [no l10n impact][target frozen]
Assignee | ||
Comment 2•15 years ago
|
||
None of the interfaces we need are scriptable; they're all hand-rolled (.h, not .idl) XPCOM implementations. The implementations themselves are in mozilla/, making conversion unpalatable/unlikely. I fear the 'best' course of action may be to create an XPCOM wrapper. This patch simply exposes the nsMsgDBFolder implementation of a conversion wrapper function used by GetMsgTextFromStream. It's a kludge, but it works. Input desired/requested.
Attachment #362557 -
Flags: review?(bienvenu)
Comment 3•15 years ago
|
||
Comment on attachment 362557 [details] [diff] [review] v1 kludge expose nsMsgDBFolder::ConvertMsgSnippetToPlainText +nsresult nsMsgDBFolder::ConvertMsgSnippetToPlainText(const nsAString& aMessageText, should be NS_IMETHODIMP other than that, seems OK, and definitely needed. Yeah, I remember not being able to get at this stuff in js when I did the spotlight indexer
Attachment #362557 -
Flags: review?(bienvenu) → review+
Assignee | ||
Comment 4•15 years ago
|
||
pushed with change http://hg.mozilla.org/comm-central/rev/4b77887894f1
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•