forwarded 565903 https://bugzilla.mozilla.org/show_bug.cgi?id=522450 thanks
Hello Emilio, On Tue, Jan 19, 2010 at 10:28:07PM +0100, Emilio Pozuelo Monfort wrote: > On 19/01/10 15:57, Emilio Pozuelo Monfort wrote: > >I have 'Place replies in the folder of the message being replied to' option > >enabled. It works if I 'reply' or 'reply to all' to a message, but not if > >I 'reply to list'. When replying to list, the reply is stored in the Sent > >folder, and not in that of the replied message. > > I've looked at it and cooked a patch that fixed it for me. I've also > attached it to the upstream bug report. > > Cheers, > Emilio > diff -ruNp icedove-3.0/mailnews/compose/src/nsMsgSend.cpp > icedove-3.0.new/mailnews/compose/src/nsMsgSend.cpp > --- icedove-3.0/mailnews/compose/src/nsMsgSend.cpp 2009-12-04 > 14:15:26.000000000 +0100 > +++ icedove-3.0.new/mailnews/compose/src/nsMsgSend.cpp 2010-01-19 > 16:14:28.000000000 +0100 > @@ -2890,6 +2890,7 @@ nsMsgComposeAndSend::InitCompositionFiel > aType == nsIMsgCompType::Reply || > aType == nsIMsgCompType::ReplyAll || > aType == nsIMsgCompType::ReplyToGroup || > + aType == nsIMsgCompType::ReplyToList || > aType == nsIMsgCompType::ReplyToSender || > aType == nsIMsgCompType::ReplyToSenderAndGroup || > aType == nsIMsgCompType::ReplyWithTemplate ) it seems that no one upstream take care about your patch. And the I think your reported issue is still there (I haven't tested it). I took a look into mailnews/compose/src/nsMsgSend.cpp and this file looks for Icedove 17.0.1 like this. >2792 // Only check whether the user wants the message in the original >message >2793 // folder if the msgcomptype is some kind of a reply. >2794 if (!aOriginalMsgURI.IsEmpty() && ( >2795 aType == nsIMsgCompType::Reply || >2796 aType == nsIMsgCompType::ReplyAll || >2797 aType == nsIMsgCompType::ReplyToGroup || >2798 aType == nsIMsgCompType::ReplyToSender || >2799 aType == nsIMsgCompType::ReplyToSenderAndGroup || >2800 aType == nsIMsgCompType::ReplyWithTemplate ) >2801 ) This issue must be fixed by mozilla, in the between time we made a own patch for one of the next releases of icedove. Localy it works right now. I think Chris will implement this in the upcomming 17.0.3 version. Guido has pushed this new patch upstream to mozilla, so hopefully it will applied in the near time in one of the next releases. Thanks for your good "pre" work! :) Regards Carsten -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
