commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthew R Hurne (JIRA)" <j...@apache.org>
Subject [jira] Commented: (LANG-423) StringEscapeUtils.escapeJava() behavior changed; backslash escapes '/' characters
Date Fri, 11 Apr 2008 17:50:04 GMT

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


Mime
View raw message