Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Fix issue #192 (escapeHtml accepting null values) #193

Closed
wants to merge 1 commit into from
Closed

Fix issue #192 (escapeHtml accepting null values) #193

wants to merge 1 commit into from

Conversation

ewrogers
Copy link

@ewrogers ewrogers commented Jun 6, 2016 •
edited
Loading

The escapeHtml() and unescapeHtml() methods now return null when the string value itself is null.

Fixes issue #192

@ewrogers ewrogers mentioned this pull request Jun 6, 2016
Open
@az7arul az7arul self-assigned this Jul 1, 2016
@az7arul
Copy link
Collaborator

az7arul commented Jul 1, 2016

@ewrogers did you delete your fork ?

@ewrogers
Copy link
Author

ewrogers commented Jul 3, 2016

I deleted it after making the PR request. Is that an issue?

@jackycute
Copy link

jackycute commented Feb 27, 2019 •
edited
Loading

I had same issue, could this PR be merged soon?

@ewrogers ewrogers closed this Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@az7arul az7arul

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ewrogers @az7arul @jackycute