Whats wrong with this code. It says error is on line 88

[php]<?php
$recipientEmail = "[email protected]"; // enter recipient email address. Don’t delete the double quotation marks
$emailSubject = “Computer service request”;
$successMessage = “Thanks for your message! We’ll be in contact shortly.”;
$error = “”; //leave blank
if(isset($_POST[‘submit’]))
{
$name = $_POST[‘name’];
$email = $_POST[‘email’];
$cellphone = $_POST[‘cellphone’];
$homephone = $_POST[‘homephone’];
$address = $_POST[‘address’];
$comments = $_POST[‘comments’];
$result = checkForSpam(Array($name));
if ((trim($name=="")) || (trim($email=="")) ||((trim($cellphone=="")) && (trim($homephone==""))))
{
$error=“Name, email address, and either cellphone or homephone number are required fields.”;
}
else if($result)
{
$error = ‘You have entered invalid characters.’;
}
else if(!validEmail($email))
{
$error = ‘*Your email address is not valid’;
$email = ‘’;
}
if($error == ‘’)
{
$priority = 3;
$php_version = phpversion();
$headers = “From: $name <$email>\r\n”;
$headers .= “MIME-Version: 1.0\r\n”;
$headers .= “Content-type: text/plain; charset=iso-8859-1\r\n”;
$headers .= “X-Priority: $priority\r\n”;
$headers .= “X-Mailer: PHP $php_version\r\n”;
$subject = $emailSubject;
$to_email = $recipientEmail;
$message = “







Name ”.$name."
E-Mail ".$email."
Cell phone No ".$cellphone."
Home phone No ".$homephone."
Address ".$address."
Details ".$comments."
" ;
mail($to_email, $subject, $message, $headers);
echo $successMessage;
}

}
if(!isset($_POST[‘submit’]) || $error != ‘’)
{
echo $error;
?>

Name: Please enter any information below related to the issue
 
Email:  
Phone (Cell): <?php echo $comments; ?>
Phone: (home):
Address:
<?php //utility functions function validEmail($email) { return(preg_match("/^[-_.[:alnum:]]+@((([[:alnum:]]|[[:alnum:]][[:alnum:]-]*[[:alnum:]])\.)+(ad|ae|aero|af|ag|ai|al|am|an|ao|aq|ar|arpa|as|at|au|aw|az|ba|bb|bd|be|bf|bg|bh|bi|biz|bj|bm|bn|bo|br|bs|bt|bv|bw|by|bz|ca|cc|cd|cf|cg|ch|ci|ck|cl|cm|cn|co|com|coop|cr|cs|cu|cv|cx|cy|cz|de|dj|dk|dm|do|dz|ec|edu|ee|eg|eh|er|es|et|eu|fi|fj|fk|fm|fo|fr|ga|gb|gd|ge|gf|gh|gi|gl|gm|gn|gov|gp|gq|gr|gs|gt|gu|gw|gy|hk|hm|hn|hr|ht|hu|id|ie|il|in|info|int|io|iq|ir|is|it|jm|jo|jp|ke|kg|kh|ki|km|kn|kp|kr|kw|ky|kz|la|lb|lc|li|lk|lr|ls|lt|lu|lv|ly|ma|mc|md|mg|mh|mil|mk|ml|mm|mn|mo|mp|mq|mr|ms|mt|mu|museum|mv|mw|mx|my|mz|na|name|nc|ne|net|nf|ng|ni|nl|no|np|nr|nt|nu|nz|om|org|pa|pe|pf|pg|ph|pk|pl|pm|pn|pr|pro|ps|pt|pw|py|qa|re|ro|ru|rw|sa|sb|sc|sd|se|sg|sh|si|sj|sk|sl|sm|sn|so|sr|st|su|sv|sy|sz|tc|td|tf|tg|th|tj|tk|tm|tn|to|tp|tr|tt|tv|tw|tz|ua|ug|uk|um|us|uy|uz|va|vc|ve|vg|vi|vn|vu|wf|ws|ye|yt|yu|za|zm|zw)$|(([0-9][0-9]?|[0-1][0-9][0-9]|[2][0-4][0-9]|[2][5][0-5])\.){3}([0-9][0-9]?|[0-1][0-9][0-9]|[2][0-4][0-9]|[2][5][0-5]))$/i" ,$email)); } function checkForSpam($fields){ $spam = false; for ($i=0;$i[/php]

Can you tell us what line is line 88…

Even though I put it in a code window, I don’t think it matches up.

What exactly does the error say?

Hello,

On line 53 you opened a { and didn’t close it. The parser says it’s on line 88 since it’s the last line in the script and the PHP parser “assumed” that all brackets ({, }) after the one you used on line 53 were nested inside that one. Close that bracket and your code should work.

Cheers!

OPZMaster,
Thanks. I guess after looking at the code for so many hours straight I kept missing that

Opzmaster:

Wow,so it was just the following snippet, I guess I forgot the } after the echo $error;
[php]
if(!isset($_POST[‘submit’]) || $error != ‘’)
{
echo $error;
[/php]

Sponsor our Newsletter | Privacy Policy | Terms of Service