242973 - when MAPISend is called with a plain text note (lpszNoteText) with newlines, we lose formatting

Status: RESOLVED FIXED

(MailNews Core :: Simple MAPI, defect)

Description

[Scott MacGregor]

when MAPISend is called with a plain text note (lpszNoteText) with newlines, we
lose the formatting.

it is because we insert that text as html.

see
http://lxr.mozilla.org/mozilla/source/mailnews/mapi/mapihook/src/msgMapiHook.cpp#502

Comment 1

[Rob Janssen]

Is a duplicate of bug 196881

Comment 2

[David :Bienvenu]

Attachment: proposed fix

send message as plain text

Comment 3

[David :Bienvenu]

Comment on attachment 152631 [details] [diff] [review]
proposed fix

send mapi messages as plain text

Comment 4

[David :Bienvenu]

Attachment: fix non-blind send case

if we're bringing up the compose ui, we can't just use plain text, so this fix
converts '\n' to <br> so that lines are retained.

Comment 5

[David :Bienvenu]

fixed on trunk, 1.0 branch, and m4 branch

Comment 6

[David :Bienvenu]

previous fix causes problems with edit | draft/template with html
messages...reopening. new fix upcoming.

Comment 7

[David :Bienvenu]

Attachment: proposed fix

check the compose type before converting crlfs (the mailto url doesn't work
because the caller code seems to strip out the crlfs, but if it didn't, this
would work )

Comment 8

[David :Bienvenu]

Attachment: proposed fix (prev attachment was from wrong dir)

Comment 9

[David :Bienvenu]

Comment on attachment 156618 [details] [diff] [review]
proposed fix (prev attachment was from wrong dir)

check the compose type before converting crlfs (the mailto url doesn't work
because the caller code seems to strip out the crlfs, but if it didn't, this
would work )

Comment 10

[David :Bienvenu]

fixed on trunk and branch.

Comment 11

[Mike Cowperthwaite]

*** Bug 196881 has been marked as a duplicate of this bug. ***