X-Git-Url: https://code.wpia.club/?a=blobdiff_plain;ds=sidebyside;f=src%2Forg%2Fcacert%2Fgigi%2Fpages%2Fadmin%2FTTPAdminPage.java;h=2ffd12e2085882175260683572bb7c815230452d;hb=9fdd23fb432d0fec14efc4f8c739c58070a3e3ba;hp=90c2918cd0a10504b279ed844cd28ae87c7d0f98;hpb=d27db93cff33d510bff61a16d595b918af1735df;p=gigi.git diff --git a/src/org/cacert/gigi/pages/admin/TTPAdminPage.java b/src/org/cacert/gigi/pages/admin/TTPAdminPage.java index 90c2918c..2ffd12e2 100644 --- a/src/org/cacert/gigi/pages/admin/TTPAdminPage.java +++ b/src/org/cacert/gigi/pages/admin/TTPAdminPage.java @@ -1,6 +1,7 @@ package org.cacert.gigi.pages.admin; import java.io.IOException; +import java.util.Arrays; import java.util.HashMap; import java.util.Map; @@ -11,9 +12,12 @@ import org.cacert.gigi.GigiApiException; import org.cacert.gigi.dbObjects.Group; import org.cacert.gigi.dbObjects.User; import org.cacert.gigi.localisation.Language; -import org.cacert.gigi.output.Form; +import org.cacert.gigi.output.template.Form; import org.cacert.gigi.output.template.IterableDataset; +import org.cacert.gigi.output.template.SprintfCommand; import org.cacert.gigi.pages.Page; +import org.cacert.gigi.pages.error.PageNotFound; +import org.cacert.gigi.util.AuthorizationContext; public class TTPAdminPage extends Page { @@ -40,10 +44,13 @@ public class TTPAdminPage extends Page { @Override public void doGet(HttpServletRequest req, HttpServletResponse resp) throws IOException { String path = req.getPathInfo(); - if (path != null && path.length() > PATH.length()) { + if (path != null && path.length() > PATH.length() + 1) { int id = Integer.parseInt(path.substring(1 + PATH.length())); User u = User.getById(id); - if ( !u.isInGroup(TTP_APPLICANT)) { + if (u == null || !u.isInGroup(TTP_APPLICANT)) { + SprintfCommand command = new SprintfCommand("The TTP-request is not available anymore. You might want to go {0}back{1}.", Arrays.asList("!'", "!'")); + req.setAttribute(PageNotFound.MESSAGE_ATTRIBUTE, command); + resp.sendError(404); return; } new TTPAdminForm(req, u).output(resp.getWriter(), getLanguage(req), new HashMap()); @@ -67,7 +74,7 @@ public class TTPAdminPage extends Page { return false; } vars.put("id", Integer.toString(users[i].getId())); - vars.put("name", users[i].getName().toString()); + vars.put("name", users[i].getPreferredName().toString()); vars.put("email", users[i].getEmail()); i++; @@ -81,7 +88,7 @@ public class TTPAdminPage extends Page { } @Override - public boolean isPermitted(User u) { - return u != null && u.isInGroup(Group.getByString("ttp-assurer")); + public boolean isPermitted(AuthorizationContext ac) { + return ac != null && ac.isInGroup(Group.getByString("ttp-assurer")); } }