Changeset 2b180b1 in github

Timestamp:

Oct 13, 2008 3:05:31 AM (5 years ago)

Author:

alecpl <alec@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

87b7959

Parents:

34942eb

Message:

#1485494: fixed signatures removal on reply

File:

• program/steps/mail/compose.inc (modified) (2 diffs)

program/steps/mail/compose.inc

@@ -500 +500 @@
   if (! $bodyIsHtml)
   {
+    // try to remove the signature
+    if (($sp = strrpos($body, '-- ')) !== false && ($sp == 0 || $body{$sp-1} == "\n"))
+      {
+      if ($body{$sp+3}==' ' || $body{$sp+3}=="\n" || $body{$sp+3}=="\r")
+        $body = substr($body, 0, max(0, $sp-1));
+      }
+
     // soft-wrap message first
     $body = rcmail_wrap_quoted($body, 75);
-  
-    // split body into single lines
-    $a_lines = preg_split('/\r?\n/', $body);
-  
-    // add > to each line
-    for($n=0; $n<sizeof($a_lines); $n++)
-    {
-      if (strpos($a_lines[$n], '>')===0)
-        $a_lines[$n] = '>'.$a_lines[$n];
-      else
-        $a_lines[$n] = '> '.$a_lines[$n];
-    }
+
+    $body = rtrim($body, "\r\n");
+
+    if ($body) {
+      // split body into single lines
+      $a_lines = preg_split('/\r?\n/', $body);
+
+      // add > to each line
+      for($n=0; $n<sizeof($a_lines); $n++) {
+        if (strpos($a_lines[$n], '>')===0)
+          $a_lines[$n] = '>'.$a_lines[$n];
+        else
+          $a_lines[$n] = '> '.$a_lines[$n];
+        }
  
-    $body = join("\n", $a_lines);
+      $body = join("\n", $a_lines);
+      }
 
     // add title line
@@ -522 +532 @@
       $MESSAGE->get_header('from'));
 
-    // try to remove the signature
-    if ($sp = strrpos($body, '-- '))
-      {
-      if ($body{$sp+3}==' ' || $body{$sp+3}=="\n" || $body{$sp+3}=="\r")
-        $body = substr($body, 0, $sp-1);
-      }
     $suffix = '';
   }