Changeset 1564 in subversion

Timestamp:

Jun 20, 2008 7:40:05 AM (5 years ago)

Author:

thomasb

Message:

Urlencode mail recipients for storing in session + little code cleanup

Location:

trunk/roundcubemail/program

Files:

• include/main.inc (modified) (1 diff)
• include/rcmail.php (modified) (1 diff)
• include/rcube_json_output.php (modified) (1 diff)
• steps/addressbook/mailto.inc (modified) (1 diff)
• steps/mail/compose.inc (modified) (1 diff)

trunk/roundcubemail/program/include/main.inc

@@ -129 +129 @@
 {
   $app = rcmail::get_instance();
-  
-  $qstring = '';
-  $base = $app->comm_path;
-  
-  if ($task && in_array($task, rcmail::$main_tasks))
-    $base = ereg_replace('_task=[a-z]+', '_task='.$task, $app->comm_path);
-  
-  if (is_array($p))
-    foreach ($p as $key => $val)
-      $qstring .= '&'.urlencode($key).'='.urlencode($val);
-  
-  return $base . ($action ? '&_action='.$action : '') . $qstring;
+  return $app->url((array)$p + array('_action' => $action, 'task' => $task));
 }
 

trunk/roundcubemail/program/include/rcmail.php

@@ -831 +831 @@
   {
     if (!is_array($p))
-      $p = array('action' => @func_get_arg(0));
-      
-    $url = $p['task'] ? './?_task=' . $p['task'] : $this->comm_path;
+      $p = array('_action' => @func_get_arg(0));
+    
+    if ($p['task'] && in_array($p['task'], rcmail::$main_tasks))
+      $url = './?_task='.$p['task'];
+    else
+      $url = $this->comm_path;
+    
     unset($p['task']);
-    
     foreach ($p as $par => $val)
-      $url .= sprintf('&%s=%s', urlencode($par), urlencode($val));
+      $url .= '&'.urlencode($par).'='.urlencode($val);
     
     return $url;

trunk/roundcubemail/program/include/rcube_json_output.php

@@ -170 +170 @@
      * @see rcmail::url()
      */
-    public function redirect($p = array(), $delay = 0)
+    public function redirect($p = array(), $delay = 1)
     {
         $location = rcmail::get_instance()->url($p);

trunk/roundcubemail/program/steps/addressbook/mailto.inc

@@ -38 +38 @@
   $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

trunk/roundcubemail/program/steps/mail/compose.inc

@@ -136 +136 @@
       // 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'];