You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by lu...@apache.org on 2021/01/23 16:47:36 UTC
[struts] 02/02: WW-5113 Drops deprecated struts.xworkTextProvider
This is an automated email from the ASF dual-hosted git repository.
lukaszlenart pushed a commit to branch WW-5113-5114-drop
in repository https://gitbox.apache.org/repos/asf/struts.git
commit f43a8b73bd09b867a8c7de89594019adaf7aa355
Author: Lukasz Lenart <lu...@apache.org>
AuthorDate: Sat Jan 23 17:47:24 2021 +0100
WW-5113 Drops deprecated struts.xworkTextProvider
---
.../java/org/apache/struts2/StrutsConstants.java | 7 -------
.../config/StrutsBeanSelectionProvider.java | 1 -
.../struts2/config/entities/ConstantConfig.java | 14 -------------
.../struts2/config_browser/ShowBeansAction.java | 23 +++++++++++-----------
4 files changed, 11 insertions(+), 34 deletions(-)
diff --git a/core/src/main/java/org/apache/struts2/StrutsConstants.java b/core/src/main/java/org/apache/struts2/StrutsConstants.java
index d4f207f..ccba9b1 100644
--- a/core/src/main/java/org/apache/struts2/StrutsConstants.java
+++ b/core/src/main/java/org/apache/struts2/StrutsConstants.java
@@ -213,13 +213,6 @@ public final class StrutsConstants {
public static final String STRUTS_ALWAYS_SELECT_FULL_NAMESPACE = "struts.mapper.alwaysSelectFullNamespace";
- /**
- * XWork default text provider
- * @deprecated use {@link #STRUTS_TEXT_PROVIDER} instead
- */
- @Deprecated
- public static final String STRUTS_XWORKTEXTPROVIDER = "struts.xworkTextProvider";
-
/** The {@link com.opensymphony.xwork2.LocaleProviderFactory} implementation class */
public static final String STRUTS_LOCALE_PROVIDER_FACTORY = "struts.localeProviderFactory";
diff --git a/core/src/main/java/org/apache/struts2/config/StrutsBeanSelectionProvider.java b/core/src/main/java/org/apache/struts2/config/StrutsBeanSelectionProvider.java
index 2551f77..20389a4 100644
--- a/core/src/main/java/org/apache/struts2/config/StrutsBeanSelectionProvider.java
+++ b/core/src/main/java/org/apache/struts2/config/StrutsBeanSelectionProvider.java
@@ -390,7 +390,6 @@ public class StrutsBeanSelectionProvider extends AbstractBeanSelectionProvider {
alias(TypeConverterCreator.class, StrutsConstants.STRUTS_CONVERTER_CREATOR, builder, props);
alias(TypeConverterHolder.class, StrutsConstants.STRUTS_CONVERTER_HOLDER, builder, props);
- alias(TextProvider.class, StrutsConstants.STRUTS_XWORKTEXTPROVIDER, builder, props, Scope.PROTOTYPE);
alias(TextProvider.class, StrutsConstants.STRUTS_TEXT_PROVIDER, builder, props, Scope.PROTOTYPE);
alias(TextProviderFactory.class, StrutsConstants.STRUTS_TEXT_PROVIDER_FACTORY, builder, props, Scope.PROTOTYPE);
alias(LocaleProviderFactory.class, StrutsConstants.STRUTS_LOCALE_PROVIDER_FACTORY, builder, props);
diff --git a/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java b/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java
index ac0cf0d..7729c8d 100644
--- a/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java
+++ b/core/src/main/java/org/apache/struts2/config/entities/ConstantConfig.java
@@ -87,7 +87,6 @@ public class ConstantConfig {
private Boolean freemarkerWrapperAltMap;
private BeanConfig xworkConverter;
private Boolean mapperAlwaysSelectFullNamespace;
- private BeanConfig xworkTextProvider;
private BeanConfig localeProviderFactory;
private String mapperIdParameterName;
private Boolean ognlAllowStaticFieldAccess;
@@ -217,7 +216,6 @@ public class ConstantConfig {
map.put(StrutsConstants.STRUTS_FREEMARKER_WRAPPER_ALT_MAP, Objects.toString(freemarkerWrapperAltMap, null));
map.put(StrutsConstants.STRUTS_XWORKCONVERTER, beanConfToString(xworkConverter));
map.put(StrutsConstants.STRUTS_ALWAYS_SELECT_FULL_NAMESPACE, Objects.toString(mapperAlwaysSelectFullNamespace, null));
- map.put(StrutsConstants.STRUTS_XWORKTEXTPROVIDER, beanConfToString(xworkTextProvider));
map.put(StrutsConstants.STRUTS_LOCALE_PROVIDER_FACTORY, beanConfToString(localeProviderFactory));
map.put(StrutsConstants.STRUTS_ID_PARAMETER_NAME, mapperIdParameterName);
map.put(StrutsConstants.STRUTS_ALLOW_STATIC_FIELD_ACCESS, Objects.toString(ognlAllowStaticFieldAccess, null));
@@ -778,18 +776,6 @@ public class ConstantConfig {
this.mapperAlwaysSelectFullNamespace = mapperAlwaysSelectFullNamespace;
}
- public BeanConfig getXworkTextProvider() {
- return xworkTextProvider;
- }
-
- public void setXworkTextProvider(BeanConfig xworkTextProvider) {
- this.xworkTextProvider = xworkTextProvider;
- }
-
- public void setXworkTextProvider(Class<?> clazz) {
- this.xworkTextProvider = new BeanConfig(clazz, clazz.getName());
- }
-
public BeanConfig getLocaleProviderFactory() {
return localeProviderFactory;
}
diff --git a/plugins/config-browser/src/main/java/org/apache/struts2/config_browser/ShowBeansAction.java b/plugins/config-browser/src/main/java/org/apache/struts2/config_browser/ShowBeansAction.java
index 8827fd8..8e97a87 100644
--- a/plugins/config-browser/src/main/java/org/apache/struts2/config_browser/ShowBeansAction.java
+++ b/plugins/config-browser/src/main/java/org/apache/struts2/config_browser/ShowBeansAction.java
@@ -20,7 +20,6 @@ package org.apache.struts2.config_browser;
import com.opensymphony.xwork2.ActionProxyFactory;
import com.opensymphony.xwork2.ObjectFactory;
-import com.opensymphony.xwork2.TextProvider;
import com.opensymphony.xwork2.conversion.ObjectTypeDeterminer;
import com.opensymphony.xwork2.conversion.impl.XWorkConverter;
import com.opensymphony.xwork2.inject.Container;
@@ -49,10 +48,9 @@ public class ShowBeansAction extends ActionNamesAction {
@Inject
public void setContainer(Container container) {
super.setContainer(container);
- bindings = new TreeMap<String, Set<Binding>>();
+ bindings = new TreeMap<>();
bindings.put(ObjectFactory.class.getName(), addBindings(container, ObjectFactory.class, StrutsConstants.STRUTS_OBJECTFACTORY));
bindings.put(XWorkConverter.class.getName(), addBindings(container, XWorkConverter.class, StrutsConstants.STRUTS_XWORKCONVERTER));
- bindings.put(TextProvider.class.getName(), addBindings(container, TextProvider.class, StrutsConstants.STRUTS_XWORKTEXTPROVIDER));
bindings.put(ActionProxyFactory.class.getName(), addBindings(container, ActionProxyFactory.class, StrutsConstants.STRUTS_ACTIONPROXYFACTORY));
bindings.put(ObjectTypeDeterminer.class.getName(), addBindings(container, ObjectTypeDeterminer.class, StrutsConstants.STRUTS_OBJECTTYPEDETERMINER));
bindings.put(ActionMapper.class.getName(), addBindings(container, ActionMapper.class, StrutsConstants.STRUTS_MAPPER_CLASS));
@@ -66,8 +64,8 @@ public class ShowBeansAction extends ActionNamesAction {
return bindings;
}
- protected Set<Binding> addBindings(Container container, Class type, String constName) {
- Set<Binding> bindings = new TreeSet<Binding>();
+ protected Set<Binding> addBindings(Container container, Class<?> type, String constName) {
+ Set<Binding> bindings = new TreeSet<>();
String chosenName = container.getInstance(String.class, constName);
if (chosenName == null) {
chosenName = "struts";
@@ -84,7 +82,7 @@ public class ShowBeansAction extends ActionNamesAction {
return bindings;
}
- String getInstanceClassName(Container container, Class type, String name) {
+ String getInstanceClassName(Container container, Class<?> type, String name) {
String instName = "Class unable to be loaded";
try {
Object inst = container.getInstance(type, name);
@@ -95,11 +93,12 @@ public class ShowBeansAction extends ActionNamesAction {
return instName;
}
- public class Binding implements Comparable<Binding> {
- private String impl;
- private String alias;
- private String constant;
- private boolean isDefault;
+ public static class Binding implements Comparable<Binding> {
+
+ private final String impl;
+ private final String alias;
+ private final String constant;
+ private final boolean isDefault;
public Binding(String impl, String alias, String constant, boolean def) {
this.impl = impl;
@@ -125,7 +124,7 @@ public class ShowBeansAction extends ActionNamesAction {
}
public int compareTo(Binding b2) {
- int ret = 0;
+ int ret;
if (isDefault) {
ret = -1;
} else if (b2.isDefault()) {