Fix rcmail::url(), don't use comm_path

program/include/rcmail.php

@@ -891,17 +891,19 @@ class rcmail
   {
     if (!is_array($p))
       $p = array('_action' => @func_get_arg(0));
-    
+
-    if ($p['task'] && in_array($p['task'], rcmail::$main_tasks))
+    if (!$p['task'] || !in_array($p['task'], rcmail::$main_tasks))
-      $url = './?_task='.$p['task'];
+      $p['task'] = $this->task;
-    else
+
-      $url = $this->comm_path;
+    $url = './';
-    
+    $delm = '?';
-    unset($p['task']);
+    foreach (array_reverse($p) as $par => $val)
-    foreach ($p as $par => $val)
+    {
-      if (isset($val))
+      if (!empty($val)) {
-        $url .= '&'.urlencode($par).'='.urlencode($val);
+        $url .= $delm.urlencode($par).'='.urlencode($val);
-    
+        $delm = '&';
+      }
+    }
     return $url;
   }
 }