Revision: 1379
          http://stripes.svn.sourceforge.net/stripes/?rev=1379&view=rev
Author:   bengunter
Date:     2010-12-29 21:19:20 +0000 (Wed, 29 Dec 2010)

Log Message:
-----------
STS-780: Changed output as suggested by Samuel Santos in an attempt to make the 
default validation error report XHTML-compatible.

Modified Paths:
--------------
    
trunk/stripes/src/net/sourceforge/stripes/action/ValidationErrorReportResolution.java

Modified: 
trunk/stripes/src/net/sourceforge/stripes/action/ValidationErrorReportResolution.java
===================================================================
--- 
trunk/stripes/src/net/sourceforge/stripes/action/ValidationErrorReportResolution.java
       2010-12-29 21:17:36 UTC (rev 1378)
+++ 
trunk/stripes/src/net/sourceforge/stripes/action/ValidationErrorReportResolution.java
       2010-12-29 21:19:20 UTC (rev 1379)
@@ -26,6 +26,7 @@
 import net.sourceforge.stripes.controller.StripesFilter;
 import net.sourceforge.stripes.exception.SourcePageNotFoundException;
 import net.sourceforge.stripes.tag.ErrorsTag;
+import net.sourceforge.stripes.util.HtmlUtil;
 import net.sourceforge.stripes.util.Log;
 import net.sourceforge.stripes.validation.ValidationError;
 
@@ -58,14 +59,12 @@
         // start the HTML error report
         response.setContentType("text/html");
         PrintWriter writer = response.getWriter();
-        writer.println("<html>");
-        writer.println("<head><title>Stripes validation error 
report</title></head>");
-        writer.println("<body style=\"font-family: Arial, sans-serif; 
font-size: 10pt;\">");
+        writer.println("<div style=\"font-family: Arial, sans-serif; 
font-size: 10pt;\">");
         writer.println("<h1>Stripes validation error report</h1><p>");
-        writer.println(exception.getMessage());
+        writer.println(HtmlUtil.encode(exception.getMessage()));
         writer.println("</p><h2>Validation errors</h2><p>");
         sendErrors(request, response);
-        writer.println("</p></body></html>");
+        writer.println("</p></div>");
     }
 
     /**
@@ -106,7 +105,7 @@
         for (List<ValidationError> list : 
getContext().getValidationErrors().values()) {
             for (ValidationError fieldError : list) {
                 writer.write(openElement);
-                writer.write(fieldError.getMessage(locale));
+                writer.write(HtmlUtil.encode(fieldError.getMessage(locale)));
                 writer.write(closeElement);
             }
         }


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

------------------------------------------------------------------------------
Learn how Oracle Real Application Clusters (RAC) One Node allows customers
to consolidate database storage, standardize their database environment, and, 
should the need arise, upgrade to a full multi-node Oracle RAC database 
without downtime or disruption
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to