help with preg_match

• Mar 7, 2008

• #2

jpadie

Technical User

Nov 24, 2003

10,094

FR

are you trying to say that the field must only contain a comma, dash, full stop, single quote, colon and alphabet (and space)

if so, i'm not sure why your escaping so much stuff. i'd only look to escape the dash. i think that's probably the issue.

Code:

$pattern = "/^[a-z \-,.:']+$/i";

you should also test for 0 rather than false (to be precise). preg_match returns an integer corresponding to the number of matches.

Code:

if ( preg_match($pattern, $subject) === 0 ){
 echo 'invalid form field';
}

 

Upvote 0 Downvote

• Mar 7, 2008

• Thread starter
• #3

mufka

ISP

Dec 18, 2000

587

US

Code:

$pattern = "/^[a-z \-,.:']+$/i";

doesn't work either. It still won't allow an ' in the field.

 

Upvote 0 Downvote

• Mar 7, 2008

• #4

jpadie

Technical User

Nov 24, 2003

10,094

FR

i can't help you then. on my machines this code

Code:

<?php
$pattern = "/^[a-z \-,.:']+$/i";
$subject[] = "I am a test string";
$subject[] = "I am a John O'Connor";
$subject[] = "I am a test with illegal [characters]";
foreach($subject as $s){
	if(preg_match($pattern, $s) === 0 ){
		echo "string not matched";
	} else {
		echo "string matched";
	}
	echo "<br/>";
}
?>

results in this

Code:

string matched
string matched
string not matched

which is as expected.

 

Upvote 0 Downvote

• Mar 7, 2008

• Thread starter
• #5

mufka

ISP

Dec 18, 2000

587

US

Using $subject instead of $_POST["subject"] fixed it. I also changed to testing for 0 while I was at it.

 

Upvote 0 Downvote

• Mar 7, 2008

• #6

jpadie

Technical User

Nov 24, 2003

10,094

FR

glad to hear it.

 

Upvote 0 Downvote