bodewig 2003/01/15 06:15:02
Modified: src/main/org/apache/tools/ant/util Tag: ANT_15_BRANCH
DOMElementWriter.java
src/testcases/org/apache/tools/ant/util Tag: ANT_15_BRANCH
DOMElementWriterTest.java
Log:
merge CDATA fix from HEAD
Revision Changes Path
No revision
No revision
1.11.2.1 +21 -3
jakarta-ant/src/main/org/apache/tools/ant/util/DOMElementWriter.java
Index: DOMElementWriter.java
===================================================================
RCS file:
/home/cvs/jakarta-ant/src/main/org/apache/tools/ant/util/DOMElementWriter.java,v
retrieving revision 1.11
retrieving revision 1.11.2.1
diff -u -r1.11 -r1.11.2.1
--- DOMElementWriter.java 15 Apr 2002 12:11:51 -0000 1.11
+++ DOMElementWriter.java 15 Jan 2003 14:15:01 -0000 1.11.2.1
@@ -1,7 +1,7 @@
/*
* The Apache Software License, Version 1.1
*
- * Copyright (c) 2000-2002 The Apache Software Foundation. All rights
+ * Copyright (c) 2000-2003 The Apache Software Foundation. All rights
* reserved.
*
* Redistribution and use in source and binary forms, with or without
@@ -243,8 +243,15 @@
/**
* Drop characters that are illegal in XML documents.
*
+ * <p>Also ensure that we are not including an <code>]]></code>
+ * marker by replacing that sequence with
+ * <code>&x5d;&x5d;&gt;</code>.</p>
+ *
* <p>See XML 1.0 2.2 <a
- *
href="http://www.w3.org/TR/1998/REC-xml-19980210#charsets">http://www.w3.org/TR/1998/REC-xml-19980210#charsets</a>.</p>
+ *
href="http://www.w3.org/TR/1998/REC-xml-19980210#charsets">http://www.w3.org/TR/1998/REC-xml-19980210#charsets</a>.
and
+ * 2.7 <a
+ *
href="http://www.w3.org/TR/1998/REC-xml-19980210#sec-cdata-sect">http://www.w3.org/TR/1998/REC-xml-19980210#sec-cdata-sect</a></p>
+
*/
public String encodedata(final String value) {
sb.setLength(0);
@@ -254,7 +261,18 @@
sb.append(c);
}
}
- return sb.toString();
+
+ String result = sb.toString();
+ int cdEnd = result.indexOf("]]>");
+ while (cdEnd != -1) {
+ sb.setLength(cdEnd);
+ sb.append("&x5d;&x5d;>")
+ .append(result.substring(cdEnd+3));
+ result = sb.toString();
+ cdEnd = result.indexOf("]]>");
+ }
+
+ return result;
}
/**
No revision
No revision
1.6.2.1 +12 -1
jakarta-ant/src/testcases/org/apache/tools/ant/util/DOMElementWriterTest.java
Index: DOMElementWriterTest.java
===================================================================
RCS file:
/home/cvs/jakarta-ant/src/testcases/org/apache/tools/ant/util/DOMElementWriterTest.java,v
retrieving revision 1.6
retrieving revision 1.6.2.1
diff -u -r1.6 -r1.6.2.1
--- DOMElementWriterTest.java 9 Apr 2002 12:13:25 -0000 1.6
+++ DOMElementWriterTest.java 15 Jan 2003 14:15:01 -0000 1.6.2.1
@@ -1,7 +1,7 @@
/*
* The Apache Software License, Version 1.1
*
- * Copyright (c) 2000-2002 The Apache Software Foundation. All rights
+ * Copyright (c) 2000-2003 The Apache Software Foundation. All rights
* reserved.
*
* Redistribution and use in source and binary forms, with or without
@@ -118,5 +118,16 @@
assertTrue("0xE000", w.isLegalCharacter('\uE000'));
assertTrue("0xFFFD", w.isLegalCharacter('\uFFFD'));
assertTrue("0xFFFE", !w.isLegalCharacter('\uFFFE'));
+ }
+
+ public void testCDATAEndEncoding() {
+ assertEquals("]>", w.encodedata("]>"));
+ assertEquals("]]", w.encodedata("]]"));
+ assertEquals("&x5d;&x5d;>", w.encodedata("]]>"));
+ assertEquals("&x5d;&x5d;>A", w.encodedata("]]>A"));
+ assertEquals("A&x5d;&x5d;>", w.encodedata("A]]>"));
+ assertEquals("A&x5d;&x5d;>A", w.encodedata("A]]>A"));
+ assertEquals("A&x5d;&x5d;>B&x5d;&x5d;>C",
+ w.encodedata("A]]>B]]>C"));
}
}
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>