From 8a0cdb4da15a758329b2b6d78cdb5a9fde0e59ae Mon Sep 17 00:00:00 2001 From: Lukasz Lenart Date: Sun, 29 Nov 2020 16:00:51 +0100 Subject: [PATCH] WW-5078 Uses more informative name for XWorkConfigurationProvider --- .../xwork2/config/ConfigurationManager.java | 8 ++++---- .../xwork2/config/impl/MockConfiguration.java | 4 ++-- ...er.java => StrutsDefaultConfigurationProvider.java} | 2 +- .../opensymphony/xwork2/util/XWorkTestCaseHelper.java | 6 +++--- .../xwork2/config/ConfigurationManagerTest.java | 4 ++-- ...nfigurationProviderGlobalResultInheritenceTest.java | 2 +- ...gurationProviderInterceptorParamOverridingTest.java | 2 +- ...ionProviderInterceptorStackParamOverridingTest.java | 2 +- .../xwork2/interceptor/ParametersInterceptorTest.java | 10 +++++----- 9 files changed, 20 insertions(+), 20 deletions(-) rename core/src/main/java/com/opensymphony/xwork2/config/providers/{XWorkConfigurationProvider.java => StrutsDefaultConfigurationProvider.java} (99%) diff --git a/core/src/main/java/com/opensymphony/xwork2/config/ConfigurationManager.java b/core/src/main/java/com/opensymphony/xwork2/config/ConfigurationManager.java index c5dcae192..41e24b1b3 100644 --- a/core/src/main/java/com/opensymphony/xwork2/config/ConfigurationManager.java +++ b/core/src/main/java/com/opensymphony/xwork2/config/ConfigurationManager.java @@ -19,7 +19,7 @@ package com.opensymphony.xwork2.config; import com.opensymphony.xwork2.config.impl.DefaultConfiguration; -import com.opensymphony.xwork2.config.providers.XWorkConfigurationProvider; +import com.opensymphony.xwork2.config.providers.StrutsDefaultConfigurationProvider; import org.apache.logging.log4j.LogManager; import org.apache.logging.log4j.Logger; import org.apache.struts2.StrutsConstants; @@ -85,8 +85,8 @@ public class ConfigurationManager { /** *

* Get the current list of ConfigurationProviders. If no custom ConfigurationProviders have been added, this method - * will return a list containing only the default ConfigurationProvider, XMLConfigurationProvider. If a custom - * ConfigurationProvider has been added, then the XmlConfigurationProvider must be added by hand. + * will return a list containing only a default ConfigurationProvider, {@link StrutsDefaultConfigurationProvider}. + * If a custom ConfigurationProvider has been added, then the StrutsDefaultConfigurationProvider must be added by hand. *

* *

@@ -100,7 +100,7 @@ public class ConfigurationManager { providerLock.lock(); try { if (containerProviders.size() == 0) { - containerProviders.add(new XWorkConfigurationProvider()); + containerProviders.add(new StrutsDefaultConfigurationProvider()); } return containerProviders; diff --git a/core/src/main/java/com/opensymphony/xwork2/config/impl/MockConfiguration.java b/core/src/main/java/com/opensymphony/xwork2/config/impl/MockConfiguration.java index b21dc45cd..997c5f1de 100644 --- a/core/src/main/java/com/opensymphony/xwork2/config/impl/MockConfiguration.java +++ b/core/src/main/java/com/opensymphony/xwork2/config/impl/MockConfiguration.java @@ -21,7 +21,7 @@ package com.opensymphony.xwork2.config.impl; import com.opensymphony.xwork2.config.*; import com.opensymphony.xwork2.config.entities.PackageConfig; import com.opensymphony.xwork2.config.entities.UnknownHandlerConfig; -import com.opensymphony.xwork2.config.providers.XWorkConfigurationProvider; +import com.opensymphony.xwork2.config.providers.StrutsDefaultConfigurationProvider; import com.opensymphony.xwork2.inject.Container; import com.opensymphony.xwork2.inject.ContainerBuilder; import com.opensymphony.xwork2.inject.Scope; @@ -50,7 +50,7 @@ public class MockConfiguration implements Configuration { //this cannot be done in the constructor, as it causes an infinite loop builder.factory(Configuration.class, MockConfiguration.class, Scope.SINGLETON); LocatableProperties props = new LocatableProperties(); - new XWorkConfigurationProvider().register(builder, props); + new StrutsDefaultConfigurationProvider().register(builder, props); builder.constant(StrutsConstants.STRUTS_DEVMODE, "false"); builder.constant(StrutsConstants.STRUTS_CONFIGURATION_XML_RELOAD, "true"); builder.constant(StrutsConstants.STRUTS_ENABLE_OGNL_EXPRESSION_CACHE, "true"); diff --git a/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java b/core/src/main/java/com/opensymphony/xwork2/config/providers/StrutsDefaultConfigurationProvider.java similarity index 99% rename from core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java rename to core/src/main/java/com/opensymphony/xwork2/config/providers/StrutsDefaultConfigurationProvider.java index 63d8fab40..ccdb78202 100644 --- a/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java +++ b/core/src/main/java/com/opensymphony/xwork2/config/providers/StrutsDefaultConfigurationProvider.java @@ -123,7 +123,7 @@ import java.util.List; import java.util.Map; import java.util.Set; -public class XWorkConfigurationProvider implements ConfigurationProvider { +public class StrutsDefaultConfigurationProvider implements ConfigurationProvider { public void destroy() { } diff --git a/core/src/main/java/com/opensymphony/xwork2/util/XWorkTestCaseHelper.java b/core/src/main/java/com/opensymphony/xwork2/util/XWorkTestCaseHelper.java index bde73ca92..e139f0535 100644 --- a/core/src/main/java/com/opensymphony/xwork2/util/XWorkTestCaseHelper.java +++ b/core/src/main/java/com/opensymphony/xwork2/util/XWorkTestCaseHelper.java @@ -20,7 +20,7 @@ package com.opensymphony.xwork2.util; import com.opensymphony.xwork2.ActionContext; import com.opensymphony.xwork2.config.*; -import com.opensymphony.xwork2.config.providers.XWorkConfigurationProvider; +import com.opensymphony.xwork2.config.providers.StrutsDefaultConfigurationProvider; import com.opensymphony.xwork2.config.providers.XmlConfigurationProvider; import com.opensymphony.xwork2.inject.Container; import com.opensymphony.xwork2.inject.ContainerBuilder; @@ -33,7 +33,7 @@ public class XWorkTestCaseHelper { public static ConfigurationManager setUp() throws Exception { ConfigurationManager configurationManager = new ConfigurationManager(Container.DEFAULT_NAME); - configurationManager.addContainerProvider(new XWorkConfigurationProvider()); + configurationManager.addContainerProvider(new StrutsDefaultConfigurationProvider()); Configuration config = configurationManager.getConfiguration(); Container container = config.getContainer(); @@ -62,7 +62,7 @@ public class XWorkTestCaseHelper { } }); - configurationManager.addContainerProvider(new XWorkConfigurationProvider()); + configurationManager.addContainerProvider(new StrutsDefaultConfigurationProvider()); for (ConfigurationProvider prov : providers) { if (prov instanceof XmlConfigurationProvider) { ((XmlConfigurationProvider)prov).setThrowExceptionOnDuplicateBeans(false); diff --git a/core/src/test/java/com/opensymphony/xwork2/config/ConfigurationManagerTest.java b/core/src/test/java/com/opensymphony/xwork2/config/ConfigurationManagerTest.java index 864f790d5..307cd3be0 100644 --- a/core/src/test/java/com/opensymphony/xwork2/config/ConfigurationManagerTest.java +++ b/core/src/test/java/com/opensymphony/xwork2/config/ConfigurationManagerTest.java @@ -22,7 +22,7 @@ import com.mockobjects.dynamic.C; import com.mockobjects.dynamic.Mock; import com.opensymphony.xwork2.FileManagerFactory; import com.opensymphony.xwork2.XWorkTestCase; -import com.opensymphony.xwork2.config.providers.XWorkConfigurationProvider; +import com.opensymphony.xwork2.config.providers.StrutsDefaultConfigurationProvider; import com.opensymphony.xwork2.conversion.TypeConverterHolder; import com.opensymphony.xwork2.inject.Container; import com.opensymphony.xwork2.inject.ContainerBuilder; @@ -168,7 +168,7 @@ public class ConfigurationManagerTest extends XWorkTestCase { configProviderMock.matchAndReturn("equals", C.ANY_ARGS, false); ConfigurationProvider mockProvider = (ConfigurationProvider) configProviderMock.proxy(); - configurationManager.addContainerProvider(new XWorkConfigurationProvider()); + configurationManager.addContainerProvider(new StrutsDefaultConfigurationProvider()); configurationManager.addContainerProvider(mockProvider); //the first time it always inits diff --git a/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderGlobalResultInheritenceTest.java b/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderGlobalResultInheritenceTest.java index a40c71497..a5c12f31c 100644 --- a/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderGlobalResultInheritenceTest.java +++ b/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderGlobalResultInheritenceTest.java @@ -39,7 +39,7 @@ public class XmlConfigurationProviderGlobalResultInheritenceTest extends Configu ConfigurationProvider provider = buildConfigurationProvider("com/opensymphony/xwork2/config/providers/xwork-test-global-result-inheritence.xml"); ConfigurationManager configurationManager = new ConfigurationManager(Container.DEFAULT_NAME); - configurationManager.addContainerProvider(new XWorkConfigurationProvider()); + configurationManager.addContainerProvider(new StrutsDefaultConfigurationProvider()); configurationManager.addContainerProvider(provider); Configuration configuration = configurationManager.getConfiguration(); diff --git a/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorParamOverridingTest.java b/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorParamOverridingTest.java index 6b6057b12..aab4a50ca 100644 --- a/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorParamOverridingTest.java +++ b/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorParamOverridingTest.java @@ -46,7 +46,7 @@ public class XmlConfigurationProviderInterceptorParamOverridingTest extends XWor p.setFileManagerFactory(factory); conf.reloadContainer(new ArrayList() { { - add(new XWorkConfigurationProvider()); + add(new StrutsDefaultConfigurationProvider()); add(p); } }); diff --git a/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorStackParamOverridingTest.java b/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorStackParamOverridingTest.java index 401c66f16..e629d4486 100644 --- a/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorStackParamOverridingTest.java +++ b/core/src/test/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProviderInterceptorStackParamOverridingTest.java @@ -49,7 +49,7 @@ public class XmlConfigurationProviderInterceptorStackParamOverridingTest extends configurationManager.addContainerProvider(p); conf.reloadContainer(new ArrayList() { { - add(new XWorkConfigurationProvider()); + add(new StrutsDefaultConfigurationProvider()); add(p); } }); diff --git a/core/src/test/java/com/opensymphony/xwork2/interceptor/ParametersInterceptorTest.java b/core/src/test/java/com/opensymphony/xwork2/interceptor/ParametersInterceptorTest.java index 2eefbde4a..381f909d3 100644 --- a/core/src/test/java/com/opensymphony/xwork2/interceptor/ParametersInterceptorTest.java +++ b/core/src/test/java/com/opensymphony/xwork2/interceptor/ParametersInterceptorTest.java @@ -28,7 +28,7 @@ import com.opensymphony.xwork2.TextProvider; import com.opensymphony.xwork2.XWorkTestCase; import com.opensymphony.xwork2.config.entities.ActionConfig; import com.opensymphony.xwork2.config.providers.MockConfigurationProvider; -import com.opensymphony.xwork2.config.providers.XWorkConfigurationProvider; +import com.opensymphony.xwork2.config.providers.StrutsDefaultConfigurationProvider; import com.opensymphony.xwork2.config.providers.XmlConfigurationProvider; import com.opensymphony.xwork2.conversion.impl.XWorkConverter; import com.opensymphony.xwork2.mock.MockActionInvocation; @@ -95,7 +95,7 @@ public class ParametersInterceptorTest extends XWorkTestCase { public void testInsecureParameters() throws Exception { // given - loadConfigurationProviders(new XWorkConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-param-test.xml")); + loadConfigurationProviders(new StrutsDefaultConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-param-test.xml")); final Map params = new HashMap() { { put("name", "(#context[\"xwork.MethodAccessor.denyMethodExecution\"]= new " + @@ -132,7 +132,7 @@ public class ParametersInterceptorTest extends XWorkTestCase { final String pollution1 = "class.classLoader.jarPath"; final String pollution2 = "model.class.classLoader.jarPath"; - loadConfigurationProviders(new XWorkConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-param-test.xml")); + loadConfigurationProviders(new StrutsDefaultConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-param-test.xml")); final Map params = new HashMap() { { put(pollution1, "bad"); @@ -171,7 +171,7 @@ public class ParametersInterceptorTest extends XWorkTestCase { final String pollution2 = "model.class.classLoader.jarPath"; final String pollution3 = "class.classLoader.defaultAssertionStatus"; - loadConfigurationProviders(new XWorkConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-class-param-test.xml")); + loadConfigurationProviders(new StrutsDefaultConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-class-param-test.xml")); final Map params = new HashMap() { { put(pollution1, "bad"); @@ -290,7 +290,7 @@ public class ParametersInterceptorTest extends XWorkTestCase { final String pollution5 = "class['classLoader']['jarPath']"; final String pollution6 = "class[\"classLoader\"]['jarPath']"; - loadConfigurationProviders(new XWorkConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-param-test.xml")); + loadConfigurationProviders(new StrutsDefaultConfigurationProvider(), new StrutsXmlConfigurationProvider("xwork-param-test.xml")); final Map params = new HashMap() { { put(pollution1, "bad");