How to add code to script?

akacj

I have a logout.php script and I want to put this code into it that asks the user if they are sure they want to logout.

echo "<center>Are you sure you want to logout?</center><br />";
echo "<center><a href=logout.php?logmeout>Yes</a> | <a href=javascript:history.back()>No</a>";

Heres my logout.php

<?php # Script 16.9 - logout.php
// This is the logout page for the site.

require_once ('includes/config.inc.php'); 
$page_title = 'Logout';
include ('includes/header.html');

// If no first_name session variable exists, redirect the user:
if (!isset($_SESSION['first_name'])) {

$url = BASE_URL . 'index.php'; // Define the URL.
ob_end_clean(); // Delete the buffer.
header("Location: $url");
exit(); // Quit the script.

} else { // Log out the user.

$_SESSION = array(); // Destroy the variables.
session_destroy(); // Destroy the session itself.
setcookie (session_name(), '', time()-300); // Destroy the cookie.

}

// Print a customized message:
echo '<h3>You are now logged out.</h3>';

include ('includes/footer.html');
?>

leatherback


else { // Log out the user.

  if(isset($_GET['logmeout']))
    {
    $_SESSION = array(); // Destroy the variables.
    session_destroy(); // Destroy the session itself.
    setcookie (session_name(), '', time()-300); // Destroy the cookie.
    }
  else
    {
    echo "<center>Are you sure you want to logout?<br />
             <a href=logout.php?logmeout>Yes</a> | <a href=javascript:history.back()>No</a>"; 
    }
}

akacj

Thanks for helping me. 🙂

How do I get it to echo "You have been logged out" after the user clicks on Yes?

leatherback

You need to rework your logic a little. I know you can do it. You only have to move these two lines to the right location in your script:

// Print a customized message:
echo '<h3>You are now logged out.</h3>';

akacj

I tried .

It says it on the page before i even hit yes or no.

otherwise any other modifcation i try to make I keep getting HTTP 500 Internal Server error.

Sorry lol im such a php newbie.

leatherback

The main question is: Do you understand what the code you have does? If not, it is now about time you did try and figure that out.

Sorry for being a pain; It is better for all of us if you tried to understand the code. the modification is very simple. But if you cannot make it you do not understand the code you have now. Try to follow the logice there. Best way to do this, is by adding comments at crucial points which you already have in there (Hint: If / Else are crucial points). From there you should be able to understand what is happening, and therefor, where the statements need to go.