[ 
https://issues.apache.org/jira/browse/LANG-505?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Henri Yandell updated LANG-505:
-------------------------------

    Attachment: LANG-505.diff

Attaching CharSequenceEscaper and various other changes to StringEscapeUtils to 
turn it into a system whereby the user can pick and choose what 'escapeXyz' 
means for them.

Still a work in progress. The XML/HTML parts need implementing, and 
javadoc/cleanup needs to happen. Mostly attaching to determine if it's a 
valuable direction or if the future is regexps.

> Rewrite StringEscapeUtils
> -------------------------
>
>                 Key: LANG-505
>                 URL: https://issues.apache.org/jira/browse/LANG-505
>             Project: Commons Lang
>          Issue Type: Task
>            Reporter: Henri Yandell
>             Fix For: 3.0
>
>         Attachments: LANG-505.diff
>
>
> I think StringEscapeUtils needs a strong rewrite. For each escape method (and 
> unescape) there tend to be three or four types of escaping happening. So not 
> being able to define which set of three or four apply is a pain point (and 
> cause of bug reports due to different desired features).
> We should be offering basic functionality, but also allowing people to say 
> "escape(Escapers.BASIC_XML, Escapers.LOW_UNICODE, Escapers.HIGH_UNICODE)".
> Also should delete escapeSql; it's a bad one imo. Dangerous in that it will 
> lead people to not use PreparedStatement and given it only escapes ', it's 
> not much use. Especially as different dialects escape that in different ways.
> Opening this ticket for discussion. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to