Actually use the default values.

getParam(String) returns a default value of "" and not null so the
previous detection of “no value is set” was actually never working, and
the default in the web interface was the default used by the browser
(i.e. the first element).

diff --git a/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java b/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java
index f476bbb..beda638 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java
+++ b/src/main/java/net/pterodactylus/sone/web/KnownSonesPage.java
@@ -66,11 +66,11 @@ 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 : defaultSortField);
-               templateContext.set("order", (sortOrder != null) ? sortOrder : defaultSortOrder);
+               templateContext.set("sort", sortField);
+               templateContext.set("order", sortOrder);
                templateContext.set("filter", filter);
                final Sone currentSone = getCurrentSone(request.getToadletContext(), false);
                Collection<Sone> knownSones = Collections2.filter(webInterface.getCore().getSones(), Sone.EMPTY_SONE_FILTER);