X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=src%2Fmove_messages.php;h=d142747de2d3b05a8198f7ebc11857b6fcb5efe1;hb=324ac3c54f72cb39a994d9a80a586fce591fca84;hp=d9a2fda19c84743767ac6d6ce597d72becc28d59;hpb=d0519c03c82b5291db9928f587c3b13aa8f6a705;p=squirrelmail.git diff --git a/src/move_messages.php b/src/move_messages.php index d9a2fda1..d142747d 100644 --- a/src/move_messages.php +++ b/src/move_messages.php @@ -1,17 +1,31 @@ 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, TRUE); + + 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(); return $composesession; } - - /* get globals */ sqgetGlobalVar('key', $key, SQ_COOKIE); sqgetGlobalVar('username', $username, SQ_SESSION); @@ -140,14 +123,39 @@ sqgetGlobalVar('expungeButton', $expungeButton, SQ_POST); sqgetGlobalVar('undeleteButton', $undeleteButton, SQ_POST); sqgetGlobalVar('markRead', $markRead, SQ_POST); sqgetGlobalVar('markUnread', $markUnread, SQ_POST); +sqgetGlobalVar('markFlagged', $markFlagged, SQ_POST); +sqgetGlobalVar('markUnflagged', $markUnflagged, SQ_POST); sqgetGlobalVar('attache', $attache, SQ_POST); sqgetGlobalVar('location', $location, SQ_POST); +sqgetGlobalVar('bypass_trash', $bypass_trash, SQ_POST); +sqgetGlobalVar('dmn', $is_dmn, SQ_POST); + + /* end of get globals */ $imapConnection = sqimap_login($username, $key, $imapServerAddress, $imapPort, 0); $mbx_response=sqimap_mailbox_select($imapConnection, $mailbox); + +global $allow_thread_sort, $auto_expunge; + +if ($allow_thread_sort && getPref($data_dir, $username, "thread_$mailbox",0)) { + $aMailbox['SORT_METHOD'] = 'THREAD'; +} else if ($allow_server_sort) { + $aMailbox['SORT_METHOD'] = 'SERVER'; +} else { + $aMailbox['SORT_METHOD'] = 'SQUIRREL'; +} +sqgetGlobalVar('aLastSelectedMailbox',$aMailbox,SQ_SESSION); +sqgetGlobalVar('server_sort_array', $server_sort_array, SQ_SESSION); +$aMailbox['UIDSET'] = $server_sort_array; +$aMailbox['SORT'] = $sort; +$aMailbox['NAME'] = $mailbox; +$aMailbox['EXISTS'] = $mbx_response['EXISTS']; +$aMailbox['AUTO_EXPUNGE'] = $auto_expunge; +$aMailbox['MSG_HEADERS'] = $msgs; + $location = set_url_var($location,'composenew',0,false); $location = set_url_var($location,'composesession',0,false); $location = set_url_var($location,'session',0,false); @@ -161,119 +169,91 @@ if ($targetMailbox != $lastTargetMailbox) { sqsession_register($lastTargetMailbox, 'lastTargetMailbox'); } $exception = false; +$change = false; 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; + } +} +$num_ids = count($id); + // 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); - } - } + $num_ids = sqimap_mailbox_expunge($imapConnection, $mailbox, true); + $change = true; } 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 ($num_ids) { + 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 ($num_ids) { + if (!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 if (isset($markFlagged)) { + sqimap_toggle_flag($imapConnection, $id, '\\Flagged', true, true); + } else if (isset($markUnflagged)) { + sqimap_toggle_flag($imapConnection, $id, '\\Flagged', false, true); + } else { // Delete messages + if (!boolean_hook_function('move_messages_button_action', NULL, 1)) { + sqimap_msgs_list_delete($imapConnection, $mailbox, $id,$bypass_trash); + if ($auto_expunge) { + $num_ids = sqimap_mailbox_expunge($imapConnection, $mailbox, true); + } + $change = true; + } + } + } else { + $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); } - $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); - } - } - } } 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++; + if ( $num_ids > 0 ) { + if ( $is_dmn && $num_ids == 1 ) { + sqimap_msgs_list_move($imapConnection,$id[0],$targetMailbox); + $num_ids = sqimap_mailbox_expunge_dmn($imapConnection,$aMailbox, $id[0]); + } else { + sqimap_msgs_list_move($imapConnection,$id,$targetMailbox); + if ($auto_expunge) { + $num_ids = sqimap_mailbox_expunge($imapConnection, $mailbox, true); } - $i++; } - sqimap_msgs_list_copy($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); - } - } + $change = true; } else { - $exception = true; + $exception = true; + } +} +if($change) { // Change the startMessage number if the mailbox was changed + if (($startMessage+$num_ids-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); + } } } // Log out this session @@ -285,5 +265,136 @@ if ($exception) { header("Location: $location"); exit; } + +function handleMessageListForm($imapConnection,&$aMailbox) { + /* incoming formdata */ + sqgetGlobalVar('moveButton', $moveButton, SQ_POST); + sqgetGlobalVar('expungeButton', $expungeButton, SQ_POST); + sqgetGlobalVar('targetMailbox', $targetMailbox, SQ_POST); + sqgetGlobalVar('expungeButton', $expungeButton, SQ_POST); + sqgetGlobalVar('undeleteButton', $undeleteButton, SQ_POST); + sqgetGlobalVar('markRead', $markRead, SQ_POST); + sqgetGlobalVar('markUnread', $markUnread, SQ_POST); + sqgetGlobalVar('markFlagged', $markFlagged, SQ_POST); + sqgetGlobalVar('markUnflagged', $markUnflagged, SQ_POST); + sqgetGlobalVar('attache', $attache, SQ_POST); + sqgetGlobalVar('location', $location, SQ_POST); + sqgetGlobalVar('bypass_trash', $bypass_trash, SQ_POST); + sqgetGlobalVar('msg', $msg, SQ_POST); + + $sError = ''; + /* retrieve the check boxes */ + $aUid = array(); + if (isset($msg) && is_array($msg)) { + foreach( $msg as $key=>$iUid ) { + // using foreach removes the risk of infinite loops that was there // + $aUid[] = $iUid; + } + } + $num_ids = count($id); + + if (count($num_ids) && !isset($expungeButton)) { + /* handle submit buttons */ + $sButton = ''; + $sButton = (isset($expungeButton)) ? 'expunge' : $sButton; + $sButton = (isset($attache)) ? 'attache' : $sButton; + $sButton = (isset($moveButton)) ? 'move' : $sButton; + $sButton = (isset($copyButton)) ? 'copy' : $sButton; + $sButton = (isset($markDelete)) ? 'setDeleted' : $sButton; + $sButton = (isset($markUndelete)) ? 'unsetDeleted' : $sButton; + $sButton = (isset($markSeen)) ? 'setSeen' : $sButton; + $sButton = (isset($markUnseen)) ? 'unsetSeen' : $sButton; + $sButton = (isset($markFlagged)) ? 'setFlagged' : $sButton; + $sButton = (isset($markUnflagged)) ? 'unsetFlagged' : $sButton; + + $aUpdatedMsgs = false; + $bExpunge = false; + switch ($sButton) { + case 'setDeleted': + // What kind of hook is this, can it be removed? + if (!boolean_hook_function('move_messages_button_action', NULL, 1)) { + $aUpdatedMsgs = sqimap_msgs_list_delete($imapConnection, $mailbox, $aUid,$bypass_trash); + $bExpunge = true; + } + break; + case 'unsetDeleted': + case 'setSeen': + case 'unsetSeen': + case 'setFlagged': + case 'unsetFlagged': + // get flag + $sFlag = (substr($sButton,0,3) == 'set') ? '\\'.substr($sButton,3) : '\\'.substr($sButton,5); + $bSet = (substr($sButton,0,3) == 'set') ? true : false; + $aUpdatedMsgs = sqimap_toggle_flag($imapConnection, $aUid, $sFlag, $bSet, true); + break; + case 'move': + $aUpdatedMsgs = sqimap_msgs_list_move($imapConnection,$aId,$targetMailbox); + $bExpunge = true; + break; + case 'attache': + $composesession = attachSelectedMessages($id, $imapConnection); + // dirty hack, add info to $aMailbox + $aMailbox['FORWARD_SESSION'] = $composesession; + break; + } + + if ($aUpdatedMsgs) { + foreach ($aUpdatedMsgs as $iUid => $aMsg) { + if (isset($aMsg['FLAGS'])) { + $aMailbox['MSG_HEADERS'][$iUid]['FLAGS'] = $aMsg['FLAGS']; + } + } + if ($bExpunge && $aMailbox['AUTO_EXPUNGE'] && + $iExpungedMessages = sqimap_mailbox_expunge($imapConnection, $aMailbox['NAME'], true)) + { + if (count($aUpdateMsgs != $iExpungedMessages)) { + // there are more messages deleted permanently then we expected + // invalidate the cache + $aMailbox['UIDSET'] = false; + $aMailbox['MSG_HEADERS'] = false; + } else { + // remove expunged messages from cache + $aUidSet = $aMailbox['UIDSET']; + $aDeleted = array(); + foreach ($aUpdatedMsgs as $iUid => $aValue) { + if (isset($aValue['FLAGS']['\\deleted']) && $aValue['FLAGS']['\\deleted']) { + $aDeleted[] = $iUid; + } + } + if (count($aDeleted)) { + // create a UID => array index temp array + $aUidSetDummy = array_flip($aUidSet); + foreach ($aDeleted as $iUid) { + unset($aUidSetDummy[$iUid]); + } + $aUidSet = array_keys($aUidSetDummy); + $aMailbox['UIDSET'] = $aUidSet; + // update EXISTS info + $aMailbox['EXISTS'] -= $iExpungedMessages; + } + } + // Change the startMessage number if the mailbox was changed + if (($aMailbox['PAGEOFFSET']+$iExpungedMessages-1) >= $aMailbox['EXISTS']) { + $aMailbox['PAGEOFFSET'] = ($aMailbox['PAGEOFFSET'] > $aMailbox['LIMIT']) ? + $aMailbox['PAGEOFFSET'] - $aMailbox['LIMIT'] : 1; + } + } + } + } else { + if (isset($expungeButton)) { + // on expunge we do not know which messages will be deleted + // so it's useless to try to sync the cache + + // Close the mailbox so we do not need to parse the untagged expunge responses + sqimap_run_command($imapConnection,'CLOSE',false,$result,$message); + $aMbxResponse = sqimap_select($imapConnection,$aMailbox['NAME']; + // update the $aMailbox array + $aMailbox['EXISTS'] = $aMbxResponse['EXISTS']; + $aMailbox['UIDSET'] = false; + } else { + $sError = _("No messages were selected."); + } + } +} ?>