Added more tests to wrap. Fixed to not cut words with more than shoul…

…d limit.
cakephp · Apr 13, 2011 · f661e37 · f661e37
1 parent 985743d
commit f661e37
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 1 deletion.
diff --git a/lib/Cake/Network/CakeEmail.php b/lib/Cake/Network/CakeEmail.php
@@ -889,7 +889,7 @@ function _wrap($message) {
 				$line = '.' . $line;
 			}
 			if (!preg_match('/\<[a-z]/i', $line)) {
-				$formatted = array_merge($formatted, explode("\n", wordwrap($line, self::LINE_LENGTH_SHOULD, "\n", true)));
+				$formatted = array_merge($formatted, explode("\n", wordwrap($line, self::LINE_LENGTH_SHOULD, "\n")));
 				continue;
 			}
 

diff --git a/lib/Cake/tests/Case/Network/CakeEmailTest.php b/lib/Cake/tests/Case/Network/CakeEmailTest.php
@@ -509,6 +509,26 @@ public function testWrap() {
 			''
 		);
 		$this->assertIdentical($result, $expected);
+
+		$text = 'Lorem ipsum <a href="http://www.cakephp.org/controller/action/param1/param2" class="nice cool fine amazing awesome">ok</a>';
+		$result = $this->CakeEmail->wrap($text);
+		$expected = array(
+			'Lorem ipsum',
+			'<a href="http://www.cakephp.org/controller/action/param1/param2" class="nice cool fine amazing awesome">',
+			'ok</a>',
+			''
+		);
+		$this->assertIdentical($result, $expected);
+
+		$text = 'Lorem ipsum withonewordverybigMorethanthelineshouldsizeofrfcspecificationbyieeeavailableonieeesite ok.';
+		$result = $this->CakeEmail->wrap($text);
+		$expected = array(
+			'Lorem ipsum',
+			'withonewordverybigMorethanthelineshouldsizeofrfcspecificationbyieeeavailableonieeesite',
+			'ok.',
+			''
+		);
+		$this->assertIdentical($result, $expected);
 	}
 
 }