Aparently, within a javascript "window" function you don't need to escape

&-s in the url.

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@4308 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/src/read_body.php b/src/read_body.php
index 736df268bda8d43122a402b1e8f74181772e99db..e861c6086056260ffa3ef9257de1e498208e8645 100644 (file)
--- a/src/read_body.php
+++ b/src/read_body.php
@@ -112,8 +112,8 @@ function printer_friendly_link($mailbox, $passed_id, $passed_ent_id, $color) {
     global $javascript_on;
 
     $params = '?passed_ent_id=' . $passed_ent_id .
     global $javascript_on;
 
     $params = '?passed_ent_id=' . $passed_ent_id .

-              '&mailbox=' . urlencode($mailbox) .
-              '&passed_id=' . $passed_id;
+              '&mailbox=' . urlencode($mailbox) .
+              '&passed_id=' . $passed_id;

 
     $print_text = _("View Printable Version");
 
 
     $print_text = _("View Printable Version");
 


@@ -130,7 +130,7 @@ function printer_friendly_link($mailbox, $passed_id, $passed_ent_id, $color) {
                   "</script>\n" .
                   "<a href=\"javascript:printFormat();\">$print_text</a>\n";
     } else {
                   "</script>\n" .
                   "<a href=\"javascript:printFormat();\">$print_text</a>\n";
     } else {

-        $result = '<A target="_blank" HREF="../src/printer_friendly_bottom.php' .
+        $result = '<a target="_blank" href="../src/printer_friendly_bottom.php' .

                   "$params\">$print_text</a>\n";
     }
     return $result;
                   "$params\">$print_text</a>\n";
     }
     return $result;