Author: tmjee Date: Wed Nov 15 16:45:36 2006 New Revision: 475508 URL: http://svn.apache.org/viewvc?view=rev&rev=475508 Log: WW-1506 (Sitemesh plugin's struts-plugin.xml uses incorrect filter for both freemarker and velocity) - reverted back the changes. - can't seems to figure out a way to fix this without changing the container. Thoughts?
Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/BeanSelectionProvider.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/ClasspathConfigurationProvider.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/LegacyPropertiesConfigurationProvider.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/StrutsXmlConfigurationProvider.java struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java struts/struts2/trunk/core/src/test/java/org/apache/struts2/TestConfigurationProvider.java struts/struts2/trunk/core/src/test/java/org/apache/struts2/interceptor/ExecuteAndWaitInterceptorTest.java Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/BeanSelectionProvider.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/BeanSelectionProvider.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/BeanSelectionProvider.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/BeanSelectionProvider.java Wed Nov 15 16:45:36 2006 @@ -20,7 +20,6 @@ */ package org.apache.struts2.config; -import java.util.List; import java.util.Properties; import org.apache.commons.logging.Log; @@ -149,7 +148,7 @@ return false; } - public void register(ContainerBuilder builder, Properties props, List<Class<?>> ignoreFailureStaticInjection) { + public void register(ContainerBuilder builder, Properties props) { alias(ObjectFactory.class, StrutsConstants.STRUTS_OBJECTFACTORY, builder, props); alias(XWorkConverter.class, StrutsConstants.STRUTS_XWORKCONVERTER, builder, props); alias(ActionProxyFactory.class, StrutsConstants.STRUTS_ACTIONPROXYFACTORY, builder, props); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/ClasspathConfigurationProvider.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/ClasspathConfigurationProvider.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/ClasspathConfigurationProvider.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/ClasspathConfigurationProvider.java Wed Nov 15 16:45:36 2006 @@ -25,7 +25,6 @@ import java.net.URL; import java.util.Collections; import java.util.HashMap; -import java.util.List; import java.util.Map; import java.util.Properties; import java.util.Set; @@ -374,7 +373,7 @@ } } - public void register(ContainerBuilder builder, Properties props, List<Class<?>> ignoreFailureStaticInjection) throws ConfigurationException { + public void register(ContainerBuilder builder, Properties props) throws ConfigurationException { // Nothing } } Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/LegacyPropertiesConfigurationProvider.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/LegacyPropertiesConfigurationProvider.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/LegacyPropertiesConfigurationProvider.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/LegacyPropertiesConfigurationProvider.java Wed Nov 15 16:45:36 2006 @@ -24,7 +24,6 @@ import java.net.URL; import java.util.ArrayList; import java.util.Iterator; -import java.util.List; import java.util.Locale; import java.util.Properties; import java.util.StringTokenizer; @@ -61,7 +60,7 @@ return false; } - public void register(ContainerBuilder builder, Properties props, List<Class<?>> ignoreFailureStaticInjection) + public void register(ContainerBuilder builder, Properties props) throws ConfigurationException { final Settings settings = Settings.getInstance(); Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/StrutsXmlConfigurationProvider.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/StrutsXmlConfigurationProvider.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/StrutsXmlConfigurationProvider.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/config/StrutsXmlConfigurationProvider.java Wed Nov 15 16:45:36 2006 @@ -88,16 +88,16 @@ * @see com.opensymphony.xwork2.config.providers.XmlConfigurationProvider#register(com.opensymphony.xwork2.inject.ContainerBuilder, java.util.Properties) */ @Override - public void register(ContainerBuilder containerBuilder, Properties props, List<Class<?>> ignoreFailureStaticInjection) throws ConfigurationException { + public void register(ContainerBuilder containerBuilder, Properties props) throws ConfigurationException { if (servletContext != null && !containerBuilder.contains(ServletContext.class)) { - containerBuilder.factory(ServletContext.class, new Factory<ServletContext>() { - public ServletContext create(Context context) throws Exception { + containerBuilder.factory(ServletContext.class, new Factory() { + public Object create(Context context) throws Exception { return servletContext; } }); } - super.register(containerBuilder, props, ignoreFailureStaticInjection); + super.register(containerBuilder, props); } /* (non-Javadoc) Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java (original) +++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java Wed Nov 15 16:45:36 2006 @@ -329,7 +329,7 @@ public void loadPackages() throws ConfigurationException {} public boolean needsReload() { return false; } - public void register(ContainerBuilder builder, Properties props, List<Class<?>> ignoreFailureStaticInjection) throws ConfigurationException { + public void register(ContainerBuilder builder, Properties props) throws ConfigurationException { props.putAll(initParams); } }); Modified: struts/struts2/trunk/core/src/test/java/org/apache/struts2/TestConfigurationProvider.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/test/java/org/apache/struts2/TestConfigurationProvider.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/test/java/org/apache/struts2/TestConfigurationProvider.java (original) +++ struts/struts2/trunk/core/src/test/java/org/apache/struts2/TestConfigurationProvider.java Wed Nov 15 16:45:36 2006 @@ -163,7 +163,7 @@ return false; } - public void register(ContainerBuilder builder, Properties props, List<Class<?>> ignoreFailureStaticInjection) throws ConfigurationException { + public void register(ContainerBuilder builder, Properties props) throws ConfigurationException { if (!builder.contains(ObjectFactory.class)) { builder.factory(ObjectFactory.class); } Modified: struts/struts2/trunk/core/src/test/java/org/apache/struts2/interceptor/ExecuteAndWaitInterceptorTest.java URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/test/java/org/apache/struts2/interceptor/ExecuteAndWaitInterceptorTest.java?view=diff&rev=475508&r1=475507&r2=475508 ============================================================================== --- struts/struts2/trunk/core/src/test/java/org/apache/struts2/interceptor/ExecuteAndWaitInterceptorTest.java (original) +++ struts/struts2/trunk/core/src/test/java/org/apache/struts2/interceptor/ExecuteAndWaitInterceptorTest.java Wed Nov 15 16:45:36 2006 @@ -232,7 +232,7 @@ configuration.addPackageConfig("", wait); } - public void register(ContainerBuilder builder, Properties props, List<Class<?>> ignoreFailureStaticInjection) throws ConfigurationException { + public void register(ContainerBuilder builder, Properties props) throws ConfigurationException { builder.factory(ObjectFactory.class); builder.factory(ActionProxyFactory.class, DefaultActionProxyFactory.class); }