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

Sarah Rose
Sarah Rose
6,918 Points
Posted by Sarah Rose
Sarah Rose
6,918 Points

I think I've missed something...

I still get the: "Please fill in the required fields: Name, Email, Category and Title" even when they are filled. :/

Did I miss something?

<?php 

if ($_SERVER["REQUEST_METHOD"] == "POST") {
$name = trim(filter_input(INPUT_POST,"name",FILTER_SANITIZE_STRING));
$email = trim(filter_input(INPUT_POST,"email",FILTER_SANITIZE_EMAIL));
$category = trim(filter_input(INPUT_POST,"category",FILTER_SANITIZE_STRING));
$title = trim(filter_input(INPUT_POST,"title",FILTER_SANITIZE_STRING));
$format = trim(filter_input(INPUT_POST,"format",FILTER_SANITIZE_STRING));
$genre = trim(filter_input(INPUT_POST,"genre",FILTER_SANITIZE_STRING));
$year = trim(filter_input(INPUT_POST,"year",FILTER_SANITIZE_STRING));
$details = trim(filter_input(INPUT_POST,"details",FILTER_SANITIZE_SPECIAL_CHARS));

  if ($name == "" || $email == "" || $category == "" || $title == "") {
    echo "Please fill in the required fields: Name, Email, Category and Title";
    exit;
  }
  if ($_POST["address"] != "") {
    echo "Bad form input";
    exit;
  }

  require("inc/phpmailer/class.phpmailer.php");

  $mail = new PHPMailer;

  if (!$mail->ValidateAddress($email)) {
   echo "Invalid Email Address"; 
   exit;
  }

  $email_body = "";
  $email_body .= "Name " . $name . "\n";
  $email_body .= "E-mail " . $email . "\n";
  $email_body .= "Suggested Item\n";
  $email_body .= "Category " . $category . "\n";
  $email_body .= "Title " . $title . "\n";
  $email_body .= "Format " . $format . "\n";
  $email_body .= "Genre " . $genre . "\n";
  $email_body .= "Year " . $year . "\n";
  $email_body.= "Details " . $details . "\n";

  $mail->setFrom($email, $name);
  $mail->addAddress('treehouse@localhost', 'Alena');     // Add a recipient

  $mail->isHTML(false);                                  // Set email format to HTML

  $mail->Subject = 'Personal Media Library Suggestion from ' . $name;
  $mail->Body    = $email_body;

  if(!$mail->send()) {
      echo 'Message could not be sent.';
      echo 'Mailer Error: ' . $mail->ErrorInfo;
    }


header("location:suggest.php?status=thanks");
}

$pageTitle = "Suggest a Media Item";
$section = "suggest";

include("inc/header.php"); ?>

<div class="section page">
  <div class="wrapper">
    <h1>Suggest a Media Item</h1>
    <?php if (isset($_GET["status"]) && $_GET["status"] == "thanks") {
      echo " <p>Thanks for your email! I&rsquo;ll check out your suggestion shortly</p>";
    } else { ?>
    <p>If you think there is something I&rsquo;m missing, lmk! Complete the form to send me an e-mail.</p>
    <form method="post" action="suggest.php">
      <table>

        <tr>
          <th> <label for="name">Name (required)</label></th>
          <td><input type="text" id="name" name="name" /></td>   
        </tr>
        <tr>
          <th> <label for="email">Email (required)</label></th>
          <td><input type="text" id="email" name="email" /></td>   
        </tr>
        <tr>
          <th> <label for="category">Category (required)</label></th>
          <td><select id="category" name="category">
              <option value="">Select One</option>
              <option value="">Book</option>
              <option value="">Movie</option>
              <option value="">Music</option>
            </select>
            </td>   
        </tr>
        <tr>
          <th> <label for="title">Title (required)</label></th>
          <td><input type="text" id="title" name="title" /></td>   
        </tr>
        <tr>
          <th> <label for="format">Format</label></th>
          <td><select id="format" name="format">
              <option value="">Select One</option>
              <optgroup label="Books">
                  <option value="Audio">Audio</option>
                  <option value="Ebook">Ebook</option>
                  <option value="Hardback">Hardback</option>
                  <option value="Paperback">Paperback</option>
                </optgroup>
              <optgroup label="Movie">Movie
                  <option value="Blu-ray">Blu-ray</option>
                  <option value="DVD">DVD</option>
                  <option value="Streaming">Streaming</option>
                  <option value="VHS">VHS</option>
            </optgroup>
              <optgroup label="Music">Music
                  <option value="CD">CD</option>
                  <option value="MP3">MP3</option>
                  <option value="Vinyl">Vinyl</option>
          <tr>
            <th> <label for="genre">Genre</label></th>
                <td>
                    <select name="genre" id="genre">
                        <option value="">Select One</option>
                        <optgroup label="Books">
                            <option value="Action">Action</option>
                            <option value="Adventure">Adventure</option>
                            <option value="Comedy">Comedy</option>
                            <option value="Fantasy">Fantasy</option>
                            <option value="Historical">Historical</option>
                            <option value="Historical Fiction">Historical Fiction</option>
                            <option value="Horror">Horror</option>
                            <option value="Magical Realism">Magical Realism</option>
                            <option value="Mystery">Mystery</option>
                            <option value="Paranoid">Paranoid</option>
                            <option value="Philosophical">Philosophical</option>
                            <option value="Political">Political</option>
                            <option value="Romance">Romance</option>
                            <option value="Saga">Saga</option>
                            <option value="Satire">Satire</option>
                            <option value="Sci-Fi">Sci-Fi</option>
                            <option value="Tech">Tech</option>
                            <option value="Thriller">Thriller</option>
                            <option value="Urban">Urban</option>
                        </optgroup>
                        <optgroup label="Movies">
                            <option value="Action">Action</option>
                            <option value="Adventure">Adventure</option>
                            <option value="Animation">Animation</option>
                            <option value="Biography">Biography</option>
                            <option value="Comedy">Comedy</option>
                            <option value="Crime">Crime</option>
                            <option value="Documentary">Documentary</option>
                            <option value="Drama">Drama</option>
                            <option value="Family">Family</option>
                            <option value="Fantasy">Fantasy</option>
                            <option value="Film-Noir">Film-Noir</option>
                            <option value="History">History</option>
                            <option value="Horror">Horror</option>
                            <option value="Musical">Musical</option>
                            <option value="Mystery">Mystery</option>
                            <option value="Romance">Romance</option>
                            <option value="Sci-Fi">Sci-Fi</option>
                            <option value="Sport">Sport</option>
                            <option value="Thriller">Thriller</option>
                            <option value="War">War</option>
                            <option value="Western">Western</option>
                        </optgroup>
                        <optgroup label="Music">
                            <option value="Alternative">Alternative</option>
                            <option value="Blues">Blues</option>
                            <option value="Classical">Classical</option>
                            <option value="Country">Country</option>
                            <option value="Dance">Dance</option>
                            <option value="Easy Listening">Easy Listening</option>
                            <option value="Electronic">Electronic</option>
                            <option value="Folk">Folk</option>
                            <option value="Hip Hop/Rap">Hip Hop/Rap</option>
                            <option value="Inspirational/Gospel">Insirational/Gospel</option>
                            <option value="Jazz">Jazz</option>
                            <option value="Latin">Latin</option>
                            <option value="New Age">New Age</option>
                            <option value="Opera">Opera</option>
                            <option value="Pop">Pop</option>
                            <option value="R&B/Soul">R&amp;B/Soul</option>
                            <option value="Reggae">Reggae</option>
                            <option value="Rock">Rock</option>
                        </optgroup>
                    </select>
                </td>
            </tr>
            </optgroup>
            </select>
            </td>   
        </tr>
        <tr>
          <th> <label for="details">Suggest Item Details</label></th>
          <td><textarea name="details" id="details"></textarea></td>   
        </tr>
        <tr style="display:none">
          <th> <label for="address">Address</label></th>
          <td><input type ="text" id="address" name="address" />
          <p>Please leave this field blank</p></td>   
        </tr>

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




    </form>
    <?php } ?>
    </div>
</div>

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

Thanks in advance

1 Answer

Adam Ridgley
Adam Ridgley
12,255 Points

Try using this code for the select category part of the form:

<select id="category" name="category">
<option value="test1">Select One</option>
<option value="test2">Book</option>
<option value="test3">Movie</option
<option value="test4">Music</option>
</select>

I think you're getting the validation message because you have empty values on the option elements.

Sarah Rose
Sarah Rose
6,918 Points
Sarah Rose
Sarah Rose
6,918 Points

Oh man so simple! So stupid I am! Thank you so much