http://www.activecollab.com/forums/topic/1620/ Recent posts on topic: clean() Functionality Enhancement en-us Sat, 06 Sep 2008 13:15:32 CDT http://www.activecollab.com/forums/post/7734/#post7734 http://www.activecollab.com/forums/post/7734/#post7734 levi I did the same thing firespade. I think this functionality is desired by most (if not all) users.

]]> Tue, 15 May 2007 09:12:05 CDT http://www.activecollab.com/forums/post/7724/#post7724 http://www.activecollab.com/forums/post/7724/#post7724 firespade Got tired of CRLF being associated in each use of the clean() method throughout activeCollab.

So I tracked down the method in '/environment/functions/general.php' on (or around) line 72.

This method looks like:
<?php
function clean($str) {
$str = preg_replace('/&(?!#[0-9]+;)/s', '&amp;', $str);
$str = str_replace(array('<', '>', '"'), array('&lt;', '&gt;', '&quot;'), $str);
return $str;
} // clean
?>

Below the str_replace() iteration you can add a nl2br() method to convert the CRLF to line break tags (eg. <br>). My new clean() method looks like..

<?php
function clean($str) {
$str = preg_replace('/&(?!#[0-9]+;)/s', '&amp;', $str);
$str = str_replace(array('<', '>', '"'), array('&lt;', '&gt;', '&quot;'), $str);
$str = nl2br($str);
return $str;
} // clean
?>

The result? Instead of never having line breaks (which seems to drive my team mad), every time ENTER is pressed you will get your expected result.

Regards,
Anthony aka firespade

]]> Mon, 14 May 2007 13:33:16 CDT