[Tickets #6859] Re: pear IMAP implementation of ::getUids() is broken

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/6859
-----------------------------------------------------------------------
 Ticket             | 6859
 Updated By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
 Summary            | pear IMAP implementation of ::getUids() is broken
 Queue              | Kolab
 Type               | Bug
 State              | Assigned
 Priority           | 1. Low
 Milestone          |
 Patch              |
 Owners             | Thomas Jarosch, Gunnar Wrobel
-----------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2008-06-12 04:20) wrote:

Hello Mathieu,

> Proposed patch:
https://www.intevation.de/roundup/kolab/file804/pear.php.diff

I took a look at your patch and I'm quite surprised that an IMAP UID
search command stopped working in Net_IMAP. According to the documentation
it's defined as "mixed search( string $search_list, [boolean $uidSearch =
false])"
and the code in Net_IMAP seems to behave like this.

Have you checked with a packet sniffer like tcpdump what goes over the
wire?
My feeling tells me this is an upstream bug in Net_IMAP.

There is also another issue with the patch:
You will sync in "temporary" objects as deleted messages are not
excluded.

In my version of 1.1.0beta2, $imap->getMessagesList() only takes one
parameter. What's the second "false" argument supposed to do?

Cheers,
Thomas