On 22/11/2013 12:44, Konstantin Kolinko wrote: > 2013/11/22 <ma...@apache.org>: >> Author: markt >> Date: Fri Nov 22 11:14:36 2013 >> New Revision: 1544488 >> >> URL: http://svn.apache.org/r1544488 >> Log: >> Tomcat 7 has a specific attribute for TLD validation so use it. >> >> Modified: >> >> tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java >> >> Modified: >> tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java >> URL: >> http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java?rev=1544488&r1=1544487&r2=1544488&view=diff >> ============================================================================== >> --- >> tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java >> (original) >> +++ >> tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ApplicationContext.java >> Fri Nov 22 11:14:36 2013 >> @@ -238,7 +238,7 @@ public class ApplicationContext >> // Special handling for XML validation as the context setting must >> // always override anything that might have been set by an >> application. >> if (Globals.JASPER_XML_VALIDATION_ATTR.equals(name) && >> - context.getXmlValidation()) { >> + context.getTldValidation()) { >> return Boolean.TRUE; >> } >> return (attributes.get(name)); > > > The code here handles JASPER_XML_VALIDATION_ATTR only. > I think there have to be the same handling for the > JASPER_XML_VALIDATION_DOCS_ATTR.
Yes. Good catch. Fixed. Thanks. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
