[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] Review Request 113254: imap resource: set the internal date as specified in the mail's
From:       "Wolfgang Bauer" <wbauer () tmo ! at>
Date:       2014-02-26 20:55:56
Message-ID: 20140226205556.31571.4924 () probe ! kde ! org
[Download RAW message or body]


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/113254/
-----------------------------------------------------------

(Updated Feb. 26, 2014, 8:55 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDEPIM and KDEPIM-Libraries.


Bugs: 230635
    http://bugs.kde.org/show_bug.cgi?id=230635


Repository: kdepim-runtime


Description
-------

Currently, for an APPEND command there is no internal date specified. \
Therefore an IMAP server sets the current date/time for an appended EMail, \
see http://tools.ietf.org/html/rfc3501#section-6.3.11 .

While this doesn't matter to KMail (it uses the date/time from the mail's \
header anyway), other clients (especially Webmail) may show/sort by this \
internal date.

This patch sets the internal date to the date/time as specified in the \
mail's header.


Diffs
-----

  resources/imap/additemtask.cpp af427ff 

Diff: https://git.reviewboard.kde.org/r/113254/diff/


Testing
-------

Copy an EMail to an IMAP server using KMail.

Without this patch, the current date/time gets set as internal date, \
causing my Webmail client to show this instead of the mail's actual \
date/time.

With this patch, this is fixed: the internal date is set to the mail's \
date/time as specified in its header, so Webmail shows it correctly as \
well.


Thanks,

Wolfgang Bauer

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic