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

section handle htmlentities #22

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@pixelninja

pixelninja commented Jul 25, 2012

Yo, I had to add htmlentities() to the section handle processor as it was spitting the dreaded Symphony entity error otherwise (My section has an ampersand in it).

Not sure if this is the desired way to do it, but it works.

@brendo

This comment has been minimized.

Show comment
Hide comment
@brendo

brendo Jul 26, 2012

Member

General::sanitize should also do the job, but it'll make sure it doesn't double encode, can you verify?

Member

brendo commented Jul 26, 2012

General::sanitize should also do the job, but it'll make sure it doesn't double encode, can you verify?

@brendo brendo closed this in 8c1692b Dec 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment