_drupal_wrap_mail_line() does not force-wrap too long words after 996 characters

Looks good.

Can you add a simple test for this case?

Should still apply cleanly to D8, D7, and D6 (give or take a /core directory).

We already have DrupalHtmlToTextTestCase::testVeryLongLineWrap() which should cover this - but obviously hasn't.

Here for drupal 8 :)

This fails on my local installation. I think this is good to go.

This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed 0ed1f2e and pushed to 8.0.x. Thanks!

diff --git a/core/modules/system/src/Tests/Mail/HtmlToTextTest.php b/core/modules/system/src/Tests/Mail/HtmlToTextTest.php
index a90fc7e..3bf7ffc 100644
--- a/core/modules/system/src/Tests/Mail/HtmlToTextTest.php
+++ b/core/modules/system/src/Tests/Mail/HtmlToTextTest.php
@@ -357,7 +357,7 @@ public function testVeryLongLineWrap() {
       $maximum_line_length = max($maximum_line_length, strlen($line . $eol));
     }
     $verbose = 'Maximum line length found was ' . $maximum_line_length . ' octets.';
-    $this->assert($maximum_line_length <= 1000, $verbose);
+    $this->assertTrue($maximum_line_length <= 1000, $verbose);
   }
 
   /**

Minor fix on commit.

I don't think this has been committed to Drupal 7 based on the commit references in comments #14 and #16.

Changing status of patch for Drupal 7 to RTBC since the patch in #15 for D7 is the same as that committed to D8.

Committed to 7.x - thanks!

This issue has the "needs backport to D6" tag, but I'm guessing that's not happening at this point. Feel free to reopen for Drupal 6 if you want to try :)

This is incorrect anyway, since wordwrap() can't deal with utf-8 (and mail bodies can be in utf-8!).
See https://www.drupal.org/node/364670 for more accurate solution.