On Mar 4, 2010, at 4:48 PM, Vincent Massol wrote:
On Mar 4, 2010, at 4:44 PM, Thomas Mortagne wrote:
On Thu, Mar 4, 2010 at 16:44, Thomas Mortagne <[email protected]> wrote:
On Thu, Mar 4, 2010 at 16:42, Thomas Mortagne <[email protected]> wrote:
On Thu, Mar 4, 2010 at 16:31, Vincent Massol <[email protected]> wrote:
On Mar 4, 2010, at 4:27 PM, tmortagne (SVN) wrote:
Author: tmortagne Date: 2010-03-04 16:27:02 +0100 (Thu, 04 Mar 2010) New Revision: 27446
Added: platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/util/TOCGeneratorTest.java Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/util/TOCGenerator.java Log: XWIKI-4971: Header generated ids are wrong in 1.0 syntax
Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/util/TOCGenerator.java =================================================================== --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/util/TOCGenerator.java 2010-03-04 14:57:59 UTC (rev 27445) +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/util/TOCGenerator.java 2010-03-04 15:27:02 UTC (rev 27446) @@ -119,7 +119,7 @@
public static String makeHeadingID(String text, int occurence, XWikiContext context) { - text = "H" + Util.encodeURI(text, context); + text = "H" + text; text = text.replaceAll("[^a-zA-Z0-9]", "");
if (occurence > 0) {
Added: platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/util/TOCGeneratorTest.java =================================================================== --- platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/util/TOCGeneratorTest.java (rev 0) +++ platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/util/TOCGeneratorTest.java 2010-03-04 15:27:02 UTC (rev 27446) @@ -0,0 +1,16 @@ +package com.xpn.xwiki.util; + +import junit.framework.Assert; + +import org.junit.Test; + +public class TOCGeneratorTest
missing license header + @since/@version javadoc.
Sorry, written the test too fast
Why @since/@version javadoc ?
You mean the class @version info ?
Yes @version and @since for the class.
For @since I put it everywhere but ok to limit it to APIs. Thanks -Vincent
Thanks -Vincent
Thanks -Vincent
+{ + @Test + public void testMakeHeadingID() + { + Assert.assertEquals("Hheader", TOCGenerator.makeHeadingID("header", 0, null)); + Assert.assertEquals("Hheaderwithspace", TOCGenerator.makeHeadingID("header with space", 0, null)); + Assert.assertEquals("Hheader-1", TOCGenerator.makeHeadingID("header", 1, null)); + } +}
Property changes on: platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/util/TOCGeneratorTest.java ___________________________________________________________________ Name: svn:mime-type + text/plain