Added paginator support for asearch
Added next/prev links in read_body to walk through search results
Added individual sort settings / mailbox
Added what I forgot to mention
exists with in_array (which returns just that boolean), and fix the
left over call to array_search to also check for NULL return.
This should fix the problems that Fredrik, me and other PHP 4.1 users
had with Marc's recent changes.
From PHP docs:
"Note: Prior to PHP 4.2.0, array_search() returns NULL on failure instead of FALSE."
1) do not do a search 1:* UID request in case we do not sort. That isn't
needed, just use a non uid fetch and request the UID together with the rest
of the headers.
2) Started on adding envelope parsing to sqimap_get_small_header list so we
can remove the imap related code from the messages class.
2) cleaned up the functions in mailbox_display, don't use trillions of
arguments when you can provide an array as well.
3) adapted right_main.php and search.php to fit the adapted functions in
mailbox_display
I hope the code makes more sense now an is easier to understand so that we
can start on separating the logic from the html output and prepare for
templates. Volunteers?
I hit this bug by coincidance and thnx to a server crash. Cyrus returned
information like UID, RFC822.SIZE, INTERNALDATE (that info is probably
cached by cyrus) correct, but an empty header. At the end of the response
there was an untagged OK response with as server message: Message <nnn> no
longer exists.
SEARCH responses. That is against RFC3501 !!!
example:
A005 UID SEARCH UID 1:*
* SEARCH 5949 8160 8401 8402 8413 8470 8471 8472 8497 8499
* SEARCH 8502 8504 8514
A005 OK SEARCH completed
We only use that call in case of non server side sorting.
Added basic @package/@subpackage tags
The decode/ files were documented slightly more thoroughly, although they're almost self-explanatory anyway.
The individual functions in the functions/ files need to be documented though, and now that this commit is in, anyone willing to write docblocks for functions should get cracking ;)
Now we can handle returned fetch responses with a different order of
returned arguments. UID i.e. can be properly detected if it is returned
after the HEADER data.
Another issue I fixed is adapting the msgs_id_str to 1:* in case we request all
headers for a mailbox.
If we don't do that the msgs_id_str can become pretty large and in my case
it was 12kb large and the imap server returned a BYE, querystring to long.
We should inspect the last issue better because there are situations where I
can still trigger the error. Setting the number of messages to display to a
large value would cause a BYE again.
