From: pdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Sun, 5 Apr 2009 04:13:39 +0000 (+0000)
Subject: FIXME
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=be51990845071cfa21f7001d1a36a2bb347e9e3e;p=squirrelmail.git

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)