Please help. Function to remove phone numbers & email address

[mark-newbies]

I have a field name brief which only contains item description, no phone numbers, web url or email as there is already email and phone number fields.

I have this code on the submit.php file, but I am getting a syntax error.

function check_fields()
{

$HTTP_POST_VARS['brief']=preg_replace("/0,1,2,3,4,5,6,7,8,9/", /www/",/@/",""$HTTP_POST_VARS['brief']);

}

Please help.. Many thanks

[mark-newbies]

mark-newbies;10998233 wrote:

I have a field name brief which only contains item description, no alpha numeric numbers, web url or email as there is already email and phone number fields.

I have this code on the submit.php file, but I am getting a syntax error.

function check_fields()
{

$HTTP_POST_VARS['brief']=preg_replace("/0,1,2,3,4,5,6,7,8,9/", /www/",/@/",""$HTTP_POST_VARS['brief']);

}

Please help.. Many thanks

not sure if the numeric numbers can be remove but I would like the @signs and url to be remove. Please help with the correct syntax or code format as I am getting error while to run the code. Many thanks in advance.

[traq]

1. Don't use $HTTP_POST_VARS. It's depreciated. Use the $_POST superglobal instead.

2. Why are you trying to filter out these things?
   2a. you're going to end up with false catches - for example, if I write, [font=monospace]look for item number 44-346-4[/font], you'll be mangling my sentence for no good reason.
   2b. you're going to have failed catches - for example, if I write [font=monospace]http://example.com[/font] (no "www"!), your function won't even notice.
   2c. people will quickly figure out how to post restricted stuff. [font=monospace]myname [at] example [dot] com[/font], for example.

3. Read the Friendly Manual regarding [man]preg_replace/man. You have the wrong number of arguments, your regexes (should be only one regex, btw) are badly written, and you're missing double-quotes in several places (this problem is what is causing the syntax errors).

Correctly written, your function might look something like this:

<?php
function check_fields()
{
$_POST['brief'] = preg_replace( "/[\d]|[@]|[w]{3}/","",$_POST['brief'] );
}

however, for the reasons I mentioned above, I would strongly recommend against this approach.

Just post rules about what information goes where, and follow up on it to make sure people are posting correctly.

[mark-newbies]

Many thanks it works

[traq]

mark-newbies;10998249 wrote:

Many thanks it works

Yes - but, like I said, it doesn't work well.

Good luck.