From: pdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Fri, 9 Jan 2015 05:21:38 +0000 (+0000)
Subject: Unsubscribe addresses with characters like + in them need to be url-encoded if they... 
X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=commitdiff_plain;h=57f170c8a33fd7d45768c71b27f0b3e9dbee0aa9

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
---

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]);