Put more intelligence into the "outputError"-function of "Form".

[gigi.git] / src / org / cacert / gigi / output / Form.java

diff --git a/src/org/cacert/gigi/output/Form.java b/src/org/cacert/gigi/output/Form.java
index 176c7046995a57147b5066a85f45cc1f577b5b26..a4c65a488c70310b311627247e0484df22c9c362 100644 (file)
--- a/src/org/cacert/gigi/output/Form.java
+++ b/src/org/cacert/gigi/output/Form.java
@@ -34,6 +34,7 @@ public abstract class Form implements Outputable {
     @Override
     public void output(PrintWriter out, Language l, Map<String, Object> vars) {
         out.println("<form method='POST' autocomplete='off'>");
+        failed = false;
         outputContent(out, l, vars);
         out.print("<input type='hidden' name='" + CSRF_FIELD + "' value='");
         out.print(getCSRFToken());
@@ -42,12 +43,39 @@ public abstract class Form implements Outputable {
 
     protected abstract void outputContent(PrintWriter out, Language l, Map<String, Object> vars);
 
-    protected void outputError(PrintWriter out, ServletRequest req, String text) {
+    boolean failed;
+
+    protected void outputError(PrintWriter out, ServletRequest req, String text, Object... contents) {
+        if ( !failed) {
+            failed = true;
+            out.println("<div class='formError'>");
+        }
         out.print("<div>");
-        out.print(Page.translate(req, text));
+        if (contents.length == 0) {
+            out.print(Page.translate(req, text));
+        } else {
+            out.print(String.format(Page.translate(req, text), contents));
+        }
         out.println("</div>");
     }
 
+    protected void outputErrorPlain(PrintWriter out, String text) {
+        if ( !failed) {
+            failed = true;
+            out.println("<div class='formError'>");
+        }
+        out.print("<div>");
+        out.print(text);
+        out.println("</div>");
+    }
+
+    public boolean isFailed(PrintWriter out) {
+        if (failed) {
+            out.println("</div>");
+        }
+        return failed;
+    }
+
     protected String getCSRFToken() {
         return csrf;
     }