X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=blobdiff_plain;f=functions%2Fimap_mailbox.php;h=6c4d916e5cc087d5f6af4cd510feaf7c6480a1db;hp=9ce0ff410c58a4f72b8b23580c11666acbc17fd2;hb=3411d4ec9a699200e2a77256df2bd8da1f346a82;hpb=a3439b271e1b5fa7e61362970dbbcc5bfc75e5b1 diff --git a/functions/imap_mailbox.php b/functions/imap_mailbox.php index 9ce0ff41..6c4d916e 100755 --- a/functions/imap_mailbox.php +++ b/functions/imap_mailbox.php @@ -3,7 +3,7 @@ /** * imap_mailbox.php * - * Copyright (c) 1999-2001 The SquirrelMail Development Team + * Copyright (c) 1999-2002 The SquirrelMail Project Team * Licensed under the GNU GPL. For full terms see the file COPYING. * * This impliments all functions that manipulate mailboxes @@ -11,46 +11,76 @@ * $Id$ */ -/************************* - ** Expunges a mailbox ** - *************************/ -function sqimap_mailbox_expunge ($imap_stream, $mailbox,$handle_errors = true) -{ - fputs ($imap_stream, sqimap_session_id() . " EXPUNGE\r\n"); - $read = sqimap_read_data($imap_stream, sqimap_session_id(), - $handle_errors, $response, $message); +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 ) { + $i = strlen( $box2 ); + } + } else { + $boxs = $box2 . $delimiter; + /* Skip next second delimiter */ + $i = strpos( $box1, $delimiter ); + $i = strpos( $box1, $delimiter, $i + 1 ); + if ( $i === false ) { + $i = strlen( $box2 ); + } else { + $i++; + } + } + + return ( substr( $box1, 0, $i ) == substr( $boxs, 0, $i ) ); +} + +/* Defines special mailboxes */ +function isSpecialMailbox( $box ) { + global $trash_folder, $sent_folder, $draft_folder, + $move_to_trash, $move_to_sent, $save_as_draft; + + $ret = ( (strtolower($box) == 'inbox') || + ( $move_to_trash && isBoxBelow( $box, $trash_folder ) ) || + ( $move_to_sent && isBoxBelow( $box, $sent_folder )) || + ($save_as_draft && $box == $draft_folder ) ); + + if ( !$ret ) { + $ret = do_hook_function( 'special_mailbox', $box ); + } + + 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); +} -/****************************************************************************** - ** Checks whether or not the specified mailbox exists - ******************************************************************************/ -function sqimap_mailbox_exists ($imap_stream, $mailbox) -{ +/* Checks whether or not the specified mailbox exists */ +function sqimap_mailbox_exists ($imap_stream, $mailbox) { if (! isset($mailbox)) { return false; } - fputs ($imap_stream, sqimap_session_id() . " LIST \"\" \"$mailbox\"\r\n"); - $mbx = sqimap_read_data($imap_stream, sqimap_session_id(), - true, $response, $message); + $mbx = sqimap_run_command($imap_stream, "LIST \"\" \"$mailbox\"", + true, $response, $message); return isset($mbx[0]); } -/****************************************************************************** - ** Selects a mailbox - ******************************************************************************/ +/* Selects a mailbox */ function sqimap_mailbox_select ($imap_stream, $mailbox, - $hide=true, $recent=false) -{ + $hide = true, $recent = false, $extrainfo = false) { global $auto_expunge; - + if ( $mailbox == 'None' ) { return; } - - fputs ($imap_stream, sqimap_session_id() . " SELECT \"$mailbox\"\r\n"); - $read = sqimap_read_data($imap_stream, sqimap_session_id(), - true, $response, $message); + + $read = sqimap_run_command($imap_stream, "SELECT \"$mailbox\"", + true, $response, $message); if ($recent) { for ($i=0; $i 0) ) ) { - $indent = $dm_count - (countCharInString($folder_prefix, $delimiter)); + $indent = $dm_count - ( substr_count($folder_prefix, $delimiter)); if ($indent > 0) { - $boxes[$g]["formatted"] = str_repeat("  ", $indent); + $boxesall[$g]['formatted'] = str_repeat('  ', $indent); } else { - $boxes[$g]["formatted"] = ''; + $boxesall[$g]['formatted'] = ''; } - $boxes[$g]["formatted"] .= readShortMailboxName($mailbox, $delimiter); + $boxesall[$g]['formatted'] .= readShortMailboxName($mailbox, $delimiter); } else { - $boxes[$g]["formatted"] = $mailbox; + $boxesall[$g]['formatted'] = $mailbox; } - - $boxes[$g]['unformatted-dm'] = $mailbox; + + $boxesall[$g]['unformatted-dm'] = $mailbox; if (substr($mailbox, -1) == $delimiter) { $mailbox = substr($mailbox, 0, strlen($mailbox) - 1); } - $boxes[$g]['unformatted'] = $mailbox; + $boxesall[$g]['unformatted'] = $mailbox; if (substr($mailbox,0,strlen($folder_prefix))==$folder_prefix) { $mailbox = substr($mailbox, strlen($folder_prefix)); } - $boxes[$g]['unformatted-disp'] = $mailbox; - $boxes[$g]['id'] = $g; - - $boxes[$g]['flags'] = array(); + $boxesall[$g]['unformatted-disp'] = $mailbox; + $boxesall[$g]['id'] = $g; + + $boxesall[$g]['flags'] = array(); if (isset($line[$g])) { ereg("\(([^)]*)\)",$line[$g],$regs); $flags = trim(strtolower(str_replace('\\', '',$regs[1]))); if ($flags) { - $boxes[$g]['flags'] = explode(' ', $flags); + $boxesall[$g]['flags'] = explode(' ', $flags); } } } - - return $boxes; + + 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; @@ -251,9 +308,9 @@ function user_strcasecmp($a, $b) { if (($a_del == $delimiter) && ($b_del == $delimiter)) { $result = 0; } else if (($a_del == $delimiter) && ($b_del != $delimiter)) { - $result = 1; - } else if (($a_del != $delimiter) && ($b_del != $delimiter)) { $result = -1; + } else if (($a_del != $delimiter) && ($b_del == $delimiter)) { + $result = 1; } else { $result = strcasecmp($a{$c}, $b{$c}); } @@ -262,7 +319,7 @@ function user_strcasecmp($a, $b) { break; } } - + /* If one string is a prefix of the other... */ if ($result == 0) { if ($a_length < $b_length) { @@ -272,290 +329,248 @@ 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. - ******************************************************************************/ -function sqimap_mailbox_list ($imap_stream) { - global $data_dir, $username, $list_special_folders_first; - global $folder_prefix, $trash_folder, $sent_folder, $draft_folder; - global $move_to_trash, $move_to_sent, $save_as_draft; - global $delimiter; - - $inbox_in_list = false; - $inbox_subscribed = false; - - require_once('../src/load_prefs.php'); - require_once('../functions/array.php'); - - /** LSUB array **/ - fputs ($imap_stream, sqimap_session_id() . - " LSUB \"$folder_prefix\" \"*\"\r\n"); - $lsub_ary = sqimap_read_data ($imap_stream, sqimap_session_id(), - 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]; - } - - fputs ($imap_stream, sqimap_session_id() . " LIST \"\" \"$mbx\"\r\n"); - $read = sqimap_read_data ($imap_stream, sqimap_session_id(), - 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; +/* + * 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; + + 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'); + + /* 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; + } } - } - - /** - * 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) { - fputs ($imap_stream, sqimap_session_id() . " LIST \"\" \"INBOX\"\r\n"); - $inbox_ary = sqimap_read_data ($imap_stream, sqimap_session_id(), - 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]; + $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_list_ary[] = $inbox_ary[0]; - $sorted_lsub_ary[] = find_mailbox_name($inbox_ary[0]); - } - - $boxes = sqimap_mailbox_parse ($sorted_list_ary, $sorted_lsub_ary); - - /** Now, lets sort for special folders **/ - $boxesnew = Array(); - - /* Find INBOX */ - for ($i = 0; $i < count($boxes); $i++) { - if (strtolower($boxes[$i]["unformatted"]) == "inbox") { - $boxesnew[] = $boxes[$i]; - $used[$i] = true; - $i = count($boxes); + $sorted_lsub_ary = $new_ary; + if (isset($sorted_lsub_ary)) { + usort($sorted_lsub_ary, 'user_strcasecmp'); } - } - - /* List special folders and their subfolders, if requested. */ - if ($list_special_folders_first == true) { - /* First list the trash folder. */ - for ($i = 0 ; $i < count($boxes) ; $i++) { - if ($move_to_trash && - eregi('^' . quotemeta($trash_folder) . '(' . - quotemeta($delimiter) . '.*)?$', $boxes[$i]['unformatted'])) { - $boxesnew[] = $boxes[$i]; - $used[$i] = true; + + /* 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; } } - - /* Then list the sent folder. */ - for ($i = 0 ; $i < count($boxes) ; $i++) { - if ($move_to_sent && - eregi('^' . quotemeta($sent_folder) . '(' . - quotemeta($delimiter) . '.*)?$', $boxes[$i]['unformatted'])) { - $boxesnew[] = $boxes[$i]; - $used[$i] = 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]); } - - /* Lastly, list the list the draft folder. */ - for ($i = 0 ; $i < count($boxes) ; $i++) { - if ($save_as_draft && - eregi('^' . quotemeta($draft_folder) . '(' . - quotemeta($delimiter) . '.*)?$', $boxes[$i]['unformatted'])) { - $boxesnew[] = $boxes[$i]; - $used[$i] = true; + + $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; } } - - /* Put INBOX.* folders ahead of the rest. */ - for ($i = 0; $i < count($boxes); $i++) { - if (eregi('^inbox\\.', $boxes[$i]["unformatted"]) && - (!isset($used[$i]) || $used[$i] == false)) { - $boxesnew[] = $boxes[$i]; - $used[$i] = true; + + /* 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; + } } + } - } - - /* Rest of the folders */ - for ($i = 0; $i < count($boxes); $i++) { - if ((strtolower($boxes[$i]["unformatted"]) != "inbox") && - (!isset($used[$i]) || $used[$i] == false)) { - $boxesnew[] = $boxes[$i]; - $used[$i] = 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; - global $delimiter; - - if (!function_exists ("ary_sort")) { - include_once('../functions/array.php'); - } - +/* + * Returns a list of all folders, subscribed or not + */ +function sqimap_mailbox_list_all($imap_stream) { + global $list_special_folders_first, $folder_prefix, $delimiter; + + require_once('../functions/array.php'); + $ssid = sqimap_session_id(); - $lsid = strlen( $ssid ); + $lsid = strlen( $ssid ); fputs ($imap_stream, $ssid . " LIST \"$folder_prefix\" *\r\n"); $read_ary = sqimap_read_data ($imap_stream, $ssid, true, $response, $message); $g = 0; - $phase = "inbox"; - + $phase = 'inbox'; + for ($i = 0; $i < count($read_ary); $i++) { /* Another workaround for EIMS */ if (isset($read_ary[$i + 1]) && - ereg("^(\\* [A-Z]+.*)\\{[0-9]+\\}([ \n\r\t]*)$", + 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 ) { - + if (substr($read_ary[$i], 0, $lsid) != $ssid ) { + /* Store the raw IMAP reply */ - $boxes[$g]["raw"] = $read_ary[$i]; - + $boxes[$g]['raw'] = $read_ary[$i]; + /* Count number of delimiters ($delimiter) in folder name */ $mailbox = find_mailbox_name($read_ary[$i]); - $dm_count = countCharInString($mailbox, $delimiter); + $dm_count = substr_count($mailbox, $delimiter); if (substr($mailbox, -1) == $delimiter) { /* If name ends in delimiter - decrement count by one */ - $dm_count--; + $dm_count--; } - - /* Format folder name, but only if it's a INBOX.* or have */ - /* a parent. */ - $boxesbyname[$mailbox] = $g; + + /* 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)) || + if((eregi('^inbox'.quotemeta($delimiter), $mailbox)) || (ereg('^'.$folder_prefix, $mailbox)) || - ( isset($boxesbyname[$parentfolder]) && (strlen($parentfolder) > 0) ) ) { + ( isset($boxesallbyname[$parentfolder]) && (strlen($parentfolder) > 0) ) ) { if ($dm_count) { - $boxes[$g]["formatted"] = str_repeat("  ", $dm_count); + $boxes[$g]['formatted'] = str_repeat('  ', $dm_count); } else { - $boxes[$g]["formatted"] = ''; + $boxes[$g]['formatted'] = ''; } - $boxes[$g]["formatted"] .= readShortMailboxName($mailbox, $delimiter); + $boxes[$g]['formatted'] .= readShortMailboxName($mailbox, $delimiter); } else { - $boxes[$g]["formatted"] = $mailbox; + $boxes[$g]['formatted'] = $mailbox; } - - $boxes[$g]["unformatted-dm"] = $mailbox; + + $boxes[$g]['unformatted-dm'] = $mailbox; if (substr($mailbox, -1) == $delimiter) { $mailbox = substr($mailbox, 0, strlen($mailbox) - 1); } - $boxes[$g]["unformatted"] = $mailbox; - $boxes[$g]["unformatted-disp"] = - ereg_replace('^' . $folder_prefix, '', $mailbox); - $boxes[$g]["id"] = $g; - - /** Now lets get the flags for this mailbox **/ - fputs ($imap_stream, sqimap_session_id() . " LIST \"\" \"$mailbox\"\r\n"); - $read_mlbx = sqimap_read_data ($imap_stream, sqimap_session_id(), - true, $response, $message); - + $boxes[$g]['unformatted'] = $mailbox; + $boxes[$g]['unformatted-disp'] = ereg_replace('^' . $folder_prefix, '', $mailbox); + $boxes[$g]['id'] = $g; + + /* Now lets get the flags for this mailbox */ + $read_mlbx = sqimap_run_command ($imap_stream, "LIST \"\" \"$mailbox\"", + 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); - $flags = substr($flags, 0, strpos($flags, ")")); + + $flags = substr($read_mlbx[0], strpos($read_mlbx[0], '(')+1); + $flags = substr($flags, 0, strpos($flags, ')')); $flags = str_replace('\\', '', $flags); $flags = trim(strtolower($flags)); if ($flags) { - $boxes[$g]['flags'] = explode(" ", $flags); - } - else { + $boxes[$g]['flags'] = explode(' ', $flags); + } else { $boxes[$g]['flags'] = array(); } } $g++; } if(is_array($boxes)) { - $boxes = ary_sort ($boxes, "unformatted", 1); + $boxes = ary_sort ($boxes, 'unformatted', 1); } - + return $boxes; }