Rename “localSones” method to a more traditional “getSones”.

[Sone.git] / src / main / java / net / pterodactylus / sone / web / LoginPage.java

diff --git a/src/main/java/net/pterodactylus/sone/web/LoginPage.java b/src/main/java/net/pterodactylus/sone/web/LoginPage.java
index 4965bbf..39c3cdc 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/web/LoginPage.java
+++ b/src/main/java/net/pterodactylus/sone/web/LoginPage.java
@@ -51,20 +51,14 @@ public class LoginPage extends SoneTemplatePage {
         * {@inheritDoc}
         */
        @Override
-       protected void processTemplate(Request request, Template template) {
-               Set<Sone> localSones = webInterface.core().localSones();
+       protected void processTemplate(Request request, Template template) throws RedirectException {
+               super.processTemplate(request, template);
+               Set<Sone> localSones = webInterface.core().getSones();
                template.set("sones", localSones);
-       }
-
-       /**
-        * {@inheritDoc}
-        */
-       @Override
-       protected String getRedirectTarget(Request request) {
                if (request.getMethod() == Method.POST) {
-                       String soneId = request.getHttpRequest().getParam("sone-id");
+                       String soneId = request.getHttpRequest().getPartAsStringFailsafe("sone-id", 100);
                        Sone selectedSone = null;
-                       for (Sone sone : webInterface.core().localSones()) {
+                       for (Sone sone : webInterface.core().getSones()) {
                                if (sone.getId().equals(soneId)) {
                                        selectedSone = sone;
                                        break;
@@ -72,10 +66,17 @@ public class LoginPage extends SoneTemplatePage {
                        }
                        if (selectedSone != null) {
                                setCurrentSone(request.getToadletContext(), selectedSone);
-                               return "index.html";
+                               throw new RedirectException("index.html");
                        }
                }
-               return null;
+       }
+
+       /**
+        * {@inheritDoc}
+        */
+       @Override
+       public boolean isEnabled(ToadletContext toadletContext) {
+               return getCurrentSone(toadletContext) == null;
        }
 
 }