Re[2]: Timestamp on Sent Messages

Vadim Zeitlin <[email protected]>
Newsgroups gmane.mail.mahogany.devel
Message-ID <[email protected]>
On Tue, 19 Jul 2005 14:11:15 +0200 Xavier Nodet <[email protected]> wrote:

XN> On Mon, 1 Dec 2003 23:04:02 +0100 (Romance Standard Time) Vadim Zeitlin <[email protected]> wrote:
XN> 
XN> > On Mon, 1 Dec 2003 03:03:03 -0800 (PST) Michael A Chase <[email protected]> wrote:
XN> 
XN> > MAC> I recently sent an email that had been sitting in the Drafts folder for
XN> > MAC> a week.  The timestamp reflected when I first saved the message, not
XN> > MAC> when I sent it.
XN> >
XN> >  This is a bug.
XN> 
XN> The attached patch corrects it.

 I admit to not quite understanding how it works but if it fixes the
problem, do check it in please...

XN> cvs -z3 diff -u -wb -- src\mail\SendMessageCC.cpp (in directory U:\src\M\)
XN> Index: src/mail/SendMessageCC.cpp
XN> ===================================================================
XN> RCS file: /cvsroot/mahogany/M/src/mail/SendMessageCC.cpp,v
XN> retrieving revision 1.230
XN> diff -u -w -b -r1.230 SendMessageCC.cpp
XN> --- src/mail/SendMessageCC.cpp	4 Oct 2004 12:30:22 -0000	1.230
XN> +++ src/mail/SendMessageCC.cpp	19 Jul 2005 10:39:02 -0000
XN> @@ -1271,7 +1271,9 @@
XN>     //
XN>     // NB: we do allow the user to override the date header because this is
XN>     //     useful when editing a previously postponed message
XN> -   if ( !HasHeaderEntry(_T("Date")) )
XN> +   // NB: If the message is not a resend, we'll add the Date header when
XN> +   //     sending the message (and not when saving it as Draft)
XN> +   if ( m_Envelope->remail && !HasHeaderEntry(_T("Date")) )
XN>     {

 I'd rather think that the date header should be only added if !forStorage
but I must be missing something?

 Thanks,
VZ



-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.