/site/index: add validation for catalog number to fix IllegalArgumentException.

Fix #341

Author: cssru

src/main/java/ru/mystamps/web/controller/SeriesController.java

@@ -30,6 +30,8 @@
 import javax.servlet.http.HttpServletResponse;
 import javax.validation.groups.Default;
 
+import org.apache.commons.lang3.StringUtils;
+
 import org.springframework.beans.propertyeditors.StringTrimmerEditor;
 import org.springframework.stereotype.Controller;
 import org.springframework.ui.Model;
@@ -345,12 +347,12 @@ public String searchSeriesByCatalog(
 		@RequestParam("catalogName") String catalogName,
 		Model model,
 		Locale userLocale,
-		HttpServletResponse response)
+		RedirectAttributes redirectAttributes)
 		throws IOException {
 
-		if (catalogNumber == null) {
-			response.sendError(HttpServletResponse.SC_NOT_FOUND);
-			return null;
+		if (StringUtils.isBlank(catalogNumber)) {
+			redirectAttributes.addFlashAttribute("numberIsEmpty", true);
+			return "redirect:" + Url.INDEX_PAGE;
 		}
 
 		String lang = LocaleUtils.getLanguageOrNull(userLocale);

src/main/resources/application-prod.properties

@@ -26,6 +26,7 @@ spring.messages.cache-seconds: -1
 spring.messages.fallback-to-system-locale: false
 spring.messages.basename: \
     ru/mystamps/i18n/Messages, \
+    ru/mystamps/i18n/ValidationMessages, \
     ru/mystamps/i18n/SpringSecurityMessages, \
     ru/mystamps/i18n/MailTemplates
 

src/main/resources/application-test.properties

@@ -28,6 +28,7 @@ spring.messages.cache-seconds: 60
 spring.messages.fallback-to-system-locale: false
 spring.messages.basename: \
     ru/mystamps/i18n/Messages, \
+    ru/mystamps/i18n/ValidationMessages, \
     ru/mystamps/i18n/SpringSecurityMessages, \
     ru/mystamps/i18n/MailTemplates
 

src/main/resources/application-travis.properties

@@ -26,6 +26,7 @@ spring.messages.cache-seconds: -1
 spring.messages.fallback-to-system-locale: false
 spring.messages.basename: \
     ru/mystamps/i18n/Messages, \
+    ru/mystamps/i18n/ValidationMessages, \
     ru/mystamps/i18n/SpringSecurityMessages, \
     ru/mystamps/i18n/MailTemplates
 

src/main/resources/ru/mystamps/i18n/ValidationMessages.properties

@@ -24,6 +24,7 @@ value.too-short = Value is less than allowable minimum of {min} characters
 value.too-long = Value is greater than allowable maximum of {max} characters
 value.invalid-length = Value length must be equals to {max} characters
 value.hyphen = Value must not start or end with hyphen
+value.empty = Value must not be empty
 
 category-name-en.invalid = Category name must consist only latin letters, hyphen or spaces
 category-name-ru.invalid = Category name must consist only Russian letters, hyphen or spaces

src/main/resources/ru/mystamps/i18n/ValidationMessages_ru.properties

@@ -24,6 +24,7 @@ value.too-short = Значение должно быть не менее {min} 
 value.too-long = Значение должно быть не более {max} символов
 value.invalid-length = Значение должно быть длинной {max} символов
 value.hyphen = Значение не должно начинаться или заканчиваться знаком дефиса
+value.empty = Значение не должно быть пустым
 
 category-name-en.invalid = Название категории может содержать только латинские буквы, дефис или пробел
 category-name-ru.invalid = Название категории может содержать только кирилические буквы, дефис или пробел

src/main/webapp/WEB-INF/views/site/index.html

@@ -247,9 +247,14 @@ <h4 class="panel-title" th:text="#{t_search_by_catalog}">Search by catalog</h4>
 						</div>
 						<div class="panel-body">
 							<form method="post" action="../series/search_result.html" th:action="@{${SEARCH_SERIES_BY_CATALOG}}">
-								<div class="form-group">
+								<div class="form-group" th:classappend="${numberIsEmpty != null ? 'has-error' : ''}">
 									<label for="catalogNumber" th:text="|#{t_number}:|">Number:</label>
 									<input id="catalogNumber" name="catalogNumber" type="search" class="form-control" required="required" />
+									<!--/*/
+									<span th:if="${numberIsEmpty != null}" th:text="#{value.empty}" class="help-block">
+										Value must not be empty
+									</span>
+									/*/-->
 								</div>
 								<div class="form-group">
 									<label for="catalogName" th:text="|#{t_catalog}:|">Catalog:</label>