upd: narrowing type-safety around Organisation

[gigi.git] / src / org / cacert / gigi / pages / orga / ViewOrgPage.java

diff --git a/src/org/cacert/gigi/pages/orga/ViewOrgPage.java b/src/org/cacert/gigi/pages/orga/ViewOrgPage.java
index 17ad0707b9737497789c85ecc9b6c002a1d9d84d..d5839013222d6a70d543970bc12a77d7c414c141 100644 (file)
--- a/src/org/cacert/gigi/pages/orga/ViewOrgPage.java
+++ b/src/org/cacert/gigi/pages/orga/ViewOrgPage.java
@@ -9,7 +9,6 @@ import java.util.Map;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import org.cacert.gigi.GigiApiException;
 import org.cacert.gigi.dbObjects.Organisation;
 import org.cacert.gigi.dbObjects.User;
 import org.cacert.gigi.localisation.Language;
@@ -40,41 +39,37 @@ public class ViewOrgPage extends Page {
 
     @Override
     public void doPost(HttpServletRequest req, HttpServletResponse resp) throws IOException {
-        try {
-            User u = LoginPage.getUser(req);
-            if (req.getParameter("do_affiliate") != null || req.getParameter("del") != null) {
-                AffiliationForm form = Form.getForm(req, AffiliationForm.class);
-                if (form.submit(resp.getWriter(), req)) {
+        User u = LoginPage.getUser(req);
+        if (req.getParameter("do_affiliate") != null || req.getParameter("del") != null) {
+            AffiliationForm form = Form.getForm(req, AffiliationForm.class);
+            if (form.submitProtected(resp.getWriter(), req)) {
+                resp.sendRedirect(DEFAULT_PATH + "/" + form.getOrganisation().getId());
+            }
+            return;
+        } else {
+            if ( !u.isInGroup(CreateOrgPage.ORG_ASSURER)) {
+                resp.sendError(403, "Access denied");
+                return;
+            }
+
+            if (req.getParameter("addDomain") != null) {
+                OrgDomainAddForm form = Form.getForm(req, OrgDomainAddForm.class);
+                if (form.submitProtected(resp.getWriter(), req)) {
                     resp.sendRedirect(DEFAULT_PATH + "/" + form.getOrganisation().getId());
                 }
-                return;
-            } else {
-                if ( !u.isInGroup(CreateOrgPage.ORG_ASSURER)) {
-                    resp.sendError(403, "Access denied");
-                    return;
+            } else if (req.getParameter("delete") != null) {
+                DomainManagementForm form = Form.getForm(req, DomainManagementForm.class);
+                if (form.submitProtected(resp.getWriter(), req)) {
+                    resp.sendRedirect(DEFAULT_PATH + "/" + form.getTarget().getId());
                 }
-
-                if (req.getParameter("addDomain") != null) {
-                    OrgDomainAddForm form = Form.getForm(req, OrgDomainAddForm.class);
-                    if (form.submit(resp.getWriter(), req)) {
-                        resp.sendRedirect(DEFAULT_PATH + "/" + form.getOrganisation().getId());
-                    }
-                } else if (req.getParameter("delete") != null) {
-                    DomainManagementForm form = Form.getForm(req, DomainManagementForm.class);
-                    if (form.submit(resp.getWriter(), req)) {
-                        resp.sendRedirect(DEFAULT_PATH + "/" + form.getTarget().getId());
-                    }
-                } else {
-                    CreateOrgForm form = Form.getForm(req, CreateOrgForm.class);
-                    if (form.submit(resp.getWriter(), req)) {
-                        resp.sendRedirect(DEFAULT_PATH + "/" + form.getResult().getId());
-                    }
+            } else {
+                CreateOrgForm form = Form.getForm(req, CreateOrgForm.class);
+                if (form.submitProtected(resp.getWriter(), req)) {
+                    resp.sendRedirect(DEFAULT_PATH + "/" + form.getResult().getId());
                 }
             }
-
-        } catch (GigiApiException e) {
-            e.format(resp.getWriter(), getLanguage(req));
         }
+
     }
 
     @Override
@@ -137,7 +132,7 @@ public class ViewOrgPage extends Page {
                 Organisation org = orgas[count++];
                 vars.put("id", Integer.toString(org.getId()));
                 vars.put("name", org.getName());
-                vars.put("country", org.getState());
+                vars.put("country", org.getState().getCountryCode());
                 return true;
             }
         };