Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

PHP

Extra credit: Please check my code

I'm doing the extra credit for building a simple php application. I tried to add a checkbox in form, for "terms and conditions". Please check my code. Did I do it right

<?php 

if ($_SERVER["REQUEST_METHOD"] == "POST") {

    $name = $_POST["name"];
    $email = $_POST["email"];
    $message = $_POST["message"];
    $conditions = $_POST["conditions"];
    $email_body = "";
    $email_body = $email_body . "Name: " . $name . "\n";
    $email_body = $email_body . "Email: " . $email . "\n";
    $email_body = $email_body . "Message: " . $message;
    $conditions = $email_body . $name . " agreed the terms/conditions";

    //TODO: Send Email

    header("Location: contact.php?status=thanks");
    exit;

}
?>
<?php


$pageTitle = "Contact Mike";
$section = "contact";
 include('inc/header.php'); 
 ?>     

    <div class="section page">

        <div class="wrapper">

            <h1>Contact</h1>

            <?php if (isset($_GET["status"]) AND $_GET["status"] == "thanks") { ?>
                <p>Thanks for the email! I&rsquo;ll be in touch shortly.</p>
            <?php } else { ?>

                <p>I&rsquo;d love to hear from you! Complete the form to send me an email.</p>

                <form method="post" action="contact.php">

                    <table>
                        <tr>
                            <th>
                                <label for="name">Name</label> 
                            </th>
                            <td>
                                <input type="text" name="name" id="name">
                            </td>
                        </tr>
                        <tr>
                            <th>
                                <label for="email">Email</label> 
                            </th>
                            <td>
                                <input type="text" name="email" id="email">
                            </td>
                        </tr>
                        <tr>
                            <th>
                                <label for="message">Message</label> 
                            </th>
                            <td>
                                <textarea name="message" id="message"></textarea>
                            </td>
                        </tr>
                        <tr>
                            <th>
                                <label for="conditions">Terms and Conditions</label> 
                            </th>
                            <td>
                                I agree to the terms and conditions
                                <input type="checkbox" name="conditions">
                            </td>
                        </tr>

                    </table>
                    <input type="submit" value="Send">

                </form>

            </div>

        </div>

    <?php } ?>

<?php include('inc/footer.php'); ?>

1 Answer

The simple answer - does it work ? :-p

if you get the desired functionality (which I guess is the user has to select the t+c box before they can continue?), then you've done it right!

I see you haven't validated the box check yet (if that's what you're hoping to do). You'll come up to validations soon, if not already :-) but you could try adding something like that.

I couldn't really figure out how to add validation.