[
https://issues.apache.org/jira/browse/LANG-423?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12588056#action_12588056
]
Matthew R Hurne commented on LANG-423:
--------------------------------------
Hehe, looking at that diff, it appears someone already implemented a nearly
identical fix in svn.
> StringEscapeUtils.escapeJava() behavior changed; backslash escapes '/'
> characters
> ---------------------------------------------------------------------------------
>
> Key: LANG-423
> URL: https://issues.apache.org/jira/browse/LANG-423
> Project: Commons Lang
> Issue Type: Bug
> Affects Versions: 2.4
> Reporter: Matthew R Hurne
> Attachments: patch.diff
>
>
> StringEscapeUtils.escapeJava(String str) escapes forward-slash ("/")
> characters. Prior to 2.4, this was not the case. This is caused by the
> patch in LANG-363, which changed the behavior of
> StringEscapeUtils.escapeJavaScript(String str) to escape forward-slashes for
> compatibliity with IE. The change was implemented in code common to both
> methods; StringEscapeUtils.escapeJavaStyleString() .
> This side-effect change to StringEscapeUtils.escapeJava() is a problem in my
> project. I don't see any reason why it is necessary to escape
> forward-slashes in escapeJava(), so I suggest the behavior be changed so that
> escapeJavaScript() escapes forward-slashes, but escapeJava() does not.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
