Save as draft broken in recent SQM 1.5.2 SVN versions (patch attached)

Thierry Godefroy <[email protected]>
Newsgroups gmane.mail.squirrelmail.devel
Message-ID <[email protected]>
Greetings,

The parameters list for the mail() method in the Deliver_IMAP() class was recently changed (one $imap_stream parameter added) and compose.php was patched in one place (for actual mail demlicery code) but misses a patch for the "save as draft" call to this function, breaking "Save as draft" altogether.

The attached patch fixes this bug.

Regards,

Thierry.


      ____________________________________________________________________________________
Be a better friend, newshound, and 
know-it-all with Yahoo! Mobile.  Try it now.  http://mobile.yahoo.com/;_ylt=Ahu06i62sR8HDtDypao8Wcj9tAcJ

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft 
Defy all challenges. Microsoft(R) Visual Studio 2008. 
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/

-----
squirrelmail-devel mailing list
Posting guidelines: http://squirrelmail.org/postingguidelines
List address: [email protected]
List archives: http://news.gmane.org/gmane.mail.squirrelmail.devel
List info (subscribe/unsubscribe/change options): https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel
squirrelmail-20080513-SVN_devel-BrokenSaveAsDraft.patch (text/x-patch, 814 B)
diff -urN squirrelmail/src/compose.php squirrelmail-patched/src/compose.php
--- squirrelmail/src/compose.php	2008-04-28 04:19:12.000000000 +0200
+++ squirrelmail-patched/src/compose.php	2008-05-13 11:04:12.000000000 +0200
@@ -1606,7 +1606,7 @@
         if (sqimap_mailbox_exists ($imap_stream, $draft_folder)) {
             require_once(SM_PATH . 'class/deliver/Deliver_IMAP.class.php');
             $imap_deliver = new Deliver_IMAP();
-            $success = $imap_deliver->mail($composeMessage, $imap_stream, $reply_id, $reply_ent_id, $draft_folder);
+            $success = $imap_deliver->mail($composeMessage, $imap_stream, $reply_id, $reply_ent_id, $imap_stream, $draft_folder);
             sqimap_logout($imap_stream);
             unset ($imap_deliver);
             $composeMessage->purgeAttachments();
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.