#18. Kevin on 14 January 2009

@ John Peterson: Of course: that's not at all like PHP. And that's what we're trying to recreate here: So the main version should be as is it right now.

However. Just like there's a namespaced version of PHP.JS, I think it should be possible for us to generate prototype versions with the compiler.

Currently this is a low priority for me and I would first like to create a public interface to the compiler on the phpjs.org site.
... [more]
But I will keep this in mind as a future feature!

#17. Brett Zamir on 14 January 2009

I should add the extra qualification that overloading the Object or Array prototype would probably be quite objectionable to many and need some serious revising of the PHP-JS functions (which I'm thinking should probably be revised regardless) to check for hasOwnProperty in each iterating for loop.

#16. Brett Zamir on 14 January 2009

HI John,

I think concerns on this range from "What happens if a standard version gets implemented later?" to "Will this interfere with other implementations added to the prototype (as a kind of "global" problem?"), etc.. However, I personally think this could be very convenient for those willing to take the risk (or who reset the prototype after their code is done with it).

Maybe when we get the string functions done ( ;) ), we can make one version of the PHP-JS package which adds to the prototype (also for Array, Object, Number, or maybe Function or Boolean if you can think of a reason!), as I'm hoping we can see a configurable version made at some point (as an independent project, if not through Kevin's official version).

#15. John Peterson on 14 January 2009

Why not just do this so much simplier...

String.prototype.nl2br = function() {
var breakTag = '<br />';
return (this + '').replace(/([^>]?)\n/g, '$1'+ breakTag +'\n');
... [more] }

then you can just do:

mystring.nl2br()

#14. Kevin on 13 October 2008

@ Atli Þór: That's a Great contribution Atli Þór. I wish I could give your more credit, than the current system can give you. Thanks man.

#13. Atli Þór on 10 October 2008

Not sure why my last post got messed up like that.
If it was bad formatting on my part I apologize.

#12. Atli Þór on 10 October 2008

Hi.

There seems to be a minor flaw in the function for strings that start with a line-breaks, or have multiple consecutive line-breaks.

Using this:

nl2br("\nOne\nTwo\n\nThree\n")

it leaves out the first new-line and replaces the two new-lines between "Two" and "Three" with a single break.

Making the regex non-greedy seems to fix this tho:

return (str + '').replace(/([^>]?)\n/g, '$1
\n');

Additionally, PHP 5.3 will accept a second parameter, indicating whether the break should be XHTML compliant or not.

This would allow for that as well:

function nl2br( str, is_xhtml ) {
    breakTag = "
";
    if(typeof is_xhtml == "boolean" || is_xhtml == false) {
        breakTag = "<br>";
    }
    return (str + '').replace(/([^>]?)\n/g, '$1'+ breakTag +'\n');
}

#11. Kevin on 29 September 2008

@ Onno Marsman: Nice :) Function has been updated according to your proposal!

#10. Onno Marsman on 26 September 2008

lol, still no good. Kevin: you know what I mean. I've only added (str+'')

#9. Onno marsman on 26 September 2008

I guess html is not escaped correctly, I meant:

function nl2br( str ) {
    return (str+'').replace(/([^>])\n/g, '$1<br />\n');
}

#8. Onno Marsman on 26 September 2008

To make sure it's a string i.e. nl2br(6):

function nl2br( str ) {
     return (str+'').replace(/([^>])\n/g, '$1
\n');
}

#7. Music Russia on 08 September 2008

Sorry, Kevin, that's just my stupid mistake, i confused one var with another. I spend about 15 minutes until figured it out.

Here's the part of my code and it 100% works now:

var nfull = document.getElementById('full').value;
nfull = nfull.replace(/([^>])\\n/g, '$1
');
document.getElementById('txtpr').innerHTML = nfull;

Thank you for the hint!

#6. Kevin on 08 September 2008

@ Music Russia: Please be more specific, can you for example provide the code how you call it? And explain what breaks? Thanks a lot!

#5. Music Russia on 06 September 2008

Do not work in my case.

#4. Kevin on 21 May 2008

@ Joshua: Thank you, we're still working on a tool to customize & save your own php.js version, maybe that would be something for you then!

#3. Joshua on 21 May 2008

Nice piece of js code, very usefull :) I'm not using the whole package but just little pieces of code, great work!

#2. Kevin on 15 April 2008

@ Philip Peterson: I've updated the function with your suggestions. Thank you!

#1. Philip Peterson on 15 April 2008

There are a few inconsistencies here:

first off, the replacement text should be "<br />\n" (with the space before the backslash in <br /> and including the newline). Also, I don't have time to do this, but you may want to check out how PHP handles \r\n and \r in this function, as it may be useful to implement support for if the PHP function does it.