wordwrap problem

[MDanz]

When i check the source code I get this error

 

rotator cuff<br />strengthenin<br..

 

you can see the br tag is cutt off.  The code is below.

 

$thename = "rotator cuff strengthening";

 

	$newname = wordwrap($thename, 12, "<br />", true);

$limit = 33;

	if (strlen($newname) > $limit)
	{ 
		 $newname2 = substr($newname, 0, $limit) . '..'; 

	 }
		 else
	{
		$newname2 = $newname;

	}			

[redixx]

Try this:

 

function wordwrap($text, $limit)
{
if (strlen($text) < $limit) {
	return $text;
}

$text = substr($text, 0, $limit);

$pos = strpos($text,' ');
$text = substr($text, 0, $pos);

return $text;
}

 

First, it cuts the text to $limit. Then, it searches for the position of the last occurring space and then cuts the text at that location, so you don't get any cut-off words.

[Fadion]

You better limit the string first and than wordwrap it. That way you won't be truncating line breaks.

 

Adding to it, word wrapping is better done with CSS than PHP. You can limit the string with PHP and than wrap it with a fixed-width container and a word-wrap:break-word CSS property. It's safe enough.

[redixx]

Oops, I guess I read the OP wrong... oh well.

[MDanz]

thanks for the replies but do you know why i get the error? it should be a full <br /> tag instead of <br..

 

 

[redixx]

Because you truncated it with substr().