From: tokul <tokul@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Mon, 12 Jun 2006 16:26:47 +0000 (+0000)
Subject: msgs_list_copy arguments and return codes differ from msgs_list_move.
X-Git-Url: https://vcs.fsf.org/?a=commitdiff_plain;h=b400dda32b92dc7b1f0e5f1724bdaabdec13a330;p=squirrelmail.git

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 e296416b..03c74d72 100644
--- 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':