upd: enforce a more strict Form call pattern.

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

diff --git a/src/org/cacert/gigi/pages/account/MyDetails.java b/src/org/cacert/gigi/pages/account/MyDetails.java
index 321e90939a879d8d9636911ed93725037b4a44ac..e6ba3ead537408fa6f1e36a0abfe480f01e4884a 100644 (file)
--- a/src/org/cacert/gigi/pages/account/MyDetails.java
+++ b/src/org/cacert/gigi/pages/account/MyDetails.java
@@ -33,15 +33,11 @@ public class MyDetails extends Page {
 
     @Override
     public boolean beforePost(HttpServletRequest req, HttpServletResponse resp) throws IOException {
-        if (req.getParameter("orgaForm") != null && Form.getForm(req, MyOrganisationsForm.class).submitExceptionProtected(req)) {
-            resp.sendRedirect(PATH);
-            return true;
+        if (req.getParameter("orgaForm") != null) {
+            return Form.getForm(req, MyOrganisationsForm.class).submitExceptionProtected(req, resp);
         }
         if (req.getParameter("action") != null || req.getParameter("removeName") != null || req.getParameter("deprecateName") != null || req.getParameter("preferred") != null) {
-            if (Form.getForm(req, MyDetailsForm.class).submitExceptionProtected(req)) {
-                resp.sendRedirect(PATH);
-                return true;
-            }
+            return Form.getForm(req, MyDetailsForm.class).submitExceptionProtected(req, resp);
         }
         return false;
     }