* Enables message moving between folders on the IMAP server.
*
* $Id$
+ * @package squirrelmail
*/
-/* Path for SquirrelMail required files. */
+/** Path for SquirrelMail required files. */
define('SM_PATH','../');
/* SquirrelMail required files. */
global $compose_new_win;
if ( !sqgetGlobalVar('composesession', $composesession, SQ_SESSION) ) {
- $composesession = 0;
-}
-
-/* obsolete ?? */
-function putSelectedMessagesIntoString($msg) {
- $j = 0;
- $i = 0;
- $firstLoop = true;
- // If they have selected nothing msg is size one still, but will
- // be an infinite loop because we never increment j. so check to
- // see if msg[0] is set or not to fix this.
- while (($j < count($msg)) && ($msg[0])) {
- if ($msg[$i]) {
- if ($firstLoop != true) {
- $selectedMessages .= "&";
- } else {
- $firstLoop = false;
- }
- $selectedMessages .= "selMsg[$j]=$msg[$i]";
- $j++;
- }
- $i++;
- }
+ $composesession = 0;
}
function attachSelectedMessages($msg, $imapConnection) {
- global $username, $attachment_dir,
+ global $username, $attachment_dir, $startMessage,
$data_dir, $composesession, $uid_support,
- $msgs, $thread_sort_messages, $allow_server_sort, $show_num,
- $compose_messages;
+ $msgs, $show_num, $compose_messages;
if (!isset($compose_messages)) {
- $compose_messages = array();
- sqsession_register($compose_messages,'compose_messages');
+ $compose_messages = array();
+ sqsession_register($compose_messages,'compose_messages');
}
if (!$composesession) {
- $composesession = 1;
- sqsession_register($composesession,'composesession');
- } else {
- $composesession++;
- sqsession_register($composesession,'composesession');
- }
-
- $hashed_attachment_dir = getHashedDir($username, $attachment_dir, $composesession);
-
- if ($thread_sort_messages || $allow_server_sort) {
- $start_index=0;
+ $composesession = 1;
+ sqsession_register($composesession,'composesession');
} else {
- $start_index = ($startMessage-1) * $show_num;
+ $composesession++;
+ sqsession_register($composesession,'composesession');
}
- $i = 0;
- $j = 0;
$hashed_attachment_dir = getHashedDir($username, $attachment_dir);
$composeMessage = new Message();
$composeMessage->rfc822_header = $rfc822_header;
$composeMessage->reply_rfc822_header = '';
- while ($j < count($msg)) {
- if (isset($msg[$i])) {
- $id = $msg[$i];
- $body_a = sqimap_run_command($imapConnection, "FETCH $id RFC822",true, $response, $readmessage, $uid_support);
- if ($response = 'OK') {
- $k = $i + $start_index;
- $subject = $msgs[$k]['SUBJECT'];
-
- array_shift($body_a);
- $body = implode('', $body_a);
- $body .= "\r\n";
-
- $localfilename = GenerateRandomString(32, 'FILE', 7);
- $full_localfilename = "$hashed_attachment_dir/$localfilename";
-
- $fp = fopen( $full_localfilename, 'wb');
- fwrite ($fp, $body);
- fclose($fp);
- $composeMessage->initAttachment('message/rfc822',$subject.'.eml',
- $full_localfilename);
- }
- $j++;
- }
- $i++;
+ foreach($msg as $id) {
+ $body_a = sqimap_run_command($imapConnection, "FETCH $id RFC822", true, $response, $readmessage, $uid_support);
+
+ if ($response == 'OK') {
+
+ // fetch the subject for the message with $id from msgs.
+ // is there a more efficient way to do this?
+ foreach($msgs as $k => $vals) {
+ if($vals['ID'] == $id) {
+ $subject = $msgs[$k]['SUBJECT'];
+ break;
+ }
+ }
+
+ array_shift($body_a);
+ array_pop($body_a);
+ $body = implode('', $body_a);
+ $body .= "\r\n";
+
+ $localfilename = GenerateRandomString(32, 'FILE', 7);
+ $full_localfilename = "$hashed_attachment_dir/$localfilename";
+
+ $fp = fopen( $full_localfilename, 'wb');
+ fwrite ($fp, $body);
+ fclose($fp);
+ $composeMessage->initAttachment('message/rfc822',$subject.'.msg',
+ $full_localfilename);
+ }
}
+
$compose_messages[$composesession] = $composeMessage;
sqsession_register($compose_messages,'compose_messages');
session_write_close();
sqgetGlobalVar('markUnread', $markUnread, SQ_POST);
sqgetGlobalVar('attache', $attache, SQ_POST);
sqgetGlobalVar('location', $location, SQ_POST);
+sqgetGlobalVar('bypass_trash', $bypass_trash, SQ_POST);
/* end of get globals */
do_hook('move_before_move');
+
+/*
+ Move msg list sorting up here, as it is used several times,
+ makes it more efficient to do it in one place for the code
+*/
+$id = array();
+if (isset($msg) && is_array($msg)) {
+ foreach( $msg as $key=>$uid ) {
+ // using foreach removes the risk of infinite loops that was there //
+ $id[] = $uid;
+ }
+}
+
// expunge-on-demand if user isn't using move_to_trash or auto_expunge
if(isset($expungeButton)) {
$cnt = sqimap_mailbox_expunge($imapConnection, $mailbox, true);
if (($startMessage+$cnt-1) >= $mbx_response['EXISTS']) {
if ($startMessage > $show_num) {
- $location = set_url_var($location,'startMessage',$startMessage-$show_num,false);
- } else {
- $location = set_url_var($location,'startMessage',1,false);
- }
+ $location = set_url_var($location,'startMessage',$startMessage-$show_num,false);
+ } else {
+ $location = set_url_var($location,'startMessage',1,false);
+ }
}
} elseif(isset($undeleteButton)) {
// undelete messages if user isn't using move_to_trash or auto_expunge
- if (is_array($msg) == 1) {
- // Removes \Deleted flag from selected messages
- $j = $i = 0;
- $id = array();
- // If they have selected nothing msg is size one still, but will be an infinite
- // loop because we never increment j. so check to see if msg[0] is set or not to fix this.
- while ($j < count($msg)) {
- if ($msg[$i]) {
- $id[] = $msg[$i];
- $j++;
- }
- $i++;
- }
- if (count($id)) {
- sqimap_toggle_flag($imapConnection, $id, '\\Deleted',false,true);
- }
+ // Removes \Deleted flag from selected messages
+ if (count($id)) {
+ sqimap_toggle_flag($imapConnection, $id, '\\Deleted',false,true);
} else {
- $exception = true;
+ $exception = true;
}
} elseif (!isset($moveButton)) {
- // If the delete button was pressed, the moveButton variable will not be set.
- if (is_array($msg)) {
- // Marks the selected messages as 'Deleted'
- $j = $i = $cnt = 0;
- $id = array();
- // If they have selected nothing msg is size one still, but will be an infinite
- // loop because we never increment j. so check to see if msg[0] is set or not to fix this.
- while ($j < count($msg)) {
- if (isset($msg[$i])) {
- $id[] = $msg[$i];
- $j++;
+ if (count($id)) {
+ $cnt = count($id);
+ if (!isset($attache)) {
+ $button_action = concat_hook_function('move_messages_button_action');
+ if (isset($markRead)) {
+ sqimap_toggle_flag($imapConnection, $id, '\\Seen',true,true);
+ } else if (isset($markUnread)) {
+ sqimap_toggle_flag($imapConnection, $id, '\\Seen',false,true);
+ } else {
+ if (!$button_action) {
+ sqimap_msgs_list_delete($imapConnection, $mailbox, $id,$bypass_trash);
+ if ($auto_expunge) {
+ $cnt = sqimap_mailbox_expunge($imapConnection, $mailbox, true);
+ }
+ }
}
- $i++;
- }
- if (count($id) && !isset($attache)) {
- if (isset($markRead)) {
- sqimap_toggle_flag($imapConnection, $id, '\\Seen',true,true);
- } else if (isset($markUnread)) {
- sqimap_toggle_flag($imapConnection, $id, '\\Seen',false,true);
- } else {
- sqimap_msgs_list_delete($imapConnection, $mailbox, $id);
- if ($auto_expunge) {
- $cnt = sqimap_mailbox_expunge($imapConnection, $mailbox, true);
- }
- }
}
if (isset($attache)) {
- $composesession = attachSelectedMessages($msg, $imapConnection);
- $location = set_url_var($location, 'session', $composesession, false);
- if ($compose_new_win) {
- $location = set_url_var($location, 'composenew', 1, false);
- } else {
- $location = str_replace('search.php','compose.php',$location);
- $location = str_replace('right_main.php','compose.php',$location);
- }
- } else {
- if (($startMessage+$cnt-1) >= $mbx_response['EXISTS']) {
- if ($startMessage > $show_num) {
- $location = set_url_var($location,'startMessage',$startMessage-$show_num, false);
- } else {
- $location = set_url_var($location,'startMessage',1, false);
- }
- }
- }
+ $composesession = attachSelectedMessages($id, $imapConnection);
+ $location = set_url_var($location, 'session', $composesession, false);
+ if ($compose_new_win) {
+ $location = set_url_var($location, 'composenew', 1, false);
+ } else {
+ $location = str_replace('search.php','compose.php',$location);
+ $location = str_replace('right_main.php','compose.php',$location);
+ }
+ } else {
+ if (($startMessage+$cnt-1) >= $mbx_response['EXISTS']) {
+ if ($startMessage > $show_num) {
+ $location = set_url_var($location,'startMessage',$startMessage-$show_num, false);
+ } else {
+ $location = set_url_var($location,'startMessage',1, false);
+ }
+ }
+ }
} else {
- $exception = true;
+ $exception = true;
}
} else { // Move messages
- // lets check to see if they selected any messages
- if (is_array($msg)) {
- $j = $i = 0;
- $id = array();
- // If they have selected nothing msg is size one still, but will be an infinite
- // loop because we never increment j. so check to see if msg[0] is set or not to fix this.
- $cnt = count($msg);
- while ($j < $cnt) {
- if (isset($msg[$i])) {
- $id[] = $msg[$i];
- $j++;
- }
- $i++;
- }
- sqimap_msgs_list_copy($imapConnection,$id,$targetMailbox);
+
+ if (count($id)) {
+ sqimap_msgs_list_move($imapConnection,$id,$targetMailbox);
if ($auto_expunge) {
$cnt = sqimap_mailbox_expunge($imapConnection, $mailbox, true);
- } else {
- $cnt = 0;
- }
-
- if (($startMessage+$cnt-1) >= $mbx_response['EXISTS']) {
- if ($startMessage > $show_num) {
- $location = set_url_var($location,'startMessage',$startMessage-$show_num, false);
- } else {
- $location = set_url_var($location,'startMessage',1, false);
- }
- }
+ } else {
+ $cnt = 0;
+ }
+
+ if (($startMessage+$cnt-1) >= $mbx_response['EXISTS']) {
+ if ($startMessage > $show_num) {
+ $location = set_url_var($location,'startMessage',$startMessage-$show_num, false);
+ } else {
+ $location = set_url_var($location,'startMessage',1, false);
+ }
+ }
} else {
- $exception = true;
+ $exception = true;
}
}
// Log out this session