X-Git-Url: https://git.pterodactylus.net/?p=Sone.git;a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2FOptionsPage.java;h=51ca6cd34cb9cf9e7c7f7168eb30f35471f23cb1;hp=bbc1084862fef1e291070a4904fb4edef732eedf;hb=c996e8a5f69c0ccb4714847e8e70a453c377df87;hpb=a64878abea445300810c143ea129f4dac7725206

diff --git a/src/main/java/net/pterodactylus/sone/web/OptionsPage.java b/src/main/java/net/pterodactylus/sone/web/OptionsPage.java
index bbc1084..51ca6cd 100644
--- a/src/main/java/net/pterodactylus/sone/web/OptionsPage.java
+++ b/src/main/java/net/pterodactylus/sone/web/OptionsPage.java
@@ -17,6 +17,9 @@
 
 package net.pterodactylus.sone.web;
 
+import java.util.ArrayList;
+import java.util.List;
+
 import net.pterodactylus.sone.core.Core.Preferences;
 import net.pterodactylus.sone.data.Sone;
 import net.pterodactylus.sone.web.page.Page.Request.Method;
@@ -56,21 +59,38 @@ public class OptionsPage extends SoneTemplatePage {
 		Preferences preferences = webInterface.getCore().getPreferences();
 		Sone currentSone = webInterface.getCurrentSone(request.getToadletContext(), false);
 		if (request.getMethod() == Method.POST) {
+			List<String> fieldErrors = new ArrayList<String>();
 			if (currentSone != null) {
 				boolean autoFollow = request.getHttpRequest().isPartSet("auto-follow");
 				currentSone.getOptions().getBooleanOption("AutoFollow").set(autoFollow);
 				webInterface.getCore().saveSone(currentSone);
 			}
 			Integer insertionDelay = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("insertion-delay", 16));
-			preferences.setInsertionDelay(insertionDelay);
+			if (!preferences.validateInsertionDelay(insertionDelay)) {
+				fieldErrors.add("insertion-delay");
+			} else {
+				preferences.setInsertionDelay(insertionDelay);
+			}
 			Integer postsPerPage = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("posts-per-page", 4), null);
-			preferences.setPostsPerPage(postsPerPage);
+			if (!preferences.validatePostsPerPage(postsPerPage)) {
+				fieldErrors.add("posts-per-page");
+			} else {
+				preferences.setPostsPerPage(postsPerPage);
+			}
 			boolean requireFullAccess = request.getHttpRequest().isPartSet("require-full-access");
 			preferences.setRequireFullAccess(requireFullAccess);
 			Integer positiveTrust = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("positive-trust", 3));
-			preferences.setPositiveTrust(positiveTrust);
+			if (!preferences.validatePositiveTrust(positiveTrust)) {
+				fieldErrors.add("positive-trust");
+			} else {
+				preferences.setPositiveTrust(positiveTrust);
+			}
 			Integer negativeTrust = Numbers.safeParseInteger(request.getHttpRequest().getPartAsStringFailsafe("negative-trust", 4));
-			preferences.setNegativeTrust(negativeTrust);
+			if (!preferences.validateNegativeTrust(negativeTrust)) {
+				fieldErrors.add("negative-trust");
+			} else {
+				preferences.setNegativeTrust(negativeTrust);
+			}
 			String trustComment = request.getHttpRequest().getPartAsStringFailsafe("trust-comment", 256);
 			if (trustComment.trim().length() == 0) {
 				trustComment = null;
@@ -83,7 +103,10 @@ public class OptionsPage extends SoneTemplatePage {
 			boolean reallyClearOnNextRestart = Boolean.parseBoolean(request.getHttpRequest().getPartAsStringFailsafe("really-clear-on-next-restart", 5));
 			preferences.setReallyClearOnNextRestart(reallyClearOnNextRestart);
 			webInterface.getCore().saveConfiguration();
-			throw new RedirectException(getPath());
+			if (fieldErrors.isEmpty()) {
+				throw new RedirectException(getPath());
+			}
+			templateContext.set("fieldErrors", fieldErrors);
 		}
 		if (currentSone != null) {
 			templateContext.set("auto-follow", currentSone.getOptions().getBooleanOption("AutoFollow").get());