Multipurpose page parse error

Deborah13

I want to have a multipurpose page that allows the user a second chance before they peminantly delete something. I think this should work (Although I am completely new to all this). However I keep gettin the following error:

Parse error: syntax error, unexpected $end in C:\wamp\www\deleteeventt.php on line 61

which means I am missing a { or a } or something along those lines, but I just can't spot it!

Maybe it's completely wrong altogether?
Here it is anyway:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "">
<html xmlns="">
<head>
<title>The Internet Joke Database</title>
<meta http-equiv="content-type" content="text/html; charset=iso-8859-1" />
</head>
<body>
<?php 

if (isset($_GET['delete'])): // If the user wants to delete an event
{
// Delete the event event from the events table
// and also delete that events from the attendance table.
$EventCode = $_GET['EventCode'];
$ok1 = @mysql_query("DELETE FROM events WHERE EventCode='$EventCode'");
$ok2 = @mysql_query("DELETE FROM attendance WHERE EventCode='$EventCode'");
if ($ok1 and $ok2) 
{
echo '<p>Event deleted successfully!</p>';
} 
else 
{
echo '<p>Error deleting Event from database!<br />'.
'Error: ' . mysql_error() . '</p>';
}
echo '<p><a href="viewallevents.php">Return to view all events</a></p>';
}

else: // Default page display
{ 

//Connect to the database server
$dbcnx = @mysql_connect('localhost', 'root', 'pass');
if (!$dbcnx) 
{
exit('<p>Unable to connect to the ' .
'database server at this time.</p>');
}

//select the prototype database
if (!@mysql_select_db('prototype')) 
{
exit('<p>Unable to locate the ARD ' .
'database at this time.</p>');
} 

echo '<p> Are you sure you would like to delete this event? </p>';

// When clicked, this link will delete the event
echo '<p><a href="' . $_SERVER['PHP_SELF'] .
'?delete=1">Yes</a></p>';

//If the user changes their mind it will bring them bact to vie all events
echo '<p><a href="viewallevents.php">Cancel</a></p>';

}
?>
</body>
</html>

Please please please tell me what I am doing wrong!

bpat1434

If you're using if() and else with ":" after them, then you don't need the "{}". If you don't use ":", then you need the "{}".

if($something == $somethingelse):
    // Do something...
elseif($somethingelse == $anotherthing):
    // Do something else....
else:
    // Do a final thing...
elseif;

Or, use the curly-braces way:

if($something == $somethingelse)
{
    // Do Something...
}
elseif($somethingelse == $anotherthing)
{
    // Do Something else...
}
else
{
    // Do a final thing...
}

Get it? You are trying to use both ways in your code above, fix that and you should be golden 😉

scrupul0us

you should also add a limit to your queries in case something goes wrong you wont wipe out an entire table

$ok1 = @mysql_query("DELETE FROM events WHERE EventCode='$EventCode Limit' Limit 1");
$ok2 = @mysql_query("DELETE FROM attendance WHERE EventCode='$EventCode' Limit 1");

should do it

you also want to process $_GET['EventCode'] at the very least with mysql_real_escape_string to prevent against bad things happening