fix orphaned subfolders on the folder list when they are subfolders of a special...
[squirrelmail.git] / functions / imap_mailbox.php
index f3082dbf0bb7ee3e90d3146b4d36b4016490d623..351993c91c5421c6068abb9a13a754147fb8b5e3 100755 (executable)
  * $Id$
  */
 
-GLOBAL $boxesnew;
+global $boxesnew;
 
 function isBoxBelow( $box2, $box1 ) {
-
     global $delimiter, $folder_prefix, $imap_server_type;
 
     if ( $imap_server_type == 'uw' ) {
         $boxs = $box2;
         $i = strpos( $box1, $delimiter, strlen( $folder_prefix ) );
-        if ( $i === FALSE ) {
+        if ( $i === false ) {
             $i = strlen( $box2 );
         }
     } else {
         $boxs = $box2 . $delimiter;
-        // Skip next second delimiter
+        /* Skip next second delimiter */
         $i = strpos( $box1, $delimiter );
         $i = strpos( $box1, $delimiter, $i + 1  );
-        if ( $i === FALSE ) {
+        if ( $i === false ) {
             $i = strlen( $box2 );
         } else {
             $i++;
         }
     }
 
-    return( substr( $box1, 0, $i ) == substr( $boxs, 0, $i ) );
-
+    return ( substr( $box1, 0, $i ) == substr( $boxs, 0, $i ) );
 }
 
-/*
-    Defines Special Mail Boxes
-*/
+/* Defines special mailboxes */
 function isSpecialMailbox( $box ) {
-
     global $trash_folder, $sent_folder, $draft_folder,
            $move_to_trash, $move_to_sent, $save_as_draft;
 
@@ -56,36 +51,28 @@ function isSpecialMailbox( $box ) {
         $ret = do_hook_function( 'special_mailbox', $box );
     }
 
-    return( $ret );
-
+    return $ret;
 }
 
-/*************************
- **  Expunges a mailbox **
- *************************/
-function sqimap_mailbox_expunge ($imap_stream, $mailbox,$handle_errors = TRUE) {
-    $read = sqimap_run_command($imap_stream, 'EXPUNGE',
-                               $handle_errors, $response, $message);
+/* Expunges a mailbox */
+function sqimap_mailbox_expunge ($imap_stream, $mailbox, $handle_errors = true) {
+    $read = sqimap_run_command($imap_stream, 'EXPUNGE', $handle_errors,
+                               $response, $message);
 }
 
-
-/******************************************************************************
- **  Checks whether or not the specified mailbox exists 
- ******************************************************************************/
+/* Checks whether or not the specified mailbox exists */
 function sqimap_mailbox_exists ($imap_stream, $mailbox) {
     if (! isset($mailbox)) {
         return false;
     }
     $mbx = sqimap_run_command($imap_stream, "LIST \"\" \"$mailbox\"",
-                              TRUE, $response, $message);
+                              true, $response, $message);
     return isset($mbx[0]);
 }
 
-/******************************************************************************
- **  Selects a mailbox
- ******************************************************************************/
+/* Selects a mailbox */
 function sqimap_mailbox_select ($imap_stream, $mailbox,
-                                $hide=TRUE, $recent=false, $extrainfo=false) {
+                                $hide = true, $recent = false, $extrainfo = false) {
     global $auto_expunge;
 
     if ( $mailbox == 'None' ) {
@@ -93,7 +80,7 @@ function sqimap_mailbox_select ($imap_stream, $mailbox,
     }
 
     $read = sqimap_run_command($imap_stream, "SELECT \"$mailbox\"",
-                               TRUE, $response, $message);
+                               true, $response, $message);
     if ($recent) {
         for ($i=0; $i<count($read); $i++) {
             if (strpos(strtolower($read[$i]), 'recent')) {
@@ -103,8 +90,7 @@ function sqimap_mailbox_select ($imap_stream, $mailbox,
         return $r[1];
     } else {
         if ($auto_expunge) {
-            $tmp = sqimap_run_command($imap_stream, 'EXPUNGE',
-                                  false, $a, $b);
+            $tmp = sqimap_run_command($imap_stream, 'EXPUNGE', false, $a, $b);
         }
         if (isset( $extrainfo ) && $extrainfo) {
             $result = array();
@@ -133,114 +119,52 @@ function sqimap_mailbox_select ($imap_stream, $mailbox,
     }
 }
 
-
-
-/******************************************************************************
- **  Creates a folder
- ******************************************************************************/
-function sqimap_mailbox_create ($imap_stream, $mailbox, $type)
-{
+/* Creates a folder */
+function sqimap_mailbox_create ($imap_stream, $mailbox, $type) {
     global $delimiter;
     if (strtolower($type) == 'noselect') {
-        $mailbox = $mailbox.$delimiter;
+        $mailbox .= $delimiter;
     }
     $read_ary = sqimap_run_command($imap_stream, "CREATE \"$mailbox\"",
-                                   TRUE, $response, $message);
-
+                                   true, $response, $message);
     sqimap_subscribe ($imap_stream, $mailbox);
 }
 
-
-
-/******************************************************************************
- **  Subscribes to an existing folder 
- ******************************************************************************/
-function sqimap_subscribe ($imap_stream, $mailbox)
-{
+/* Subscribes to an existing folder */
+function sqimap_subscribe ($imap_stream, $mailbox) {
     $read_ary = sqimap_run_command($imap_stream, "SUBSCRIBE \"$mailbox\"",
-                                 TRUE, $response, $message);
+                                   true, $response, $message);
 }
 
-
-
-/******************************************************************************
- **  Unsubscribes to an existing folder 
- ******************************************************************************/
-function sqimap_unsubscribe ($imap_stream, $mailbox)
-{
+/* Unsubscribes to an existing folder */
+function sqimap_unsubscribe ($imap_stream, $mailbox) {
     global $imap_server_type;
-    
     $read_ary = sqimap_run_command($imap_stream, "UNSUBSCRIBE \"$mailbox\"",
-                                 TRUE, $response, $message);
+                                   true, $response, $message);
 }
 
-
-
-/******************************************************************************
- **  This function simply deletes the given folder
- ******************************************************************************/
-function sqimap_mailbox_delete ($imap_stream, $mailbox)
-{
+/* Deletes the given folder */
+function sqimap_mailbox_delete ($imap_stream, $mailbox) {
     $read_ary = sqimap_run_command($imap_stream, "DELETE \"$mailbox\"",
-                                 TRUE, $response, $message);
+                                   true, $response, $message);
     sqimap_unsubscribe ($imap_stream, $mailbox);
-       filter_check($mailbox, 'delete');
+    do_hook_function("rename_or_delete_folder", $args = array($mailbox, 'delete', ''));
 }
 
-/***********************************************************************
- ** Determines if the user is subscribed to the folder or not
- **********************************************************************/
+/* Determines if the user is subscribed to the folder or not */
 function sqimap_mailbox_is_subscribed($imap_stream, $folder) {
-
     $boxesall = sqimap_mailbox_list ($imap_stream);
     foreach ($boxesall as $ref) {
         if ($ref['unformatted'] == $folder) {
-            return TRUE;
+            return true;
         }
     }
     return false;
 }
 
-/* fix filters when renaming/deleting folders */
-
-function filter_check ($old_folder, $action, $new_folder = 'INBOX') {
-    global $plugins, $data_dir, $username;
-    $filters = array();
-    $count = count($plugins);
-    for ($i=0;$i<$count;$i++) {
-        if ($plugins[$i] == 'filters') {
-            require_once("../plugins/filters/filters.php");
-            $filters = load_filters();
-            break;
-        }
-    }
-    if (empty($filters)) {
-        return;
-    }
-    $filter_count = count($filters);
-    $p=0;
-    for ($i=0;$i<$filter_count;$i++) {
-        if ($old_folder == $filters[$i]['folder']) {
-            if ($action == 'rename') {
-                $filters[$i]['folder'] = $new_folder;
-                setPref($data_dir, $username, "filter".$i, $filters[$i]['where'].",".$filters[$i]['what'].",".$new_folder);
-            }
-            elseif ($action == 'delete') {
-                remove_filter($p);
-                $p = $p-1;
-            }
-        }
-    $p++;
-    }
-}
-
-
-/*
-    Renames a mailbox
-*/
+/* Renames a mailbox */
 function sqimap_mailbox_rename( $imap_stream, $old_name, $new_name ) {
-
-    if ( $old_name <> $new_name ) {
+    if ( $old_name != $new_name ) {
         global $delimiter, $imap_server_type;
         if ( substr( $old_name, -1 ) == $delimiter  ) {
             $old_name = substr( $old_name, 0, strlen( $old_name ) - 1 );
@@ -251,11 +175,10 @@ function sqimap_mailbox_rename( $imap_stream, $old_name, $new_name ) {
         }
         $boxesall = sqimap_mailbox_list($imap_stream);
         $cmd = 'RENAME "' . quoteIMAP($old_name) . '" "' .  quoteIMAP($new_name) . '"';
-        $data = sqimap_run_command($imap_stream, $cmd,
-                                 TRUE, $response, $message);
+        $data = sqimap_run_command($imap_stream, $cmd, true, $response, $message);
         sqimap_unsubscribe($imap_stream, $old_name.$postfix);
         sqimap_subscribe($imap_stream, $new_name.$postfix);
-        filter_check($old_name, 'rename', $new_name);
+        do_hook_function("rename_or_delete_folder",$args = array($old_name, 'rename', $new_name));
         $l = strlen( $old_name ) + 1;
         $p = 'unformatted';
         foreach ( $boxesall as $box ) {
@@ -263,56 +186,53 @@ function sqimap_mailbox_rename( $imap_stream, $old_name, $new_name ) {
                 $new_sub = $new_name . $delimiter . substr($box[$p], $l);
                 if ($imap_server_type == 'cyrus') {
                     $cmd = 'RENAME "' . quoteIMAP($box[$p]) . '" "' .  quoteIMAP($new_sub) . '"';
-                    $data = sqimap_run_command($imap_stream, $cmd, TRUE, 
+                    $data = sqimap_run_command($imap_stream, $cmd, true,
                                                $response, $message);
                 }
                 sqimap_unsubscribe($imap_stream, $box[$p]);
                 sqimap_subscribe($imap_stream, $new_sub);
-                filter_check($box[$p], 'rename', $new_sub);
+                do_hook_function("rename_or_delete_folder",
+                                 $args = array($box[$p], 'rename', $new_sub));
             }
         }
     }
-
 }
 
-/******************************************************************************
- **  Formats a mailbox into 4 parts for the $boxesall array
- **
- **  The four parts are:
- **
- **    raw            - Raw LIST/LSUB response from the IMAP server
- **    formatted      - nicely formatted folder name
- **    unformatted    - unformatted, but with delimiter at end removed
- **    unformatted-dm - folder name as it appears in raw response
- **    unformatted-disp - unformatted without $folder_prefix
- **
- ******************************************************************************/
-function sqimap_mailbox_parse ($line, $line_lsub)
-{
+/*
+ * Formats a mailbox into 4 parts for the $boxesall array
+ *
+ * The four parts are:
+ *
+ *     raw            - Raw LIST/LSUB response from the IMAP server
+ *     formatted      - nicely formatted folder name
+ *     unformatted    - unformatted, but with delimiter at end removed
+ *     unformatted-dm - folder name as it appears in raw response
+ *     unformatted-disp - unformatted without $folder_prefix
+ */
+function sqimap_mailbox_parse ($line, $line_lsub) {
     global $folder_prefix, $delimiter;
-    
+
     /* Process each folder line */
     for ($g=0; $g < count($line); $g++) {
-        
+
         /* Store the raw IMAP reply */
         if (isset($line[$g])) {
             $boxesall[$g]["raw"] = $line[$g];
         }
         else {
-            $boxesall[$g]["raw"] = "";
+            $boxesall[$g]["raw"] = '';
         }
-        
-        
+
+
         /* Count number of delimiters ($delimiter) in folder name */
         $mailbox = trim($line_lsub[$g]);
         $dm_count =  substr_count($mailbox, $delimiter);
         if (substr($mailbox, -1) == $delimiter) {
-            /* If name ends in delimiter - decrement count by one */
-            $dm_count--;  
+            /* If name ends in delimiter, decrement count by one */
+            $dm_count--;
         }
-        
-        /* Format folder name, but only if it's a INBOX.* or have */
-        /* a parent. */
+
+        /* Format folder name, but only if it's a INBOX.* or has a parent. */
         $boxesallbyname[$mailbox] = $g;
         $parentfolder = readMailboxParent($mailbox, $delimiter);
         if ( (strtolower(substr($mailbox, 0, 5)) == "inbox") ||
@@ -321,7 +241,7 @@ function sqimap_mailbox_parse ($line, $line_lsub)
                (strlen($parentfolder) > 0) ) ) {
             $indent = $dm_count - ( substr_count($folder_prefix, $delimiter));
             if ($indent > 0) {
-                $boxesall[$g]['formatted']  = str_repeat("&nbsp;&nbsp;", $indent);
+                $boxesall[$g]['formatted']  = str_repeat('&nbsp;&nbsp;', $indent);
             }
             else {
                 $boxesall[$g]['formatted'] = '';
@@ -356,16 +276,17 @@ function sqimap_mailbox_parse ($line, $line_lsub)
     return $boxesall;
 }
 
-/**
+/*
  * Sorting function used to sort mailbox names.
- *   + Original patch from dave_michmerhuizen@yahoo.com
- *   + Allows case insensitivity when sorting folders
- *   + Takes care of the delimiter being sorted to the end, causing
- *     subfolders to be listed in below folders that are prefixed
- *     with their parent folders name.
- *        For example: INBOX.foo, INBOX.foobar, and INBOX.foo.bar
- *        Without special sort function: foobar between foo and foo.bar
- *        With special sort function: foobar AFTER foo and foo.bar :)
+ *     + Original patch from dave_michmerhuizen@yahoo.com
+ *     + Allows case insensitivity when sorting folders
+ *     + Takes care of the delimiter being sorted to the end, causing
+ *       subfolders to be listed in below folders that are prefixed
+ *       with their parent folders name.
+ *
+ *       For example: INBOX.foo, INBOX.foobar, and INBOX.foo.bar
+ *       Without special sort function: foobar between foo and foo.bar
+ *       With special sort function: foobar AFTER foo and foo.bar :)
  */
 function user_strcasecmp($a, $b) {
     global $delimiter;
@@ -408,176 +329,174 @@ function user_strcasecmp($a, $b) {
         }
     }
 
-    return ($result);
+    return $result;
 }
 
 
-/******************************************************************************
- **  Returns sorted mailbox lists in several different ways.
- * See comment on sqimap_mailbox_parse() for info about the returned array.
- ******************************************************************************/
+/*
+ * Returns sorted mailbox lists in several different ways. 
+ * See comment on sqimap_mailbox_parse() for info about the returned array.
+ */
 function sqimap_mailbox_list($imap_stream) {
+    global $boxesnew, $default_folder_prefix;
 
-       GLOBAL $boxesnew;
-
-       if ( !isset( $boxesnew ) ) {
-
-           GLOBAL $data_dir, $username, $list_special_folders_first,
-                  $folder_prefix, $trash_folder, $sent_folder, $draft_folder,
-                  $move_to_trash, $move_to_sent, $save_as_draft,
-                  $delimiter;
-       
-           $inbox_in_list = $inbox_subscribed = FALSE;
-       
-           require_once('../src/load_prefs.php');
-           require_once('../functions/array.php');
-       
-           /** LSUB array **/
-           $lsub_ary = sqimap_run_command ($imap_stream, "LSUB \"$folder_prefix\" \"*\"",
-                                         TRUE, $response, $message);
-       
-           /* Section about removing the last element was removed */
-           /* We don't return "* OK" anymore from sqimap_read_data */
-       
-           $sorted_lsub_ary = array();
-           for ($i=0;$i < count($lsub_ary); $i++) {
-               /* Workaround for EIMS */
-               /* Doesn't work if the mailbox name is multiple lines */
-               if (isset($lsub_ary[$i + 1]) &&
-                   ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
-                        $lsub_ary[$i], $regs)) {
-                   $i ++;
-                   $lsub_ary[$i] = $regs[1] . '"' . addslashes(trim($lsub_ary[$i])) .
-                       '"' . $regs[2];
-               }
-               $temp_mailbox_name = find_mailbox_name($lsub_ary[$i]);
-               $sorted_lsub_ary[] = $temp_mailbox_name;
-               if (strtoupper($temp_mailbox_name) == 'INBOX') {
-                   $inbox_subscribed = TRUE;
-               }
-           }
-           $new_ary = array();
-           for ($i=0; $i < count($sorted_lsub_ary); $i++) {
-               if (!in_array($sorted_lsub_ary[$i], $new_ary)) {
-                   $new_ary[] = $sorted_lsub_ary[$i];
-               }
-           }
-           $sorted_lsub_ary = $new_ary;
-           if (isset($sorted_lsub_ary)) {
-               usort($sorted_lsub_ary, 'user_strcasecmp');
-           }
-       
-           /** LIST array **/
-           $sorted_list_ary = array();
-           for ($i=0; $i < count($sorted_lsub_ary); $i++) {
-               if (substr($sorted_lsub_ary[$i], -1) == $delimiter) {
-                   $mbx = substr($sorted_lsub_ary[$i], 0, strlen($sorted_lsub_ary[$i])-1);
-               }
-               else {
-                   $mbx = $sorted_lsub_ary[$i];
-               }
-       
-               $read = sqimap_run_command ($imap_stream, "LIST \"\" \"$mbx\"",
-                                         TRUE, $response, $message);
-               /* Another workaround for EIMS */
-               if (isset($read[1]) &&
-                   ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
-                        $read[0], $regs)) {
-                   $read[0] = $regs[1] . '"' . addslashes(trim($read[1])) .
-                              '"' . $regs[2];
-               }
-       
-               if (isset($sorted_list_ary[$i])) {
-                   $sorted_list_ary[$i] = '';
-               }
-       
-               if (isset($read[0])) {
-                   $sorted_list_ary[$i] = $read[0];
-               }
-               else {
-                   $sorted_list_ary[$i] = '';
-               }
-       
-               if (isset($sorted_list_ary[$i]) &&
-                   strtoupper(find_mailbox_name($sorted_list_ary[$i])) == 'INBOX') {
-                   $inbox_in_list = TRUE;
-               }
-           }
-       
-           /**
-            * Just in case they're not subscribed to their inbox,
-            * we'll get it for them anyway
-            */
-           if ($inbox_subscribed == false || $inbox_in_list == false) {
-               $inbox_ary = sqimap_run_command ($imap_stream, "LIST \"\" \"INBOX\"",
-                                              TRUE, $response, $message);
-               /* Another workaround for EIMS */
-               if (isset($inbox_ary[1]) &&
-                   ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
-                        $inbox_ary[0], $regs)) {
-                   $inbox_ary[0] = $regs[1] . '"' . addslashes(trim($inbox_ary[1])) .
-                       '"' . $regs[2];
-               }
-       
-               $sorted_list_ary[] = $inbox_ary[0];
-               $sorted_lsub_ary[] = find_mailbox_name($inbox_ary[0]);
-           }
-       
-           $boxesall = sqimap_mailbox_parse ($sorted_list_ary, $sorted_lsub_ary);
-       
-           /** Now, lets sort for special folders **/
-           $boxesnew = $used = array();
-       
-           /* Find INBOX */
-           foreach ( $boxesall as $k => $box ) {
-               if ( strtolower($box['unformatted']) == 'inbox') {
-                   $boxesnew[] = $box;
-                   $used[$k] = TRUE;
-               } else {
-                   $used[$k] = FALSE;
-               }
-           }
-       
-           /* List special folders and their subfolders, if requested. */
-           if ($list_special_folders_first == TRUE) {
-       
-               foreach ( $boxesall as $k => $box ) {
-                   if ( !$used[$k] &&
-                        isSpecialMailbox( $box['unformatted'] ) ) {
-                       $boxesnew[] = $box;
-                       $used[$k] = TRUE;
-                   }
-               }
-       
-           }
-       
-           /* Rest of the folders */
-           foreach ( $boxesall as $k => $box ) {
-               if ( !$used[$k] ) {
-                   $boxesnew[] = $box;
-               }
-           }
-    }
+    if ( !isset( $boxesnew ) ) {
+
+        global $data_dir, $username, $list_special_folders_first,
+               $folder_prefix, $trash_folder, $sent_folder, $draft_folder,
+               $move_to_trash, $move_to_sent, $save_as_draft,
+               $delimiter;
+
+        $inbox_in_list = false;
+        $inbox_subscribed = false;
+
+        require_once('../src/load_prefs.php');
+        require_once('../functions/array.php');
 
-    return( $boxesnew );
+        /* LSUB array */
+        $lsub_ary = sqimap_run_command ($imap_stream, "LSUB \"$folder_prefix\" \"*%\"",
+                                        true, $response, $message);
 
+        /*
+         * Section about removing the last element was removed 
+         * We don't return "* OK" anymore from sqimap_read_data
+         */
+
+        $sorted_lsub_ary = array();
+        for ($i=0;$i < count($lsub_ary); $i++) {
+            /*
+             * Workaround for EIMS
+             * Doesn't work if the mailbox name is multiple lines
+             */
+            if (isset($lsub_ary[$i + 1]) &&
+                ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
+                     $lsub_ary[$i], $regs)) {
+                $i ++;
+                $lsub_ary[$i] = $regs[1] . '"' . addslashes(trim($lsub_ary[$i])) . '"' . $regs[2];
+            }
+            $temp_mailbox_name = find_mailbox_name($lsub_ary[$i]);
+            $sorted_lsub_ary[] = $temp_mailbox_name;
+            if (strtoupper($temp_mailbox_name) == 'INBOX') {
+                $inbox_subscribed = true;
+            }
+        }
+        $new_ary = array();
+        for ($i=0; $i < count($sorted_lsub_ary); $i++) {
+            if (!in_array($sorted_lsub_ary[$i], $new_ary)) {
+                $new_ary[] = $sorted_lsub_ary[$i];
+            }
+        }
+        $sorted_lsub_ary = $new_ary;
+        if (isset($sorted_lsub_ary)) {
+            usort($sorted_lsub_ary, 'user_strcasecmp');
+        }
+
+        /* LIST array */
+        $sorted_list_ary = array();
+        for ($i=0; $i < count($sorted_lsub_ary); $i++) {
+            if (substr($sorted_lsub_ary[$i], -1) == $delimiter) {
+                $mbx = substr($sorted_lsub_ary[$i], 0, strlen($sorted_lsub_ary[$i])-1);
+            }
+            else {
+                $mbx = $sorted_lsub_ary[$i];
+            }
+
+            $read = sqimap_run_command ($imap_stream, "LIST \"\" \"$mbx\"",
+                                        true, $response, $message);
+            /* Another workaround for EIMS */
+            if (isset($read[1]) &&
+                ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
+                     $read[0], $regs)) {
+                $read[0] = $regs[1] . '"' . addslashes(trim($read[1])) . '"' . $regs[2];
+            }
+
+            if (isset($sorted_list_ary[$i])) {
+                $sorted_list_ary[$i] = '';
+            }
+
+            if (isset($read[0])) {
+                $sorted_list_ary[$i] = $read[0];
+            }
+            else {
+                $sorted_list_ary[$i] = '';
+            }
+
+            if (isset($sorted_list_ary[$i]) &&
+                strtoupper(find_mailbox_name($sorted_list_ary[$i])) == 'INBOX') {
+                $inbox_in_list = true;
+            }
+        }
+
+        /*
+         * Just in case they're not subscribed to their inbox,
+         * we'll get it for them anyway
+         */
+        if (!$inbox_subscribed || !$inbox_in_list) {
+            $inbox_ary = sqimap_run_command ($imap_stream, "LIST \"\" \"INBOX\"",
+                                             true, $response, $message);
+            /* Another workaround for EIMS */
+            if (isset($inbox_ary[1]) &&
+                ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
+                     $inbox_ary[0], $regs)) {
+                $inbox_ary[0] = $regs[1] . '"' . addslashes(trim($inbox_ary[1])) .
+                    '"' . $regs[2];
+            }
+
+            $sorted_list_ary[] = $inbox_ary[0];
+            $sorted_lsub_ary[] = find_mailbox_name($inbox_ary[0]);
+        }
+
+        $boxesall = sqimap_mailbox_parse ($sorted_list_ary, $sorted_lsub_ary);
+
+        /* Now, lets sort for special folders */
+        $boxesnew = $used = array();
+
+        /* Find INBOX */
+        foreach ( $boxesall as $k => $box ) {
+            if ( strtolower($box['unformatted']) == 'inbox') {
+                $boxesnew[] = $box;
+                $used[$k] = true;
+            } else {
+                $used[$k] = false;
+            }
+        }
+        /* List special folders and their subfolders, if requested. */
+        if ($list_special_folders_first) {
+            foreach ( $boxesall as $k => $box ) {
+                if ( !$used[$k] && isSpecialMailbox( $box['unformatted'] ) ) {
+                    $boxesnew[] = $box;
+                    $used[$k] = true;
+                }
+                if (!$used[$k] && preg_match("/$default_folder_prefix(Sent|Drafts|Trash)/", $box['unformatted']) ) {
+                    $boxesnew[] = $box;
+                    $used[$k] = true;
+                }
+            }
+
+        }
+
+        /* Rest of the folders */
+        foreach ( $boxesall as $k => $box ) {
+            if ( !$used[$k] ) {
+                $boxesnew[] = $box;
+            }
+        }
+    }
+    return $boxesnew;
 }
 
 /*
  *  Returns a list of all folders, subscribed or not
  */
 function sqimap_mailbox_list_all($imap_stream) {
-
-
-    GLOBAL $list_special_folders_first, $folder_prefix,
-           $delimiter;
+    global $list_special_folders_first, $folder_prefix, $delimiter;
 
     require_once('../functions/array.php');
 
     $ssid = sqimap_session_id();
     $lsid = strlen( $ssid );
     fputs ($imap_stream, $ssid . " LIST \"$folder_prefix\" *\r\n");
-    $read_ary = sqimap_read_data ($imap_stream, $ssid, TRUE, $response, $message);
+    $read_ary = sqimap_read_data ($imap_stream, $ssid, true, $response, $message);
     $g = 0;
     $phase = 'inbox';
 
@@ -587,9 +506,7 @@ function sqimap_mailbox_list_all($imap_stream) {
             ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
                  $read_ary[$i], $regs)) {
             $i ++;
-            $read_ary[$i] = $regs[1] . '"' .
-                            addslashes(trim($read_ary[$i])) .
-                            '"' . $regs[2];
+            $read_ary[$i] = $regs[1] . '"' . addslashes(trim($read_ary[$i])) . '"' . $regs[2];
         }
         if (substr($read_ary[$i], 0, $lsid) != $ssid ) {
 
@@ -604,8 +521,7 @@ function sqimap_mailbox_list_all($imap_stream) {
                 $dm_count--;
             }
 
-            /* Format folder name, but only if it's a INBOX.* or have */
-            /* a parent. */
+            /* Format folder name, but only if it's a INBOX.* or has a parent. */
             $boxesallbyname[$mailbox] = $g;
             $parentfolder = readMailboxParent($mailbox, $delimiter);
             if((eregi('^inbox'.quotemeta($delimiter), $mailbox)) ||
@@ -628,21 +544,17 @@ function sqimap_mailbox_list_all($imap_stream) {
                 $mailbox = substr($mailbox, 0, strlen($mailbox) - 1);
             }
             $boxes[$g]['unformatted'] = $mailbox;
-            $boxes[$g]['unformatted-disp'] =
-                ereg_replace('^' . $folder_prefix, '', $mailbox);
+            $boxes[$g]['unformatted-disp'] = ereg_replace('^' . $folder_prefix, '', $mailbox);
             $boxes[$g]['id'] = $g;
 
-            /** Now lets get the flags for this mailbox **/
+            /* Now lets get the flags for this mailbox */
             $read_mlbx = sqimap_run_command ($imap_stream, "LIST \"\" \"$mailbox\"",
-                                           TRUE, $response, $message);
+                                             true, $response, $message);
 
             /* Another workaround for EIMS */
             if (isset($read_mlbx[1]) &&
-                ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$",
-                     $read_mlbx[0], $regs)) {
-                $read_mlbx[0] = $regs[1] . '"' .
-                                addslashes(trim($read_mlbx[1])) .
-                                '"' . $regs[2];
+                ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$", $read_mlbx[0], $regs)) {
+                $read_mlbx[0] = $regs[1] . '"' . addslashes(trim($read_mlbx[1])) . '"' . $regs[2];
             }
 
             $flags = substr($read_mlbx[0], strpos($read_mlbx[0], '(')+1);