Not redirect only show message how?

PHP MAIL

<?php
/*
This first bit sets the email address that you want the form to be submitted to.
You will need to change this value to a valid email address that you can access.
*/
$webmaster_email = "info.example@gmail.com";

/*
This bit sets the URLs of the supporting pages.
If you change the names of any of the pages, you will need to change the values here.
*/
$feedback_page = "feedback_form.html";
$error_page = "error_message.html";
$thankyou_page = "thank_you.html";

/*
This next bit loads the form field data into variables.
If you add a form field, you will need to add it here.
*/
$email_address = $_REQUEST['email_address'] ;
$first_name = $_REQUEST['first_name'] ;
$msg = 
"First Name: " . $first_name . "\r\n" . 
"Email: " . $email_address . "\r\n"  ;

/*
The following function checks for email injection.
Specifically, it checks for carriage returns - typically used by spammers to inject a CC list.
*/
function isInjected($str) {
	$injections = array('(\n+)',
	'(\r+)',
	'(\t+)',
	'(%0A+)',
	'(%0D+)',
	'(%08+)',
	'(%09+)'
	);
	$inject = join('|', $injections);
	$inject = "/$inject/i";
	if(preg_match($inject,$str)) {
		return true;
	}
	else {
		return false;
	}
}

// If the user tries to access this script directly, redirect them to the feedback form,
if (!isset($_REQUEST['email_address'])) {
header( "Location: $feedback_page" );
}

// If the form fields are empty, redirect to the error page.
elseif (empty($first_name) || empty($email_address)) {
header( "Location: $error_page" );
}

/* 
If email injection is detected, redirect to the error page.
If you add a form field, you should add it here.
*/
elseif ( isInjected($email_address) || isInjected($first_name) ) {
header( "Location: $error_page" );
}

// If we passed all previous tests, send the email then redirect to the thank you page.
else {

	mail( "$webmaster_email", "Feedback Form Results", $msg );

	header( "Location: $thankyou_page" );
}
?>

How I make this not redirect but only show error else success message?

What have you tried?

The code seems quite self explanatory with all the comments explaining clearly what each section does.
I think you could easily pick out the parts that do the redirects and change that action for what you want it to do, such as set the content you want to display instead of a redirect.

3 Likes

I tried to entered this code if(isset($email)){ echo 'Success! Thanks for submitting'; } instead of [quote=ā€œshivkumar, post:1, topic:289715ā€]
header( ā€œLocation: $thankyou_pageā€ );
}
[/quote]

But not ā€¦ Successā€¦

And what is the value of $email?

and why do you check the variable for existence, instead of the actual content, or even the success of sending the mail?

No editing ā€¦
Sameā€¦

Sirā€¦Could you please send me a proper php?

What does that mean?

You have already been told numerous times, by myself and others, that the forums are not a coding service.

2 Likes

This topic was automatically closed 91 days after the last reply. New replies are no longer allowed.