2 problems (nl2br and regular expression)

[apzer0]

1. Hello, I have a form that the user inserts some text and the text goes trought a few functions, one of them is nl2br.
   When I (Or the user) want to edit that text, it is shown at the edit textbox with all the <br /> tags.
   I looked around php.net and found out that the nl2br function does not replace the \n with <br /> but just add <br /> (Which results in <br />\n) so I used $Text=str_replace("\n", "<br />",$Text); insted of nl2br and $Text=str_replace("<br />","\n",$Edit['Text']); when taking the text out for editing.
   It works ok except for one thing, every time the text is edited a new line is added at the end of it.
   Any ideas how to stop the code from adding an new line ate the end of the text (Without using trim)?
2. I have a script (For the same textbox) that is supose to stop an empty form (Even if only one field is empty it will generate an error).
   The script stops empty forms and if only one field is empty it stops it too but I want it to stop forms that have only white tags (Space, new line...) in it too which it doesn't =/
   The script is:
   if (preg_match("/\S/i",$POST['Name'] && $POST['Text'])){
   Any idea why?
   Sorry if my spelling isn't that good :queasy:

[nemonoman]

1. Check your textarea code:
   <textarea>$mytext</textarea>
   acts differently than
   <textarea>
   $mytext
   </textarea>

The second example would add a newline to end $mytext when posted

[drew010]

for problem number 1. make sure there isnt a newline at the end of the text box. this would cause the other space.
your second problem is because of the regular expression. \S means any non-whitespace character and its only looking to match one instance at the beginning of the string. You also cant use && the way you did there. you would need to do the regex twice or apply it to both strings in the same if.

if (preg_match("/^(\s*)$/", $_POST['Text'])){ 
//is proper, or
if (preg_match("/^(\s*)$/", $_POST['Text']) || preg_match("/^(\s*)$/", $_POST['Name'])) { 

[apzer0]

Thanks you guys really helped me 🙂