Trying to do a re-direct

l00ph0le

Ok, So I have a page that I want to display under certain circumstances.

In a nutshell, I have a mysql table named status. In that table a have a field named status. If the word suspended is in that field, I want to re-direct to another page entirely.

I am quite new to php, and I know I am missing something here, just not sure what.
Basically here is my outline:

--Included files--
Query status from status table
if status = suspended,
re-direct to new page stop executing code on this page.

<?
//headers and DB
	require_once('db.php');
	include('header.php');

$query = "SELECT status from onp_status";
$result = mysql_query($query);
if ($result == 'suspended') {
	header("Location: suspend.php");
	exit;
	}
mysql_free_result($result);

?> 
<head>

rowanparker

You could use ereg() instead as that will find it in the string rather than it being the string (look on php.net if you don't understand).

laserlight

You are probably looking to use mysql_result() as you cannot directly compare a resource result with a string. For example:

<?php
//headers and DB
require_once('db.php');
include('header.php');

$query = "SELECT status from onp_status";
$result = mysql_query($query);
if (mysql_result($result, 0) == 'suspended') {
    $abs_path = $_SERVER['HTTP_HOST'] . dirname($_SERVER['PHP_SELF']);
    header('Location: http://' . $abs_path . '/suspend.php');
    exit;
}
mysql_free_result($result);

?>

Note that the location header should be an absolute URI.

l00ph0le

LaserLight

What you typed works only if I place the include header.php at the bottom which is fine.

However, with out hammering you on questions, can you break down what each part of the changes you are making are doing?

it would help me understand a little bit better.

laserlight

The main change I made was to use mysql_result(). The secondary change was to turn the URI in your location header into an absolute URI.

If header.php contains say, some HTML, then you have to place it after the location header is sent, or use a buffered output workaround (which I do not recommend).

rowanparker

Well done laserlight, I missed that bit 🙂