# [PATCH] Implementation of #35638 [include update timestamp in imap overview data] - Thread: https://externals.io/message/47097 - Original archive: https://news-web.php.net/php.internals/47097 - Started by Charles Duffy on 2010-03-10T02:52:20+00:00 - Messages: 7 - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #47097 — Charles Duffy — 2010-03-10T02:52:20+00:00 URL: https://externals.io/message/47097#47097 imap_fetch_overview() presently does not include any information on when the remote store has last been updated. Not having this information can force IMAP clients to make more requests than should strictly be necessary, and a ticket requesting it has been open since 2006. (As an aside -- my employer's predecessor-in-interest developed a patch for this issue internally in 2004, but failed to file a ticket or report the issue upstream; unfortunately, I've been unable to track down the (ir)responsible party to give them a good yelling-at). Unlike the version attached in the bugtracker (which is presently down), this one updates the test suite, and also reduces a bit of code duplication between the various imap_fetch_overview_* tests. If any further updates would be necessary to help this patch be promptly adopted, please let me know. Thank you kindly! --- ## #47922 — Charles Duffy — 2010-04-13T18:51:22+00:00 URL: https://externals.io/message/47097#47922 · In reply to: #47097 This patch has been awaiting review for over a month. Is there anything I can do to help hurry things along? --- ## #47937 — Pierre Joye — 2010-04-14T08:46:15+00:00 URL: https://externals.io/message/47097#47937 · In reply to: #47922 Hi, Which feature request is it? Please add this patch there as well if you did not do it already. Cheers, > This patch has been awaiting review for over a month. Is there anything I > can do to help hurry things along? > > -- > > > > -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org --- ## #47939 — Ferenc Kovacs — 2010-04-14T09:16:41+00:00 URL: https://externals.io/message/47097#47939 · In reply to: #47097 " Which feature request is it? Please add this patch there as well if you did not do it already." he did it. Tyrael ---------- Forwarded message ---------- From: Charles Duffy Date: Wed, Mar 10, 2010 at 4:52 AM Subject: [PHP-DEV] [PATCH] Implementation of #35638 [include update timestamp in imap overview data] To: internals@lists.php.net Cc: Charles_Duffy@dell.com imap_fetch_overview() presently does not include any information on when the remote store has last been updated. Not having this information can force IMAP clients to make more requests than should strictly be necessary, and a ticket requesting it has been open since 2006. (As an aside -- my employer's predecessor-in-interest developed a patch for this issue internally in 2004, but failed to file a ticket or report the issue upstream; unfortunately, I've been unable to track down the (ir)responsible party to give them a good yelling-at). Unlike the version attached in the bugtracker (which is presently down), this one updates the test suite, and also reduces a bit of code duplication between the various imap_fetch_overview_* tests. If any further updates would be necessary to help this patch be promptly adopted, please let me know. Thank you kindly! --- ## #47940 — Pierre Joye — 2010-04-14T09:24:33+00:00 URL: https://externals.io/message/47097#47940 · In reply to: #47939 > " Which feature request is it? Please add this patch there as well if > you did not do it already." > > he did it. The key part of this reply was the question :) # ? -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org --- ## #47942 — Pierre Joye — 2010-04-14T09:50:09+00:00 URL: https://externals.io/message/47097#47942 · In reply to: #47922 hi, Applied, thanks for your work (and patience). > This patch has been awaiting review for over a month. Is there anything I > can do to help hurry things along? > > -- > > > > -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org --- ## #47943 — Ferenc Kovacs — 2010-04-14T09:57:00+00:00 URL: https://externals.io/message/47097#47943 · In reply to: #47940 > > > […] > > The key part of this reply was the question :) # ? > > -- > Pierre > > @pierrejoye | http://blog.thepimp.net | http://www.libgd.org > I don't know what mail client are you using, but the subject containts the bugs.php.net ticket number #35638 http://bugs.php.net/bug.php?id=35638 The patch was added when Charles sent the original email a month ago, what are you missing? Tyrael