[jonnekke]

Hi there,

I'm kind of new here. Also I'm kind of new to PHP. I'm still learning.
I got a problem with a form I'm using.

If I filled it in and press the submit button it goed to a php page and stays there instead of going to the Thanx page.

I attached my html and php file.

Please help me out.

Greetz,

j

[xtremenw]

With out digging to deep into the script, the fact that it is going to the thnx.html page is a good thing. It probably did everything it was supposed to. On the bottom of the file is:

Code:

	if (!mail($MailToAddress, $MailSubject, $mailMessage,$mailHeader)) \{ echo "Error sending e-mail!";\}\
	else \{ header("Location: ".$redirectURL); \}	\

If the script can not send the mail, echo "error...", else header("Location: ...");. The header() function is an internal PHP function. This funcation is mainly, from what i have seen, used for browser redirection.

[Richard]

Originally Posted by xtremenw

With out digging to deep into the script, the fact that it is going to the thnx.html page is a good thing. It probably did everything it was supposed to. On the bottom of the file is:

Code:

	if (!mail($MailToAddress, $MailSubject, $mailMessage,$mailHeader)) \{ echo "Error sending e-mail!";\}\
	else \{ header("Location: ".$redirectURL); \}	\

If the script can not send the mail, echo "error...", else header("Location: ...");. The header() function is an internal PHP function. This funcation is mainly, from what i have seen, used for browser redirection.

The page isn't getting redirected to the thank you page, and I think its because of the code you've highlighted. Try removing the ! from the IF statement, then if the e-mailing is successful it will redirect to the $redirectURL (I'm assuming this is the thank you page).

Also, you're correct about the header() function, it is mainly used for brower redirection, although its basically able to send any raw HTTP header parameters, as pointed out here; PHP.net - header().

[Keith]

Originally Posted by Richard

The page isn't getting redirected to the thank you page, and I think its because of the code you've highlighted. Try removing the ! from the IF statement, then if the e-mailing is successful it will redirect to the $redirectURL

Huh?

PHP Code:

if (!mail($MailToAddress, $MailSubject, $mailMessage,$mailHeader)) { echo "Error sending e-mail!"; }

else { header("Location: ".$redirectURL); } 

As it is written, if it fails it displays an error, else it spits out the thank you page.

If you remove the !, it will display an error when the mail() is successful, and the thank you page if it fails.

[jordie]

Hi,

Firstly, where are you learning php from? You are using old methods... such as:

PHP Code:

while(list($key, $val) = each($_GET)) { 

can now be replaced with

PHP Code:

foreach($_GET as $key=>$val) { 

also, try what richard said, swap the condition around AND also put a die(); after the header redirect cause the script still will run through after the header instruction:

PHP Code:

    if (mail($MailToAddress, $MailSubject, $mailMessage,$mailHeader)) { 

        header("Location: ".$redirectURL);

                die();

    }

    else { 

         echo "Error sending e-mail!";

    } 

If it still isn't working, put a little debug test after the conditions closing brace like

PHP Code:

    if (mail($MailToAddress, $MailSubject, $mailMessage,$mailHeader)) { 

        header("Location: ".$redirectURL);

                die();

    }

    else { 

         echo "Error sending e-mail!";

    }

echo "Testing 1 2 3..."; 

to see if the script is making it that far

[Richard]

Originally Posted by Keith

Huh?

PHP Code:

if (!mail($MailToAddress, $MailSubject, $mailMessage,$mailHeader)) { echo "Error sending e-mail!"; }

else { header("Location: ".$redirectURL); } 

As it is written, if it fails it displays an error, else it spits out the thank you page.

If you remove the !, it will display an error when the mail() is successful, and the thank you page if it fails.

lol, completely ignore my post, I don't know what I was thinking to be honest. Maybe I shouldn't post so late, lol

[jonnekke]

I tried all of the given solutions, either none of them worked with me

- I removed the !
- changed the code fore what was given

but it didn't worked out :s


is there another solution??....