Browse Source

modify validations

Shinsuke Sugaya 9 years ago
parent
commit
3754776b70

+ 6 - 4
src/main/java/org/codelibs/fess/app/web/admin/searchlist/AdminSearchlistAction.java

@@ -118,13 +118,13 @@ public class AdminSearchlistAction extends FessAdminAction {
     }
     }
 
 
     protected HtmlResponse doSearch(final ListForm form) {
     protected HtmlResponse doSearch(final ListForm form) {
+        validate(form, messages -> {}, () -> asHtml(path_AdminSearchlist_IndexJsp));
 
 
         if (StringUtil.isBlank(form.query)) {
         if (StringUtil.isBlank(form.query)) {
             // redirect to index page
             // redirect to index page
             form.query = null;
             form.query = null;
             return redirect(getClass());
             return redirect(getClass());
         }
         }
-        validate(form, messages -> {}, () -> asHtml(path_ErrorJsp));
         return asHtml(path_AdminSearchlist_IndexJsp).renderWith(data -> {
         return asHtml(path_AdminSearchlist_IndexJsp).renderWith(data -> {
             doSearchInternal(data, form);
             doSearchInternal(data, form);
         });
         });
@@ -139,7 +139,7 @@ public class AdminSearchlistAction extends FessAdminAction {
             if (logger.isDebugEnabled()) {
             if (logger.isDebugEnabled()) {
                 logger.debug(e.getMessage(), e);
                 logger.debug(e.getMessage(), e);
             }
             }
-            throwValidationError(e.getMessageCode(), () -> asHtml(path_ErrorJsp));
+            throwValidationError(e.getMessageCode(), () -> asHtml(path_AdminSearchlist_IndexJsp));
         } catch (final ResultOffsetExceededException e) {
         } catch (final ResultOffsetExceededException e) {
             if (logger.isDebugEnabled()) {
             if (logger.isDebugEnabled()) {
                 logger.debug(e.getMessage(), e);
                 logger.debug(e.getMessage(), e);
@@ -192,14 +192,16 @@ public class AdminSearchlistAction extends FessAdminAction {
         validate(form, messages -> {}, () -> asHtml(path_AdminSearchlist_IndexJsp));
         validate(form, messages -> {}, () -> asHtml(path_AdminSearchlist_IndexJsp));
         final String docId = form.docId;
         final String docId = form.docId;
         if (jobHelper.isCrawlProcessRunning()) {
         if (jobHelper.isCrawlProcessRunning()) {
-            throwValidationError(messages -> messages.addErrorsCannotDeleteDocBecauseOfRunning(GLOBAL), () -> asHtml(path_AdminSearchlist_IndexJsp));
+            throwValidationError(messages -> messages.addErrorsCannotDeleteDocBecauseOfRunning(GLOBAL),
+                    () -> asHtml(path_AdminSearchlist_IndexJsp));
         }
         }
         try {
         try {
             final QueryBuilder query = QueryBuilders.termQuery(fieldHelper.docIdField, docId);
             final QueryBuilder query = QueryBuilders.termQuery(fieldHelper.docIdField, docId);
             fessEsClient.deleteByQuery(fieldHelper.docIndex, fieldHelper.docType, query);
             fessEsClient.deleteByQuery(fieldHelper.docIndex, fieldHelper.docType, query);
             saveInfo(messages -> messages.addSuccessDeleteSolrIndex(GLOBAL));
             saveInfo(messages -> messages.addSuccessDeleteSolrIndex(GLOBAL));
         } catch (final Exception e) {
         } catch (final Exception e) {
-            throwValidationError(messages -> messages.addErrorsFailedToDeleteDocInAdmin(GLOBAL), () -> asHtml(path_AdminSearchlist_IndexJsp));
+            throwValidationError(messages -> messages.addErrorsFailedToDeleteDocInAdmin(GLOBAL),
+                    () -> asHtml(path_AdminSearchlist_IndexJsp));
         }
         }
         return redirectWith(getClass(), moreUrl("search").params("query", form.query));
         return redirectWith(getClass(), moreUrl("search").params("query", form.query));
     }
     }