[RESOLVED] If statement issues

[thewomb]

hey guys. cant see what im doing wrong here.

see comment // heres is where the issue begins in the code.

all works well till it gets to the eregi to validate the email then it just stops. it doesnt seem to want to pass that ielseif onto the "elseif (!$_POST['phone'])" bit

thanx in advance..

even if anyone has any ideas as to tidyin up the code let me know.

TW

<?
if(!$_POST ['name'])
{
$name_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- Please enter your name</a>';
//echo $name_error;
} 
elseif (!$_POST['subject'])
{
$subject_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- Please enter a subject</a>';
//echo $subject_error;
} 
elseif (!$_POST['body'])
{
$body_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- Please enter a message</a>';
//echo $body_error;
} 
elseif (!$_POST['senderx'])
{ 
$email_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- Please enter an email address</a>';
//echo $email_error;
}
// heres is where the issue begins
elseif (!eregi("^[a-z0-9._-]+@[a-z0-9._-]+.[a-z]{2,4}$", $_POST['senderx']))    { 
$email_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- your email is not valid</a>';
echo $email_error;
}   

elseif (eregi("^[a-z0-9._-]+@[a-z0-9._-]+.[a-z]{2,4}$", $_POST['senderx']))
{
$email_error = 'email is validated';
echo $email_error;
} 
elseif (!$_POST['phone'])
{
$phone_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- Please enter a phone number</a>';
//echo $phone_error;
} 
elseif (!$_POST['mobile'])
{
$mobile_error = '<a href="javascript:history.go(-1)"  onMouseOver="self.status=document.referrer;return true">&lt;&lt;-- Please enter a mobile number</a>';
//echo $mobile_error;
} 
elseif ($word_ok!==false)
{
if($word_ok=="yes")
{
$_SESSION['name'] = $_POST ['name'];	
	$_SESSION['senderx'] = $_POST ['senderx'];	
		$_SESSION['phone'] = $_POST ['phone'];	
			$_SESSION['mobile'] = $_POST ['mobile'];	
				$_SESSION['subject'] = $_POST ['subject'];	
					$_SESSION['body'] = $_POST ['body'];
			//		echo var_dump($_SESSION);
$host  = $_SERVER['HTTP_HOST'];
$uri  = rtrim(dirname($_SERVER['PHP_SELF']), '/\\');
$extra = 'contact_done_new.php';
header("Location: http://$host$uri/$extra");
exit;
	} 
}
?>

[NogDog]

Not sure if this is the situation, but once the original if any of the following elseif's evaluates as true any remaining elseif or else will be skipped.

[zabmilenko]

He has dollar signs in his double quoted strings, particularly the regex ones where the dollar signs are supposed to be end-of-string anchors.

elseif (!eregi("^[a-z0-9._-]+@[a-z0-9._-]+.[a-z]{2,4}$", $_POST['senderx']))

Should be:

elseif (!eregi('^[a-z0-9._-]+@[a-z0-9._-]+.[a-z]{2,4}$', $_POST['senderx']))

et cetera

Edit: Also NogDog is correct. Watch your fallthrus :-)

[thewomb]

thanx NogDog.

u were right, simple when you know how he he

elseif (eregi('^[a-z0-9._-]+@[a-z0-9._-]+.[a-z]{2,4}$', $_POST['senderx']))
{
$email_error = 'email is validated';
echo $email_error;
} 

if loop died when validation was true.

cheers

TW