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)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b2

People

(Reporter: asuth, Assigned: asuth)

Details

(Whiteboard: [no l10n impact][target frozen])

Attachments

(1 file)

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?
Priority: -- → P2
moving to b2
Target Milestone: Thunderbird 3.0b1 → Thunderbird 3.0b2
Flags: blocking-thunderbird3? → blocking-thunderbird3+
Whiteboard: [no l10n impact]
Whiteboard: [no l10n impact] → [no l10n impact][target frozen]
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 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+
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.