From fde466c58c004e8f7eff7b59c78f62eef250cde9 Mon Sep 17 00:00:00 2001 From: thomascube Date: Fri, 20 Jun 2008 11:40:05 +0000 Subject: Urlencode mail recipients for storing in session + little code cleanup --- program/steps/addressbook/mailto.inc | 4 ++-- program/steps/mail/compose.inc | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'program/steps') diff --git a/program/steps/addressbook/mailto.inc b/program/steps/addressbook/mailto.inc index 101b7c043..90a779644 100644 --- a/program/steps/addressbook/mailto.inc +++ b/program/steps/addressbook/mailto.inc @@ -37,8 +37,8 @@ if (!empty($mailto)) { $mailto_str = join(', ', $mailto); $mailto_id = substr(md5($mailto_str), 0, 16); - $_SESSION['mailto'][$mailto_id] = $mailto_str; - $OUTPUT->command('redirect', rcmail_url('compose', array('_mailto' => $mailto_id), 'mail')); + $_SESSION['mailto'][$mailto_id] = urlencode($mailto_str); + $OUTPUT->redirect(array('task' => 'mail', '_action' => 'compose', '_mailto' => $mailto_id)); } else $OUTPUT->show_message('nocontactsfound', 'warning'); diff --git a/program/steps/mail/compose.inc b/program/steps/mail/compose.inc index 607cf9b05..800a9de85 100644 --- a/program/steps/mail/compose.inc +++ b/program/steps/mail/compose.inc @@ -135,7 +135,7 @@ function rcmail_compose_headers($attrib) // we have a set of recipients stored is session if (($mailto_id = $_SESSION['compose']['param']['_mailto']) && $_SESSION['mailto'][$mailto_id]) - $fvalue = $_SESSION['mailto'][$mailto_id]; + $fvalue = urldecode($_SESSION['mailto'][$mailto_id]); else if (!empty($_SESSION['compose']['param']['_to'])) $fvalue = $_SESSION['compose']['param']['_to']; -- cgit v1.2.3