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 Enhancing a Simple PHP Application Adding Search: Controller & View Escaping Output Review

Don Noray
Don Noray
19,238 Points
Posted by Don Noray
Don Noray
19,238 Points

Adding Search: Controller & View

Escaping Output review: A visitor to our site has just performed a search for chocolate, which has returned no results. Change this code to display the search term in the search box again, making sure to protect the page against malicious code that might have been entered.

I entered: value="<?php if (isset($s)) { echo htmlspecialchars($s); } ?>"

Results: Bummer! To place text inside a text input field, you should give it a value attribute.

3 Answers

Michael Garcia
Michael Garcia
7,752 Points
Michael Garcia
Michael Garcia
7,752 Points

This worked for me Don:

see below, as I messed up this comment.

Michael Garcia
Michael Garcia
7,752 Points
Michael Garcia
Michael Garcia
7,752 Points 
<input type="text" value="<?php if (!$products){ echo htmlspecialchars($search_term); }?>" name="s">
steven schwerin
steven schwerin
7,722 Points
steven schwerin
steven schwerin
7,722 Points

Thanks, Michael,

Would you explain this code to me? I don't understand why we can't echo $_GET["s"] instead of $search_term and also the (!$products).