php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Request #23377 imap_thread has hard-coded search param
Submitted: 2003-04-28 00:01 UTC Modified: 2017-10-24 01:57 UTC
Votes:2
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:0 (0.0%)
Same OS:1 (100.0%)
From: bill dot mccoy at pictureiq dot com Assigned: kalle (profile)
Status: Closed Package: IMAP related
PHP Version: * OS: *
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: bill dot mccoy at pictureiq dot com
New email:
PHP Version: OS:

 

 [2003-04-28 00:01 UTC] 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.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2003-04-28 01:19 UTC] derick@php.net 
Marking this as a feature request, as it's not really a bug (it was intended this way).

Derick
 [2003-04-28 09:59 UTC] 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-06-04 16:38 UTC] ben at NOSPAM dot fcpl dot com 
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.
 [2010-12-03 17:28 UTC] jani@php.net
-Package: Feature/Change Request +Package: IMAP related
 [2010-12-03 17:28 UTC] jani@php.net
-Operating System: Linux, WinXP +Operating System: * -PHP Version: 4.3.1 +PHP Version: *
 [2017-10-24 01:57 UTC] kalle@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: kalle
 [2017-10-24 01:57 UTC] kalle@php.net 
If this is still a request for PHP7 today, then please re-open a new report or possible PR on Github, thanks!
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Tue Apr 23 18:01:34 2024 UTC