15556 - User pref "mail.forward_message_mode" should stick with 4.x

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[jefft]

User pref "mail.forward_message_mode" should stick with the 4.x implementation
to ease the migration. Although we are not having forward as quoted mode, we
should tread both 1 & 2 as forward inline.

Comment 1

[jefft]

Treat old , mail.forward_message_mode, forward quoted(1) as inline (2). Use
forward as attachment, the default if we failed on getting the pref.
nsMessenger.cpp changed.

Comment 2

[laurel]

Migration keeps value as is, however 5.0 prefs UI doesn't change based on any
value for the migrated pref whereas it does work to write correct value if the
pref is set through 5.0 UI.

Will verify this bug when bug #21219 is fixed.

Comment 3

[laurel]

Just an update:  related bug #21219 was fixed and reopened; still some issues
surrounding this pref in 5.0. Will still wait to verify this bug until final
resolution of #21219.

Comment 4

[laurel]

I'm going to go ahead and mark this verified, although the UI is still not
showing Inline after migration of a profile having "forward quoted" value 1 set
for the mail.forward_message_mode.

1.  4.x profile is set to forward quoted.
2.  Migrate, check prefs.js for 6.0:  still value left at 1.
3.  Check 6.0 ui:  set to attachment.
4.  Forward a message in 6.0 right after migration: forwards inline.

So... in this case, the UI still doesn't show the right thing, but the right
thing is done when forwarding.

Marking OK using oct09 mn6 branch commercial build.

Comment 5

[(not reading, please use [e-mail] instead)]

laurel, is there a bug logged on the UI problem?