X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fweb%2FKnownSonesPage.java;h=a74132230a584cc900b4529bd841029747f48c81;hb=53a71d1d6b91e4d56af49a06f2e06bc4d11bf3eb;hp=dd1b1fc41f8df56f18321ac843f3cdd7c4a8116b;hpb=99888ce13cc17d49f5e217ab6f2c9ad5ef168792;p=Sone.git

diff --git a/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java b/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java
index dd1b1fc..a741322 100644
--- a/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java
+++ b/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java
@@ -17,6 +17,8 @@
 
 package net.pterodactylus.sone.web;
 
+import static net.pterodactylus.sone.utils.NumberParsers.parseInt;
+
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
@@ -25,10 +27,10 @@ import java.util.List;
 import net.pterodactylus.sone.data.Sone;
 import net.pterodactylus.sone.web.page.FreenetRequest;
 import net.pterodactylus.util.collection.Pagination;
-import net.pterodactylus.util.number.Numbers;
 import net.pterodactylus.util.template.Template;
 import net.pterodactylus.util.template.TemplateContext;
 
+import com.google.common.base.Optional;
 import com.google.common.base.Predicate;
 import com.google.common.base.Predicates;
 import com.google.common.collect.Collections2;
@@ -41,6 +43,9 @@ import com.google.common.collect.Ordering;
  */
 public class KnownSonesPage extends SoneTemplatePage {
 
+	private static final String defaultSortField = "activity";
+	private static final String defaultSortOrder = "desc";
+
 	/**
 	 * Creates a “known Sones” page.
 	 *
@@ -63,28 +68,28 @@ public class KnownSonesPage extends SoneTemplatePage {
 	@Override
 	protected void processTemplate(FreenetRequest request, TemplateContext templateContext) throws RedirectException {
 		super.processTemplate(request, templateContext);
-		String sortField = request.getHttpRequest().getParam("sort");
-		String sortOrder = request.getHttpRequest().getParam("order");
+		String sortField = request.getHttpRequest().getParam("sort", defaultSortField);
+		String sortOrder = request.getHttpRequest().getParam("order", defaultSortOrder);
 		String filter = request.getHttpRequest().getParam("filter");
-		templateContext.set("sort", (sortField != null) ? sortField : "name");
-		templateContext.set("order", (sortOrder != null) ? sortOrder : "asc");
+		templateContext.set("sort", sortField);
+		templateContext.set("order", sortOrder);
 		templateContext.set("filter", filter);
-		final Sone currentSone = getCurrentSone(request.getToadletContext(), false);
+		final Optional<Sone> currentSone = getCurrentSone(request.getToadletContext(), false);
 		Collection<Sone> knownSones = Collections2.filter(webInterface.getCore().getSones(), Sone.EMPTY_SONE_FILTER);
-		if ((currentSone != null) && "followed".equals(filter)) {
+		if (currentSone.isPresent() && "followed".equals(filter)) {
 			knownSones = Collections2.filter(knownSones, new Predicate<Sone>() {
 
 				@Override
 				public boolean apply(Sone sone) {
-					return currentSone.hasFriend(sone.getId());
+					return currentSone.get().hasFriend(sone.getId());
 				}
 			});
-		} else if ((currentSone != null) && "not-followed".equals(filter)) {
+		} else if (currentSone.isPresent() && "not-followed".equals(filter)) {
 			knownSones = Collections2.filter(knownSones, new Predicate<Sone>() {
 
 				@Override
 				public boolean apply(Sone sone) {
-					return !currentSone.hasFriend(sone.getId());
+					return !currentSone.get().hasFriend(sone.getId());
 				}
 			});
 		} else if ("new".equals(filter)) {
@@ -140,7 +145,7 @@ public class KnownSonesPage extends SoneTemplatePage {
 				Collections.sort(sortedSones, Sone.NICE_NAME_COMPARATOR);
 			}
 		}
-		Pagination<Sone> sonePagination = new Pagination<Sone>(sortedSones, 25).setPage(Numbers.safeParseInteger(request.getHttpRequest().getParam("page"), 0));
+		Pagination<Sone> sonePagination = new Pagination<Sone>(sortedSones, 25).setPage(parseInt(request.getHttpRequest().getParam("page"), 0));
 		templateContext.set("pagination", sonePagination);
 		templateContext.set("knownSones", sonePagination.getItems());
 	}