Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Jakarta Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: local policy)
From: Gary Gregory <GGregory@seagullsoftware.com>
To: "commons-dev@jakarta.apache.org" <commons-dev@jakarta.apache.org>
Date: Wed, 2 Apr 2008 17:16:42 -0400
Subject: RE: [jira] Resolved: (LANG-421)
 StringEscapeUtils.escapeJava(String) escapes '/' characters
Thread-Topic: [jira] Resolved: (LANG-421)
 StringEscapeUtils.escapeJava(String) escapes '/' characters
Thread-Index: AciU3Dpoe0ZLSHrlT5aCsTW+VHWOZAAKiHVA
Message-ID: 
 <94C476C03BFF5E42AC3518FDAC9643C473E30E63C2@HQMAIL.rocketsoftware.com>
References: <1384645141.1207152564423.JavaMail.jira@brutus>
In-Reply-To: <1384645141.1207152564423.JavaMail.jira@brutus>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Arg, is there any interest in pushing out a 2.4.1 for this?

Thank you,
Gary

> -----Original Message-----
> From: Gary Gregory (JIRA) [mailto:jira@apache.org]
> Sent: Wednesday, April 02, 2008 9:09 AM
> To: Gary Gregory
> Subject: [jira] Resolved: (LANG-421) StringEscapeUtils.escapeJava(String)=
 escapes
> '/' characters
>
>
>      [ https://issues.apache.org/jira/browse/LANG-
> 421?page=3Dcom.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>
> Gary Gregory resolved LANG-421.
> -------------------------------
>
>        Resolution: Fixed
>     Fix Version/s: Nightly Builds
>                    3.0
>
> Fixed. Added unit test method based on attachment provided.
>
> > 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
> >            Assignee: Gary Gregory
> >            Priority: Blocker
> >             Fix For: 3.0, Nightly Builds
> >
> >         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 simila=
r
> 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 Java=
script
> 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 =3D "String with a slash (/) in it";
> >
> >         final String expected =3D input;
> >         final String actual   =3D StringEscapeUtils.escapeJava( input )=
;
> >         /**
> >          * In 2.4 StringEscapeUtils.escapeJava(String) escapes '/' char=
acters,
> >          * 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.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org