X-Git-Url: https://code.wpia.club/?p=gigi.git;a=blobdiff_plain;f=src%2Forg%2Fcacert%2Fgigi%2Fpages%2FPage.java;h=054a1e52020bf1172a095ea26a82ce5c3b4595dd;hp=0bd02f789f115022277cd02391f41e676970b1c3;hb=17a15662212d973d12ed4cea3f5eaa9c0d1169ed;hpb=e409ba881965634f63f0b67824bc93dda4ec4327 diff --git a/src/org/cacert/gigi/pages/Page.java b/src/org/cacert/gigi/pages/Page.java index 0bd02f78..054a1e52 100644 --- a/src/org/cacert/gigi/pages/Page.java +++ b/src/org/cacert/gigi/pages/Page.java @@ -14,6 +14,7 @@ import org.cacert.gigi.PermissionCheckable; import org.cacert.gigi.dbObjects.User; import org.cacert.gigi.localisation.Language; import org.cacert.gigi.output.template.Template; +import org.cacert.gigi.util.AuthorizationContext; /** * This class encapsulates a sub page of Gigi. A template residing nearby this @@ -57,6 +58,29 @@ public abstract class Page implements PermissionCheckable { * if output goes wrong. */ public boolean beforeTemplate(HttpServletRequest req, HttpServletResponse resp) throws IOException { + if (req.getMethod().equals("POST")) { + return beforePost(req, resp); + } + return false; + } + + /** + * This method can be overridden to execute code and do stuff before the + * default template is applied when the request is a post request and the + * default implementation of + * {@link #beforeTemplate(HttpServletRequest, HttpServletResponse)} is + * called. + * + * @param req + * the request to handle. + * @param resp + * the response to write to + * @return true, if the request is consumed and the default template should + * not be applied. + * @throws IOException + * if output goes wrong. + */ + public boolean beforePost(HttpServletRequest req, HttpServletResponse resp) throws IOException { return false; } @@ -102,30 +126,29 @@ public abstract class Page implements PermissionCheckable { return title; } - public void setTitle(String title) { - this.title = title; - } - public static Language getLanguage(ServletRequest req) { HttpSession session = ((HttpServletRequest) req).getSession(); - Locale sessval = (Locale) session.getAttribute(Language.SESSION_ATTRIB_NAME); - if (sessval != null) { - Language l = Language.getInstance(sessval); - if (l != null) { - return l; + synchronized (session) { + + Locale sessval = (Locale) session.getAttribute(Language.SESSION_ATTRIB_NAME); + if (sessval != null) { + Language l = Language.getInstance(sessval); + if (l != null) { + return l; + } } - } - Enumeration langs = req.getLocales(); - while (langs.hasMoreElements()) { - Locale c = langs.nextElement(); - Language l = Language.getInstance(c); - if (l != null) { - session.setAttribute(Language.SESSION_ATTRIB_NAME, l.getLocale()); - return l; + Enumeration langs = req.getLocales(); + while (langs.hasMoreElements()) { + Locale c = langs.nextElement(); + Language l = Language.getInstance(c); + if (l != null) { + session.setAttribute(Language.SESSION_ATTRIB_NAME, l.getLocale()); + return l; + } } + session.setAttribute(Language.SESSION_ATTRIB_NAME, Locale.ENGLISH); + return Language.getInstance(Locale.ENGLISH); } - session.setAttribute(Language.SESSION_ATTRIB_NAME, Locale.ENGLISH); - return Language.getInstance(Locale.ENGLISH); } public static String translate(ServletRequest req, String string) { @@ -138,8 +161,8 @@ public abstract class Page implements PermissionCheckable { } @Override - public boolean isPermitted(User u) { - return !needsLogin() || u != null; + public boolean isPermitted(AuthorizationContext ac) { + return !needsLogin() || ac != null; } }