HTML intelligent send fires upon Save as Draft/Template

VERIFIED FIXED in M15

Status

MailNews Core
Composition
P3
normal
VERIFIED FIXED
19 years ago
10 years ago

People

(Reporter: laurel, Assigned: Jean-Francois Ducarroz)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PDT-])

(Reporter)

Description

19 years ago
Using feb 22 builds, all platforms

The HTML Intelligent Send question dialog fires upon saving a message
composition as Draft or Template.  The intelligent send stuff shouldn't surface
until we send the message. 

1.  Edit|Prefs|Mail&newsgroups|Formatting. Enable the first option "Ask me what
to do when message has HTML formatting" (default).
2.  Compose an html message to test9@foobar
3.  From the compose window, File|Save As|Draft.
   
Result:  HTML Intelligent Send dialog displays asking about what format in which
to send the message.

Expected result:  Message should save to draft without asking about format. 
Formatting question should be asked upon sending the message.
(Reporter)

Updated

19 years ago
QA Contact: lchiang → fenella
(Reporter)

Comment 1

19 years ago
Correcting summary
Summary: message → HTML intelligent send fires upon Save as Draft/Template
(Assignee)

Comment 2

19 years ago
Accepting. Should be am easy fix.
Status: NEW → ASSIGNED
Whiteboard: [dogfood]
Target Milestone: M14

Comment 3

19 years ago
Moving dogfood to keyword field from Status Summary.
Keywords: dogfood
Whiteboard: [dogfood]

Comment 4

19 years ago
Putting on PDT- radar for dogfood. An annoyance, but will not hold for this.
Whiteboard: [PDT-]
(Assignee)

Comment 5

19 years ago
The easy fix is o check is we are sending now or later the message before 
checking for eventual HTML formatting conflict.

Index: MsgComposeCommands.js
===================================================================
RCS file: 
/cvsroot/mozilla/mailnews/compose/resources/content/MsgComposeCommands.js,v
retrieving revision 1.73
diff -r1.73 MsgComposeCommands.js
451,479c451,482
<
<                   // Before sending the message, check what to do with HTML 
message, eventually abort.
<             action = DetermineHTMLAction();
<             if (action == msgCompSendFormat.AskUser)
<             {
<                 var result = {action:msgCompSendFormat.PlainText, 
abort:false};
<                   
window.openDialog("chrome://messengercompose/content/askSendFormat.xul",
<                                       "askSendFormatDialog", "chrome,modal",
<                                       result);
<                   if (result.abort)
<                       return;
<                    action = result.action;
<             }
<             switch (action)
<             {
<                 case msgCompSendFormat.PlainText:
<                     msgCompFields.SetTheForcePlainText(true);
<                     msgCompFields.SetUseMultipartAlternativeFlag(false);
<                     break;
<                 case msgCompSendFormat.HTML:
<                     msgCompFields.SetTheForcePlainText(false);
<                     msgCompFields.SetUseMultipartAlternativeFlag(false);
<                     break;
<                 case msgCompSendFormat.Both:
<                     msgCompFields.SetTheForcePlainText(false);
<                     msgCompFields.SetUseMultipartAlternativeFlag(true);
<                               break;
<                default: dump("\###SendMessage Error: invalid action value\n"); 
return;
<             }
---
>
>                       if (msgType == msgCompDeliverMode.Now || msgType == 
msgCompDeliverMode.Later)
>                       {
>                               // Before sending the message, check what to do 
with HTML message, eventually abort.
>                               action = DetermineHTMLAction();
>                               if (action == msgCompSendFormat.AskUser)
>                               {
>                                       var result = 
{action:msgCompSendFormat.PlainText, abort:false};
>                                       
window.openDialog("chrome://messengercompose/content/askSendFormat.xul",
>                                                                               
"askSendFormatDialog", "chrome,modal",
>                                                                               
result);
>                                       if (result.abort)
>                                               return;
>                                        action = result.action;
>                               }
>                               switch (action)
>                               {
>                                       case msgCompSendFormat.PlainText:
>                                               
msgCompFields.SetTheForcePlainText(true);
>                                               
msgCompFields.SetUseMultipartAlternativeFlag(false);
>                                               break;
>                                       case msgCompSendFormat.HTML:
>                                               
msgCompFields.SetTheForcePlainText(false);
>                                               
msgCompFields.SetUseMultipartAlternativeFlag(false);
>                                               break;
>                                       case msgCompSendFormat.Both:
>                                               
msgCompFields.SetTheForcePlainText(false);
>                                               
msgCompFields.SetUseMultipartAlternativeFlag(true);
>                                               break;
>                                  default: dump("\###SendMessage Error: invalid 
action value\n"); return;
>                               }
>                       }
Whiteboard: [PDT-] → [PDT-]fix in hand
(Assignee)

Comment 6

19 years ago
Code reviewed by mscott
(Assignee)

Comment 7

19 years ago
Fixed and checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
Whiteboard: [PDT-]fix in hand → [PDT-]
(Assignee)

Comment 8

19 years ago
Change TM to M15
Target Milestone: M14 → M15

Comment 9

19 years ago
Linux (2000-03-17-08 M15) commercial trunk
MAc (2000-03-17-08 M15) commercial trunk
Win 32 (2000-03-17-09 M15) commercial trunk
This problem is fixed in the trunk builds of all 3 platforms. Need to test it in
the beta 1 build.

Comment 10

19 years ago
Per ducarroz, this is is PDT- bug, so it would not be in the beta1 builds. 
Mark it verified.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.