Validation query

healy787 · February 24, 2009

Hi guys. Im getting myself in a pickle. Here is what i have:

<?

$name = $_REQUEST['name'] ;

$surname = $_REQUEST['surname'] ;

$email = $_REQUEST['email'] ;

$message = $_REQUEST['message'] ;

$subject1 = $_POST['subject1'] ;

$mailing = $_POST['mailing'] ;

$telephone = $_REQUEST['telephone'] ;

$to = "webmaster@letsgohost.co.uk";

$subject = "Message From Letsgohost Website";

$MsgHeader = "From: Lestgohost Website <support@letgohost.co.uk>\n";

$MsgHeader .= "Bcc: <webmaster@rydon-inn.co.uk> r\n";

$MsgHeader .= "MIME-Version: 1.0\n";

$MsgHeader .= "Content-type: text/html; charset=iso-8859-1\n";

$MsgBody = "

<html>

<head>

<title>Letsgohost</title>

</head>

<body>

<tr><td>Surname : $surname</td></tr>

<tr><td>Email : $email</td></tr>

<tr><td>Message subject : $subject1</td></tr>

<tr><td>Message : $message</td></tr>

<tr><td>Mailing : $mailing</td></tr>

<tr><td>Telephone : $telephone</td></tr>

</table>

</body>

</html>";

function checkEmail($email)

{

// checks proper syntax

if( !preg_match( "/^([a-zA-Z0-9])+([a-zA-Z0-9\._-])*@([a-zA-Z0-9_-])+([a-zA-Z0-9\._-]+)+$/", $email))

{

return false;

}

// gets domain name

list($username,$domain)=split('@',$email);

// checks for if MX records in the DNS

$mxhosts = array();

if(!getmxrr($domain, $mxhosts))

{

// no mx records, ok to check domain

if (!fsockopen($domain,25,$errno,$errstr,30))

{

return false;

}

else

{

return true;

}

else

{

// mx records found

foreach ($mxhosts as $host)

{

if (fsockopen($host,25,$errno,$errstr,30))

{

return true;

}

return false;

}

if (!isset($_REQUEST['email'])) {

header( "Location: http://www.letsgohost.co.uk/contact.html" );

}

elseif (empty($email) || empty($message) || empty($name)) {

header( "Expires: Mon, 20 Dec 1998 01:00:00 GMT" );

header( "Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT" );

header( "Cache-Control: no-cache, must-revalidate" );

header( "Pragma: no-cache" );

?>

<html>

<head><title>Error</title></head>

<body>

<h1>Error</h1>

Oops, it appears you haven't filled in the form correctly. Please ensure ALL fields have been filled in correctly. Please use the BACK

button in your browser and try again.

</body>

</html>

<?php

}

else {

mail($to, $subject, $MsgBody, $MsgHeader);

header("Location: http://www.letsgohost.co.uk/thanks.html");

}

if(!is_numeric($_POST['telephone'])){

echo("Telephone must be numeric (numbers only)");

}

?>

The form posts perfectly. The problem i have is when i add the function checkEmail. This seems to overide the other validation and allows me to leave the email box blank? When i remove the function the script works fine and i have to put a value in the email box. Obviously the email validator isnt working right.

Any suggestions?

Thanks people.

Sign In

Validation query

Recommended Posts

healy787

Link to comment

Share on other sites

Join the conversation

Browse

Activity

Important Information