[jira] Commented: (LANG-423) StringEscapeUtils.escapeJava() behavior changed; backslash escapes '/' characters

["Matthew R Hurne (JIRA)" <ji...@apache.org>]

    [ 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.