You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jspwiki.apache.org by ju...@apache.org on 2020/11/29 21:54:34 UTC

[jspwiki] 08/11: ensure IndexPlugin works with non-blank page references

This is an automated email from the ASF dual-hosted git repository.

juanpablo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/jspwiki.git

commit 9ae2f9fff7533beb9784ec8f9c89401b58623552
Author: juanpablo <ju...@apache.org>
AuthorDate: Sun Nov 29 22:39:16 2020 +0100

    ensure IndexPlugin works with non-blank page references
---
 jspwiki-main/src/main/java/org/apache/wiki/plugin/IndexPlugin.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/jspwiki-main/src/main/java/org/apache/wiki/plugin/IndexPlugin.java b/jspwiki-main/src/main/java/org/apache/wiki/plugin/IndexPlugin.java
index f326714..29b1a5b 100644
--- a/jspwiki-main/src/main/java/org/apache/wiki/plugin/IndexPlugin.java
+++ b/jspwiki-main/src/main/java/org/apache/wiki/plugin/IndexPlugin.java
@@ -19,6 +19,7 @@
 
 package org.apache.wiki.plugin;
 
+import org.apache.commons.lang3.StringUtils;
 import org.apache.log4j.Logger;
 import org.apache.wiki.api.core.Context;
 import org.apache.wiki.api.core.ContextEnum;
@@ -77,8 +78,8 @@ public class IndexPlugin extends AbstractReferralPlugin implements Plugin {
             context.getEngine().getManager( PageManager.class ).getPageSorter().sort(pages);
             char initialChar = ' ';
             Element currentDiv = new Element("div",xmlns_XHTML);            
-            for ( final String name : pages ) {
-                if ( name.charAt(0) != initialChar ) {
+            for( final String name : pages ) {
+                if( StringUtils.isNotBlank( name ) &&  name.charAt(0) != initialChar ) {
                     if ( initialChar != ' ' ) {
                         indexDiv.addContent(" - ");
                     }