From: pdontthink Date: Sun, 5 Apr 2009 04:13:39 +0000 (+0000) Subject: FIXME X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=commitdiff_plain;h=be51990845071cfa21f7001d1a36a2bb347e9e3e FIXME git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@13516 7612ce4b-ef26-0410-bec9-ea0150e637f0 --- diff --git a/functions/imap_asearch.php b/functions/imap_asearch.php index 8c204ebb..6443d5dd 100644 --- a/functions/imap_asearch.php +++ b/functions/imap_asearch.php @@ -368,6 +368,7 @@ function sqimap_asearch_get_sort_criteria($mailbox, $sort_by) $sort_opcodes = array ('DATE', 'FROM', 'SUBJECT', 'SIZE'); if ($internal_date_sort == true) $sort_opcodes[0] = 'ARRIVAL'; +// FIXME: Why are these commented out? I have no idea what this code does, but both of these functions sound more robust than the simple string check that's being used now. Someone who understands this code should either fix this or remove these lines completely or document why they are here commented out // if (handleAsSent($mailbox)) // if (isSentFolder($mailbox)) if ($mailbox == $sent_folder)