-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Escaping and CFS #16
Comments
hey rose, just to be clear, am i to wrap both the CFS calls and echos? i'm not sure exactly which to use the esc_ functions. for example, in about.php we have carousels present with echos. will those need to be wrapped as well? |
The escape function doesn't have to wrap the echo. Basically it will look something like this: basically anytime there is a |
ah i see. so i can leave the lone echos alone and focus on the CFS gets then. i wasn't too sure about that after going through the documentation. thanks for clearing that up! |
sorry rose, i'm unsure about one thing. do the CFS gets need to contain an echo? should i be wrapping them in escaping functions if they don't have an echo? wrap only the gets containing an echo? |
also, there are a few CFS gets for the forms, i'm having issues wrapping these. tried html, textarea, and attr. |
hmmm, ya... i input this:
|
In your theme php files, remember to use escaping functions such as esc_html as a security standard (we want to make sure we are outputting what we think we are outputting). There is more about this in the security lesson slides as well!
The text was updated successfully, but these errors were encountered: