PHP pass parameters from html form

Question

I come from C# background and im new to php, I have an html form submits it's values to the following php file and for some reason the values are not set, what am I doing wrong?

<?php
include('Mail.php');
$recipients = "[email protected]";
$name = $_POST["txtName"] ;
$from = $_POST["txtEmail"] ;
$subject = $_POST["txtAppName"] ;
$comments = $_POST["txtComment"] ;

$headers = array (
    'From' => $from,
    'To' => $recipients,
    'Subject' => $subject,
);
$body = "Name: ".$name."\r\n"."Comments: "."\r\n".$comments;
$mail_object =& Mail::factory('smtp',
    array(
        'host' => 'prwebmail',
        'auth' => true,
        'username' => 'username',
        'password' => 'pass', # As set on your project's config page
        'debug' => true, # uncomment to enable debugging
    ));
$mail_object->send($recipients, $headers, $body);
?>

And here is the html form:

<form action="sendfeedback.php" method="post">
            <div><input placeholder="Name" id="txtName" type="text" /></div>
            <div><input placeholder="Email (Optional)" id="txtEmail" type="text" /></div>
            <div><input placeholder="Application Name (Type in the application name you're using)" id="txtAppName" type="text" /></div>
            <div style="height:4px"></div>
            <div><textarea placeholder="Comments..." id="txtComment"></textarea></div>
            <div style="width:407px; text-align:right;"><input id="submitComment" type="submit" value="Send" /></div>
        </form>

And make sure the form fields have names, specifically the names you are using in your PHP script. — putvande
– putvande, Commented Feb 24, 2014 at 19:12
@putvande has it. Of course, lots of people followed with attempting reputation farming by posting the same thing four times. — user2136840
– user2136840, Commented Feb 24, 2014 at 19:14
btw I've seen it done through $_REQUEST or $_GET, what is the difference if you don't mind me asking? — Johnny
– Johnny, Commented Feb 24, 2014 at 19:19
$_GET refers to the querystring in the URL. So if your form uses GET instead of POST, the url will look like sendfeedback.php?txtName=ABC&txtEmail=XYZ..., and the $_GET variable will contain those parameters. $_REQUEST combines $_POST, $_GET, and $_COOKIE by default. More here: stackoverflow.com/a/1924958/2136840 — user2136840
– user2136840, Commented Feb 24, 2014 at 19:52

Krish R · Accepted Answer · 2014-02-24 19:13:04Z

2

There is no name attribute in your input fields, You have used id attributes, but you should use name attribute for posting the form values.

<form action="sendfeedback.php" method="post">
        <div><input name="txtName" placeholder="Name" id="txtName" type="text" /></div>
                    ^^^^^^^^^^^^^^
        <div><input name="txtEmail"  placeholder="Email (Optional)" id="txtEmail" type="text" /></div>
        <div><input  name="txtAppName"  placeholder="Application Name (Type in the application name you're using)" id="txtAppName" type="text" /></div>
        <div style="height:4px"></div>
        <div><textarea name="txtComment" placeholder="Comments..." id="txtComment"></textarea></div>
        <div style="width:407px; text-align:right;"><input id="submitComment" type="submit" value="Send" /></div>
    </form>

answered Feb 24, 2014 at 19:13

Krish R

22.7k7 gold badges54 silver badges60 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

António Almeida · Accepted Answer · 2014-02-24 19:12:39Z

1

Your form inputs should have the attribute name, like this:

<form action="sendfeedback.php" method="post">
    <div><input placeholder="Name" id="txtName" name='txtName' type="text" /></div>
    <div><input placeholder="Email (Optional)" id="txtEmail" name="txtEmail" type="text" /></div>
    <div><input placeholder="Application Name (Type in the application name you're using)" id="txtAppName" name="txtAppName" type="text" /></div>
    <div style="height:4px"></div>
    <div><textarea placeholder="Comments..." id="txtComment" name='txtComment'></textarea></div>
    <div style="width:407px; text-align:right;"><input id="submitComment" type="submit" value="Send" /></div>
</form>

answered Feb 24, 2014 at 19:12

António Almeida

10.2k8 gold badges62 silver badges72 bronze badges

Comments

sergio · Accepted Answer · 2014-02-24 19:12:53Z

The reason that in your did not set attribute name for your inputs, so chanhe it to

<form action="sendfeedback.php" method="post">
            <div><input name = 'txtName' placeholder="Name" id="txtName" type="text" /></div>
            <div><input name = 'txtEmail'  placeholder="Email (Optional)" id="txtEmail" type="text" /></div>
            <div><input name ='txtAppName' placeholder="Application Name (Type in the application name you're using)" id="txtAppName" type="text" /></div>
            <div style="height:4px"></div>
            <div><textarea name ='txtComment' placeholder="Comments..." id="txtComment"></textarea></div>
            <div style="width:407px; text-align:right;"><input id="submitComment" type="submit" value="Send" /></div>
        </form>

John Conde · Accepted Answer · 2014-02-24 19:13:02Z

1

You're missing the name attributes of your inputs:

    <form action="sendfeedback.php" method="post">
        <div><input placeholder="Name" name="txtName" id="txtName" type="text" /></div>
        <div><input placeholder="Email (Optional)" name="txtEmail" id="txtEmail" type="text" /></div>
        <div><input placeholder="Application Name (Type in the application name you're using)" name="txtAppName" id="txtAppName" type="text" /></div>
        <div style="height:4px"></div>
        <div><textarea placeholder="Comments..." name="txtComment"  id="txtComment"></textarea></div>
        <div style="width:407px; text-align:right;"><input id="submitComment" type="submit" value="Send" /></div>
    </form>

answered Feb 24, 2014 at 19:13

John Conde

220k100 gold badges464 silver badges504 bronze badges

Comments

prateekkathal · Accepted Answer · 2014-02-24 21:04:48Z

1

Add the name attribute to each of the input tags as shown in the above answers.

Also, I would appreciate if u use the form this way

if(isset($_POST['txtName'] || isset($_POST['txtPass'])) {
    $txtName = $_POST['txtName'];
    $txtPass = $_POST['txtPass'];
    //Everything else here instead require('mail.php');
}

I hope it solves ur issues and helps u in debugging and securing ur codes as well.

answered Feb 24, 2014 at 21:04

prateekkathal

3,5821 gold badge23 silver badges40 bronze badges

Collectives™ on Stack Overflow

PHP pass parameters from html form

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related