[LANG-421] StringEscapeUtils.escapeJava(String) escapes '/' characters

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 2.4
• Fix Version/s: 2.5
• Component/s: lang.*
• Assignee: Gary D. Gregory
• Reporter: Steve Hanmann
• Created: 02/Apr/08 15:42
• Updated: 10/Mar/11 22:27
• Resolved: 02/Apr/08 16:07

Description

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 ); }

}

Issue Links

• is duplicated by: LANG-423 StringEscapeUtils.escapeJava() behavior changed; backslash escapes '/' characters
• is duplicated by: LANG-473 StringEscapeUtils.escapeJava () not escaping forward slash correctly.
• is related to: LANG-437 Complaints that the IE fix for StringEscapeUtils.escapeJavaScript is causing problems elsewhere. Rollback?