I can't get my script to work

hematrix

Hi there. I have been developing a site to deal with the sales of CD ROMs over the net and I have hit a bit of a brick wall. I accept several payment methods which can be selected on the order screen. I have now developed links to individual thankyou screens, each displaying specific payment instructions depending on the method selected. I have written a script using PHP to cope with this but I can't get it to work properly. It sends me the information with no problems at all but when it comes to slect the right thankyou screen, all I get is a blank screen. Here is the coding I have written.

<?
$recipient = "sales@h-ematrix.co.uk";
$subject = "Order";
$message = "Surname - $surname Forename - $forname Address - $address Email - $email Referee - $referee CD ROM - $cdrom List - $list Payment - $payment";
$from - $_POST['email'];

$surname=$_POST['surname'];
$forename=$_POST['forname'];
$address=$_POST['address'];
$email=$_POST['email'];
$referee=$_POST['referee'];
$cdrom=$_POST['cdrom'];
$list=$_POST['list'];
$payment=$_POST['payment'];

if ($payment=="paypal"){
header("Location:[url]http://www.h-ematrix.co.uk/thankyoupp.htm[/url]");
}

elseif ($payment=="nochex"){
header("Location:[url]http://www.h-ematrix.co.uk/thankyounc.htm[/url]");
}

elseif ($payment=="po or bankers draft"){
header("Location:[url]http://www.h-ematrix.co.uk/thankyoubdpo.htm[/url]");
}

elseif ($payment=="cheque"){
header("Location:[url]http://www.h-ematrix.co.uk/thankyoupc.htm[/url]");
}

mail($recipient,$subject,$message,$from);

?>

Can anyone tell me what is wrong?

Thanks

melody

You need to remove the <html> tags from the header function. i.e. it should look like this:

if ($payment=="paypal"){ 
header("Location:[url]http://www.h-ematrix.co.uk/thankyoupp.htm[/url]"); 
}

Obviously, this will stop it popping up in a new window though. Hope this helps.

Melody

TTT

if there is error "headers already sent", use javascript

</script>

Weedpacket

Originally posted by melody
Hi

You need to remove the <html> tags from the header function. i.e. it should look like this:

That's a vBulletin bug putting those in.

If the email is going out without trouble then your problem is that $payment doesn't contain what you think it does - if it's not one of those four strings then there won't be any redirect. As a bit of a test, put exit(); after each header() call, and see if the emails still happen. If they stop, move the mail() all up to before those tests (so that the email is sent before the headers) and see if it restarts.

And put at the very end of the script a bit of text to see if that gets displayed.

jayant

<?php 
$recipient = "sales@h-ematrix.co.uk"; 
$subject = "Order"; 
$message = "Surname - $surname Forename - $forname Address - $address Email - $email Referee - $referee CD ROM - $cdrom List - $list Payment - $payment"; 
$from - $_POST['email']; 

$surname=$_POST['surname']; 
$forename=$_POST['forname']; 
$address=$_POST['address']; 
$email=$_POST['email']; 
$referee=$_POST['referee']; 
$cdrom=$_POST['cdrom']; 
$list=$_POST['list']; 
$payment=$_POST['payment']; 

mail($recipient,$subject,$message,$from); 

if ($payment=="paypal"){ 
header("Location:http://www.h-ematrix.co.uk/thankyoupp.htm"); 
} 

elseif ($payment=="nochex"){ 
header("Location:http://www.h-ematrix.co.uk/thankyounc.htm"); 
} 

elseif ($payment=="po or bankers draft"){ 
header("Location:http://www.h-ematrix.co.uk/thankyoubdpo.htm"); 
} 

elseif ($payment=="cheque"){ 
header("Location:http://www.h-ematrix.co.uk/thankyoupc.htm"); 
} 

?>