0ct0pu5/fess
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix #2301 check if content is blank

Browse source
This commit is contained in:
Shinsuke Sugaya 2019-11-11 06:40:06 +09:00
parent 4db8436d7d
commit 0df0220c09
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/main/java/org/codelibs/fess/helper/DocumentHelper.java
View file

@ -256,7 +256,7 @@ public class DocumentHelper {
}
public String appendLineNumber(final String prefix, final String content) {
if (content == null) {
if (StringUtil.isBlank(content)) {
return StringUtil.EMPTY;
}
final String[] values = content.split("\n");

6
src/test/java/org/codelibs/fess/helper/DocumentHelperTest.java
View file

@ -179,7 +179,11 @@ public class DocumentHelperTest extends UnitFessTestCase {
DocumentHelper documentHelper = new DocumentHelper();
assertEquals("", documentHelper.appendLineNumber("L", null));
assertEquals("L1:", documentHelper.appendLineNumber("L", ""));
assertEquals("", documentHelper.appendLineNumber("L", ""));
assertEquals("", documentHelper.appendLineNumber("L", " "));
assertEquals("", documentHelper.appendLineNumber("L", "\n"));
assertEquals("", documentHelper.appendLineNumber("L", "\n\n"));
assertEquals("", documentHelper.appendLineNumber("L", "\n \n"));
assertEquals("L1:aaa", documentHelper.appendLineNumber("L", "aaa"));
assertEquals("L1:aaa", documentHelper.appendLineNumber("L", "aaa\n"));
assertEquals("L1:aaa\nL2:bbb", documentHelper.appendLineNumber("L", "aaa\nbbb"));