ID: 23377 Comment by: ben at NOSPAM dot fcpl dot com Reported By: bill dot mccoy at pictureiq dot com Status: Open Bug Type: Feature/Change Request Operating System: Linux, WinXP PHP Version: 4.3.1 New Comment:
Couldn't agree more. This should be considered a bug rather than a feature request as the usefulness of the function is seriously impared without it. Previous Comments: ------------------------------------------------------------------------ [2003-04-28 09:59:39] bill dot mccoy at pictureiq dot com Well, yes, the code is operating as intended by the designer - but this begs the question of *why* it was intended to hide the search functionality defined in the IMAP specification behind a hard-coded "ALL". If there's no good reason for this unfriendly and counter-to-the-specification behavior then I think it's a defect, not a feature request. Put it another way: the raison d'etre of the PHP IMAP API is to expose the IMAP interface to PHP client programming and presently imap_thread does not do so properly, and for no apparent good reason. Therefore bug. If on the other hand there's a good reason for this behavior then by all means say so and mark it a feature request. ------------------------------------------------------------------------ [2003-04-28 01:19:46] [EMAIL PROTECTED] Marking this as a feature request, as it's not really a bug (it was intended this way). Derick ------------------------------------------------------------------------ [2003-04-28 00:01:08] bill dot mccoy at pictureiq dot com imap_thread's implementation in php_imap.c passes the hard-coded string "ALL" to the underlying IMAP C Runtime Library. This renders the function almost useless in situations where very large mailboxes require filtering (e.g. with a "SINCE DATE" search string). And even if imap_thread is only to support REFERENCE threading, the IMAP extension spec explicitly defines that the THREAD is performed on the results of the contained SEARCH. So in some sense failing to enable this means imap_thread can't be used in compliance with the spec. It is also inconsistent with imap_sort. Making 'criteria' a param for imap_thread as with imap_search & imap_sort would be (it seems to me) a tiny change with good bang for the buck. For compatibility I guess making it an optional 3rd param would be desirable. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=23377&edit=1
