Legacy code was probably wrong
authorpdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Thu, 6 Jul 2017 04:33:07 +0000 (04:33 +0000)
committerpdontthink <pdontthink@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Thu, 6 Jul 2017 04:33:07 +0000 (04:33 +0000)
git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@14702 7612ce4b-ef26-0410-bec9-ea0150e637f0

src/addrbook_search_html.php

index 7101f7ca1a8cf2241ce2c323a3770569f61e382a..38b80646b2e23a97f4868b654dbf103452b15839 100644 (file)
@@ -46,11 +46,12 @@ function addr_insert_hidden() {
            $request_mdn, $request_dr, $identity, $session, $composeMessage, $action;
 
 //FIXME Do not echo HTML from the core.  This file already uses templates mostly, so why are we echoing here at all?!?
-   if (substr($body, 0, 1) == "\r") {
-       echo addHidden('body', "\n".$body);
-   } else {
+   // someone tell me why this is needed and if so, why it isn't something like replace \r\n with \n
+   // if (substr($body, 0, 1) == "\r") {
+   //     echo addHidden('body', "\n".$body);
+   // } else {
        echo addHidden('body', $body);
-   }
+   // }
 
    if (is_object($composeMessage) && $composeMessage->entities)
        echo addHidden('attachments', urlencode(serialize($composeMessage->entities)));