[Tickets #9626] Imap_Client fetch() fails with empty UID list

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/9626
------------------------------------------------------------------------------
  Ticket             | 9626
  Created By         | Gunnar Wrobel <wrobel at pardus.de>
  Summary            | Imap_Client fetch() fails with empty UID list
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Enhancement
  State              | New
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Gunnar Wrobel <wrobel at pardus.de> (2011-03-06 14:31) wrote:

The following fails with an exception ...

         $query = new Horde_Imap_Client_Fetch_Query();
         $query->structure();

         $ret = $this->getBackend()->fetch(
             $folder,
             $query,
             array('ids' => new Horde_Imap_Client_Ids($uids))
         );

.. if $uids is array().

(1299420790.8779) C: 5 UID FETCH  (BODYSTRUCTURE)
(1299420790.8787) S: 5 BAD Invalid sequence in Uid

12. Horde_Imap_Client_Socket->_fetch()  
/home/wrobel/environment/horde-glue/horde/framework/Imap_Client/lib/Horde/Imap/Client/Base.php:1954
13. Horde_Imap_Client_Socket->_sendLine()  
/home/wrobel/environment/horde-glue/horde/framework/Imap_Client/lib/Horde/Imap/Client/Socket.php:2430
14. Horde_Imap_Client_Socket->_parseResponse()  
/home/wrobel/environment/horde-glue/horde/framework/Imap_Client/lib/Horde/Imap/Client/Socket.php:3493
15. Horde_Imap_Client_Base->_exception()  
/home/wrobel/environment/horde-glue/horde/framework/Imap_Client/lib/Horde/Imap/Client/Socket.php:3860

I can obviously catch the problem before calling fetch() but I  
wondered if it wouldn't make sense to change the API to return an  
empty array in that case.