711909 - "Replied" status of replied mail not set, when reply message saved as draft & edited multiple times then a) sent, or b) sent Send Later

Status: RESOLVED DUPLICATE of bug 530528

(MailNews Core :: Backend, defect)

Description

[WADA:World Anti-bad-Duping Agency]

+++ This bug was initially created as a clone of Bug #710584 +++
Because comment #0 of Bug #710584 was incorrect, that bug is confusing, so I reported again his bug.

Build ID:
> Mozilla/5.0 (Windows NT 5.1; rv:8.0) Gecko/20111105 Thunderbird/8.0

"Replied" status of replied mail is not set, if reply message is saved as draft & edited multiple times and sent, or reply message is saved as drafts & edited and saved in Outbox by Send Later.
This is applicable to replied mail in both local mail folder and IMAP mail folder.

[Steps to reproduce]
(0-1) Disable auto-compact in order to rule out bug 268589,
      and keep file size of file for local mail folder less than 2GB
      in order to rule out bug 639702 
      if replied mail is in local mail folder.
(0-2) Set draft folder for an identity to local mail folder at Copies&Folders.
(1) mail-1
(2-1) Reply, Save as draft, exit mail composition.
(2-2) Edit draft, Save as draft, exit mail composition
(2-3) Edit draft second time, Send => mail-1 is not set as Replied
(2) mail-2
(2-1) Reply, Save as draft, exit mail composition.
(2-2) Edit draft, Send Later(saved in Outbox of Local Folders)
(2-3) Send Unsent Messages => mail-2 is not set as Replied

This bug is already reproduced by bug 522336 comment #24.

Note:
Phenomenon of this bug is applicable to local Drafts folder only, because bug 530528 occurs before this bug if IMAP Drafts folder.

Comment 2

[WADA:World Anti-bad-Duping Agency]

Note:
This bug occurs in next build too.
> 2007-08-10-04-comm-central : one-week-after-final-FIXED-of-bug-128996[2007-07-30]
So, this bug is never regression over fix of bug 128996.

Comment 3

[WADA:World Anti-bad-Duping Agency]

Attachment: Local Drafts.msf after Reply, Save as draft, Exit composition window

(9E=replied) is seen in .msf.

> <(9D=mailbox-message://nobody@Local%20Folders/Inbox#764)(9E=replied)
>   (9F=id1)>[-0:^80(^88=10)(^8A=0)(^8B=21)(^82^93)(^85^94)(^81^96)(^83^97)

Comment 4

[WADA:World Anti-bad-Duping Agency]

Attachment: Local Drafts.msf after Edit draft, Save as draft(second save), Exit composition window

(9E=replied) is not seen in .msf. (9E=replied) looks lost when second Save as draft.

> ied-test>)(8E=ISO-8859-1)(9B=4)(9C=ffffffff)(9D
>     =mailbox-message://nobody@Local%20Folders/Inbox#764)(9F=id1)>
> {0:^80 {(k^98:c)(s=9)1:m } [408(^88=10)(^8A=0)(^8B=21)(^82^93)(^85^94)

Comment 5

[WADA:World Anti-bad-Duping Agency]

Attachment: Unsent Messages.msf after Reply, Save as draft, Exit composition window, Edit draft, Send Later

(9E=replied) is not seen in .msf. (9E=replied) looks lost when Edit draft/Send Later.

> ied-test>)(98=ISO-8859-1)(99=421)(9A=4)(9B=ffffffff)(9C
>     =mailbox-message://nobody@Local%20Folders/Inbox#764)(9D=id1)(9E
>     =16|Boyacky at RocketMail)>

Comment 6

[WADA:World Anti-bad-Duping Agency]

cc-ing to David :Bienvenu who is Assigned To: person of bug 128996.

Comment 7

[WADA:World Anti-bad-Duping Agency]

http://mxr.mozilla.org/comm-central/source/mailnews/compose/src/nsMsgCompose.cpp#1591
> 1591 nsresult nsMsgCompose::CreateMessage(const char * originalMsgURI,
> 1592                                      MSG_ComposeType type,
> 1593                                      nsIMsgCompFields * compFields)
Because URI of replied mail is kept as expected, it seems MSG_ComposeType=null is cause.

> 1602   // check if we're dealing with an opened .eml file msg
> 1604   if (fileUrl)
> 1617   else // check if we're dealing with a displayed message/rfc822 attachment
Change like next in nsMsgCompose::CreateMessage is needed?
  // check if we're dealing with an opened .eml file msg
  if (fileUrl) ...
  else if(Editing of existent draft)
       { restore MSG_ComposeType from meta data in .msf file; }   
  else // check if we're dealing with a displayed message/rfc822 attachment
Or fault in caller of nsMsgCompose::CreateMessage when Edit draft?

Comment 8

[Radim Kolar]

This is really annoying bug.

Comment 9

[Wayne Mery (:wsmwk)]

(hope the topic change is accurate)

Comment 10

[grifter.fps]

+1

Comment 11

[Magnus Melin [:mkmelin]]

This was fixed in bug 530528 (thunderbird26 +)