msgs_list_copy arguments and return codes differ from msgs_list_move.

php 4.3.10 does not complain about boolean true feeded to foreach. new php
versions do complain about it. thanks to Henrik Andreasen

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@11200 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/functions/mailbox_display.php b/functions/mailbox_display.php
index e296416bbf49ccd1799d8636fe9dc0e1fb934b2c..03c74d7222e1d8f16b00151b71f15f5d16f042d2 100644 (file)
--- a/functions/mailbox_display.php
+++ b/functions/mailbox_display.php
@@ -1286,7 +1286,9 @@ function handleMessageListForm($imapConnection,&$aMailbox,$sButton='',$aUid = ar
             $bExpunge = true;
             break;
           case 'copy':
-            $aUpdatedMsgs = sqimap_msgs_list_copy($imapConnection,$aUid,$targetMailbox,true,$mailbox);
+            // sqimap_msgs_list_copy returns true or false.
+            // If error happens - fourth argument handles it inside function.
+            sqimap_msgs_list_copy($imapConnection,$aUid,$targetMailbox,true);
             sqsession_register($targetMailbox,'lastTargetMailbox');
             break;
           case 'forward':