Browse Source

Merge pull request #223 from shatake/shatake_branch

modified Upload on suggest words
Shinsuke Sugaya 10 years ago
parent
commit
2b4d9d8a18

+ 1 - 1
src/main/java/jp/sf/fess/action/admin/SuggestBadWordAction.java

@@ -277,7 +277,7 @@ public class SuggestBadWordAction extends BsSuggestBadWordAction {
         final File oFile = tempFile;
         final File oFile = tempFile;
         try {
         try {
             final String head = new String(b, Constants.UTF_8);
             final String head = new String(b, Constants.UTF_8);
-            if (!head.startsWith("\"SuggestWord\",")) {
+            if (!(head.startsWith("\"BadWord\"") || head.startsWith("BadWord"))) {
                 log.error("Unknown file: "
                 log.error("Unknown file: "
                         + suggestBadWordForm.suggestBadWordFile);
                         + suggestBadWordForm.suggestBadWordFile);
                 throw new SSCActionMessagesException(
                 throw new SSCActionMessagesException(

+ 1 - 1
src/main/java/jp/sf/fess/action/admin/SuggestElevateWordAction.java

@@ -276,7 +276,7 @@ public class SuggestElevateWordAction extends BsSuggestElevateWordAction {
         final File oFile = tempFile;
         final File oFile = tempFile;
         try {
         try {
             final String head = new String(b, Constants.UTF_8);
             final String head = new String(b, Constants.UTF_8);
-            if (!head.startsWith("\"SuggestWord\",")) {
+            if (!(head.startsWith("\"SuggestWord\"") || head.startsWith("SuggestWord"))) {
                 log.error("Unknown file: "
                 log.error("Unknown file: "
                         + suggestElevateWordForm.suggestElevateWordFile);
                         + suggestElevateWordForm.suggestElevateWordFile);
                 throw new SSCActionMessagesException(
                 throw new SSCActionMessagesException(

+ 2 - 2
src/main/java/jp/sf/fess/service/SuggestBadWordService.java

@@ -178,7 +178,7 @@ public class SuggestBadWordService extends BsSuggestBadWordService implements
     }
     }
 
 
     private static String getValue(final List<String> list, final int index) {
     private static String getValue(final List<String> list, final int index) {
-        if (index < list.size()) {
+        if (index >= list.size()) {
             return StringUtil.EMPTY;
             return StringUtil.EMPTY;
         }
         }
         String item = list.get(index).trim();
         String item = list.get(index).trim();
@@ -187,7 +187,7 @@ public class SuggestBadWordService extends BsSuggestBadWordService implements
         }
         }
         if (item.length() > 1 && item.charAt(0) == '"'
         if (item.length() > 1 && item.charAt(0) == '"'
                 && item.charAt(item.length() - 1) == '"') {
                 && item.charAt(item.length() - 1) == '"') {
-            item = item.substring(1, item.length() - 2);
+            item = item.substring(1, item.length() - 1);
         }
         }
         return item;
         return item;
     }
     }

+ 2 - 2
src/main/java/jp/sf/fess/service/SuggestElevateWordService.java

@@ -204,7 +204,7 @@ public class SuggestElevateWordService extends BsSuggestElevateWordService
     }
     }
 
 
     static String getValue(final List<String> list, final int index) {
     static String getValue(final List<String> list, final int index) {
-        if (index < list.size()) {
+        if (index >= list.size()) {
             return StringUtil.EMPTY;
             return StringUtil.EMPTY;
         }
         }
         String item = list.get(index).trim();
         String item = list.get(index).trim();
@@ -213,7 +213,7 @@ public class SuggestElevateWordService extends BsSuggestElevateWordService
         }
         }
         if (item.length() > 1 && item.charAt(0) == '"'
         if (item.length() > 1 && item.charAt(0) == '"'
                 && item.charAt(item.length() - 1) == '"') {
                 && item.charAt(item.length() - 1) == '"') {
-            item = item.substring(1, item.length() - 2);
+            item = item.substring(1, item.length() - 1);
         }
         }
         return item;
         return item;
     }
     }