From 57f170c8a33fd7d45768c71b27f0b3e9dbee0aa9 Mon Sep 17 00:00:00 2001
From: pdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Fri, 9 Jan 2015 05:21:38 +0000
Subject: [PATCH] Unsubscribe addresses with characters like + in them need to
 be url-encoded if they are going to be used in a query string. Not sure what
 the ? to &amp; replacement was doing (was it broken?), but it is possible I
 broke something by removing it.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14487 7612ce4b-ef26-0410-bec9-ea0150e637f0
---
 plugins/listcommands/functions.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/listcommands/functions.php b/plugins/listcommands/functions.php
index 55da9db4..5e71da5c 100644
--- a/plugins/listcommands/functions.php
+++ b/plugins/listcommands/functions.php
@@ -95,7 +95,7 @@ function plugin_listcommands_menu_do() {
             } else {
                 $url = "plugins/listcommands/mailout.php?action=$cmd&amp;";
             }
-            $url .= 'send_to=' . str_replace('?','&amp;', $act);
+            $url .= 'send_to=' . urlencode($act);
 
             $links[$cmd] = makeComposeLink($url, $fieldsdescr[$cmd]);
 
-- 
2.25.1