On 13/07/2013 00:49, Konstantin Kolinko wrote: > 2013/7/12 <ma...@apache.org>: >> Author: markt >> Date: Fri Jul 12 13:25:04 2013 >> New Revision: 1502552 >> >> URL: http://svn.apache.org/r1502552 >> Log: >> Add some more unit tests for the EL processor and make sure it uses the >> import information it has to hand to resolve class names. >> >> Modified: >> tomcat/trunk/java/javax/el/ELProcessor.java >> tomcat/trunk/test/javax/el/TestELProcessor.java >> >> Modified: tomcat/trunk/java/javax/el/ELProcessor.java >> URL: >> http://svn.apache.org/viewvc/tomcat/trunk/java/javax/el/ELProcessor.java?rev=1502552&r1=1502551&r2=1502552&view=diff >> ============================================================================== >> --- tomcat/trunk/java/javax/el/ELProcessor.java (original) >> +++ tomcat/trunk/java/javax/el/ELProcessor.java Fri Jul 12 13:25:04 2013 >> @@ -74,7 +74,12 @@ public class ELProcessor { >> context, "elProcessor.defineFunctionNullParams")); >> } >> >> - Class<?> clazz = Class.forName(className); >> + // Check the imports >> + Class<?> clazz = context.getImportHandler().resolveClass(className); >> + >> + if (clazz == null) { >> + clazz = Class.forName(className); >> + } > > Is it supposed to be able to load classes that belong to a web application?
It is. > If so, then you need TCCL here. Fixed. Thanks. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
