upd: enforce a more strict Form call pattern.

[gigi.git] / src / org / cacert / gigi / pages / account / FindAgentAccess.java

diff --git a/src/org/cacert/gigi/pages/account/FindAgentAccess.java b/src/org/cacert/gigi/pages/account/FindAgentAccess.java
index 0728cdd4e221943fb7779d2157bccd5ea7b1a44d..47735f3273cf3220da81c0963441c883ec12a878 100644 (file)
--- a/src/org/cacert/gigi/pages/account/FindAgentAccess.java
+++ b/src/org/cacert/gigi/pages/account/FindAgentAccess.java
@@ -27,17 +27,17 @@ public class FindAgentAccess extends Form {
     private static final Template t = new Template(ChangePasswordPage.class.getResource("FindAgentAccess.templ"));
 
     @Override
     private static final Template t = new Template(ChangePasswordPage.class.getResource("FindAgentAccess.templ"));
 
     @Override

-    public boolean submit(HttpServletRequest req) throws GigiApiException {
+    public SubmissionResult submit(HttpServletRequest req) throws GigiApiException {

         String nv = req.getParameter("new-val");
         if (nv == null) {
         String nv = req.getParameter("new-val");
         if (nv == null) {

-            return false;
+            throw new GigiApiException("Parameter new-val missing.");

         }
         if (nv.equals("enable")) {
             target.grantGroup(target, Group.LOCATE_AGENT);
         } else {
             target.revokeGroup(target, Group.LOCATE_AGENT);
         }
         }
         if (nv.equals("enable")) {
             target.grantGroup(target, Group.LOCATE_AGENT);
         } else {
             target.revokeGroup(target, Group.LOCATE_AGENT);
         }

-        return true;
+        return new RedirectResult(FindAgentAccess.PATH);

     }
 
     @Override
     }
 
     @Override