redirect on form submit

simone · #1 (**permalink**) 11-23-03, 04:03 AM

Lol ok guys this is my last question because i've just about finished my entire script. All i was after was, when someone enters a new messege and presses submit, i want it to automatically go to the index page (where the originally are) instead of going to an ugly white page with a link back to the index.

Im using the form with the action in a seperate page, so heres the code for that page:

Code:

<HTML>
<BODY>
<?php
include("config.php");
$ip = $_SERVER['REMOTE_ADDR'];
$date = date("D jS M y");
$sql = "INSERT INTO eight (id, subject, name, email, site, aim, yim, msn, msg, date, ip) VALUES 
('', '$_POST[subject]', '$_POST[name]', '$_POST[email]', '$_POST[site]', '$_POST[aim]', '$_POST[yim]', '$_POST[msn]', '$_POST[msg]', '$date', '$ip')";
$result = mysql_query($sql);

?>
</BODY>
</HTML>

So if u guys can tell me what i have to do to get that redirect it'll be great, i've played around with a few different codes and most of them give me errors about the headers or something. Thanks in Advance!

Hackerdragons · #2 (**permalink**) 11-23-03, 05:08 AM

I Don't Really Get What You Mean But I Suppouse You Might Make THis:
<a href="filename.php?from=previous.php">On</a>
On Click İt'd Call Filename.php but add a value
$from ="previous.php";
then you can call previous file($from) by code
header( Location:"$_SERVER['DOCUMENT_ROOT']/$from" );
That's All. Hope I Help'd

I EDIT MY MESSAGE:

since it's on foprm action you must write
<form action="filename.php?from=previous.php" method="post">
or
<form action="filename.php?from=$SCRIPT_NAME" method="post">

mdhall · #3 (**permalink**) 11-23-03, 10:16 AM

Try this

Code:

<HTML>
<BODY>
<?php
include("config.php");
$ip = $_SERVER['REMOTE_ADDR'];
$date = date("D jS M y");
$sql = "INSERT INTO eight (id, subject, name, email, site, aim, yim, msn, msg, date, ip) VALUES 
('', '$_POST[subject]', '$_POST[name]', '$_POST[email]', '$_POST[site]', '$_POST[aim]', '$_POST[yim]', '$_POST[msn]', '$_POST[msg]', '$date', '$ip')";
$result = mysql_query($sql);
 mysql_close();
 echo header ("Location: index.php"); 
?>
</BODY>
</HTML>

Darkness22k · #4 (**permalink**) 11-23-03, 10:43 AM

Or this...but mdhall's is better.

Code:

<HTML>
<BODY>
<?php
include("config.php");
$ip = $_SERVER['REMOTE_ADDR'];
$date = date("D jS M y");
$sql = "INSERT INTO eight (id, subject, name, email, site, aim, yim, msn, msg, date, ip) VALUES 
('', '$_POST[subject]', '$_POST[name]', '$_POST[email]', '$_POST[site]', '$_POST[aim]', '$_POST[yim]', '$_POST[msn]', '$_POST[msg]', '$date', '$ip')";
$result = mysql_query($sql);
 mysql_close();
 include ("index.php");
?>
</BODY>
</HTML>

simone · #5 (**permalink**) 11-24-03, 12:58 AM

Darkness22k to my rescue again! ur the best

mdhall, your one gave me the same error i was getting with every other code i tried.

fyrestrtr · #6 (**permalink**) 11-24-03, 04:54 AM

You should always check to make sure your query executed successfully before redirecting. Never trust the database to work

http://www.php.net/mysql-error

Addict · #7 (**permalink**) 11-24-03, 01:20 PM

A couple changes to Darkness'.....

PHP Code:


			
<HTML>

<BODY>

<?php

include("config.php");

$ip = $_SERVER['REMOTE_ADDR'];

$date = date("D jS M y");

$sql = "INSERT INTO eight (id, subject, name, email, site, aim, yim, msn, msg, date, ip) VALUES 

('', '$_POST[subject]', '$_POST[name]', '$_POST[email]', '$_POST[site]', '$_POST[aim]', '$_POST[yim]', '$_POST[msn]', '$_POST[msg]', '$date', '$ip')";

$result = mysql_query($sql);

//ERROR CHECK

if (!mysql_errno() == '0') {

echo mysql_errno() . "<br>" . mysql_error();

}

else {

include ("index.php");

}

//END ERROR CHECK

mysql_close();

}

?>

</BODY>

</HTML>

Its always a good idea to have an error check. Like was said, don't just expect SQL to work.

fyrestrtr · #8 (**permalink**) 11-24-03, 03:22 PM

Actually, this would be a better way to check for errors :

PHP Code:


			


if (!($result = mysql_query($sql)))

{

    die(mysql_errno()." : ".mysql_error());

} else { header("Location: http://www.domain.com/index.php"); exit(); }

eddyvlad · #9 (**permalink**) 11-24-03, 04:11 PM

As far as I know, the header function need to be executed before the </head> tag. Coz I've tried after the </head> tag and it will generate an error.

Addict · #10 (**permalink**) 11-24-03, 06:10 PM

Quote:

Originally Posted by fyrestrtr

Actually, this would be a better way to check for errors :

PHP Code:


			


if (!($result = mysql_query($sql)))

{

    die(mysql_errno()." : ".mysql_error());

} else { header("Location: http://www.domain.com/index.php"); exit(); }

You can't do the header function because there is already output that has been sent to the browser. So you have to include. Of course there's no reason simaone couldn't alter the code to only send the data after it was successful or unseuccessful.

And the code above will produce the same results as yours. There's not one way better than the other on that.