[ 
https://issues.apache.org/jira/browse/LANG-421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12584648#action_12584648
 ] 

[EMAIL PROTECTED] edited comment on LANG-421 at 4/2/08 8:51 AM:
-----------------------------------------------------------

The JLS reference is 
http://java.sun.com/docs/books/jls/second_edition/html/lexical.doc.html#101083 
(3.10.5 String Literals)

      was (Author: [EMAIL PROTECTED]):
    The JLS reference is 
http://java.sun.com/docs/books/jls/second_edition/html/lexical.doc.html#101083
  
> StringEscapeUtils.escapeJava(String) escapes '/' characters
> -----------------------------------------------------------
>
>                 Key: LANG-421
>                 URL: https://issues.apache.org/jira/browse/LANG-421
>             Project: Commons Lang
>          Issue Type: Bug
>    Affects Versions: 2.4
>            Reporter: Steve Hanmann
>            Priority: Blocker
>         Attachments: StringEscapeUtilsTest.java
>
>
> Commons Lang 2.4 StringEscapeUtils.escapeJava(String) now escapes '/' 
> characters, which is not a valid "escapable" character in Java strings.  I 
> haven't tried the other Java escape/unescape methods to see if they have a 
> similar problem, or that only Java "escapable" characters are escaped by 
> escapeJava(String).
> This bug may have appeared as an unintended side-effect of the fix for 
> LANG-363.
> Also the javadoc for escapeJava is now a little off, in that '/' should now 
> be included in the sentence describing the differences between Java and 
> Javascript strings, with respect to escaping rules.
> The following is a JUnit3 test demonstrating the bug.
> import junit.framework.TestCase;
> import org.apache.commons.lang.StringEscapeUtils;
> public class StringEscapeUtilsTest extends TestCase {
>     public void testEscapeJavaWithSlash() {
>         final String input = "String with a slash (/) in it";
>         
>         final String expected = input;
>         final String actual   = StringEscapeUtils.escapeJava( input );
>         /**
>          * In 2.4 StringEscapeUtils.escapeJava(String) escapes '/' characters,
>          * which are not a valid character to escape in a Java string.  
>          */
>         assertEquals( expected, actual );
>     }
> }

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