Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id DE703200C3D for ; Tue, 14 Mar 2017 19:35:57 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id DD053160B91; Tue, 14 Mar 2017 18:35:57 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 946CE160B7E for ; Tue, 14 Mar 2017 19:35:56 +0100 (CET) Received: (qmail 92294 invoked by uid 500); 14 Mar 2017 18:35:55 -0000 Mailing-List: contact commits-help@struts.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@struts.apache.org Delivered-To: mailing list commits@struts.apache.org Received: (qmail 92188 invoked by uid 99); 14 Mar 2017 18:35:55 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 14 Mar 2017 18:35:55 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 0B497DFFC2; Tue, 14 Mar 2017 18:35:55 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: lukaszlenart@apache.org To: commits@struts.apache.org Date: Tue, 14 Mar 2017 18:35:58 -0000 Message-Id: <0bc5c4f4cdf34e84802e5aaf397bd439@git.apache.org> In-Reply-To: References: X-Mailer: ASF-Git Admin Mailer Subject: [05/24] struts git commit: Injects instead of static reference archived-at: Tue, 14 Mar 2017 18:35:58 -0000 Injects instead of static reference Project: http://git-wip-us.apache.org/repos/asf/struts/repo Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/4ef64185 Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/4ef64185 Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/4ef64185 Branch: refs/heads/master Commit: 4ef641850539557ffbc9412ff1d6627a8e08fd32 Parents: 239327e Author: Lukasz Lenart Authored: Wed Mar 8 17:43:20 2017 +0100 Committer: Lukasz Lenart Committed: Wed Mar 8 17:43:20 2017 +0100 ---------------------------------------------------------------------- .../opensymphony/xwork2/DefaultActionProxy.java | 13 +++++++--- .../xwork2/DefaultTextProvider.java | 16 +++++++++--- .../xwork2/TextProviderSupport.java | 27 ++++++++++++-------- .../xwork2/conversion/impl/XWorkConverter.java | 3 ++- .../xwork2/interceptor/AliasInterceptor.java | 14 +++++++--- .../interceptor/ParametersInterceptor.java | 19 +++++++++----- .../StaticParametersInterceptor.java | 11 ++++++-- .../org/apache/struts2/components/Date.java | 22 ++++++++-------- .../org/apache/struts2/components/I18n.java | 2 +- .../apache/struts2/dispatcher/Dispatcher.java | 10 +------- .../multipart/JakartaMultiPartRequest.java | 6 +---- .../struts2/factory/StrutsActionProxy.java | 2 +- .../org/apache/struts2/util/TokenHelper.java | 3 ++- .../multipart/PellMultiPartRequest.java | 6 +---- 14 files changed, 90 insertions(+), 64 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/DefaultActionProxy.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/DefaultActionProxy.java b/core/src/main/java/com/opensymphony/xwork2/DefaultActionProxy.java index 2d8bc38..39a484d 100644 --- a/core/src/main/java/com/opensymphony/xwork2/DefaultActionProxy.java +++ b/core/src/main/java/com/opensymphony/xwork2/DefaultActionProxy.java @@ -53,6 +53,8 @@ public class DefaultActionProxy implements ActionProxy, Serializable { protected ActionConfig config; protected ActionInvocation invocation; protected UnknownHandlerManager unknownHandlerManager; + protected LocalizedTextUtil localizedTextUtil; + protected String actionName; protected String namespace; protected String method; @@ -113,6 +115,11 @@ public class DefaultActionProxy implements ActionProxy, Serializable { this.actionEventListener = listener; } + @Inject + public void setLocalizedTextUtil(LocalizedTextUtil localizedTextUtil) { + this.localizedTextUtil = localizedTextUtil; + } + public Object getAction() { return invocation.getAction(); } @@ -205,7 +212,7 @@ public class DefaultActionProxy implements ActionProxy, Serializable { } protected String prepareNotAllowedErrorMessage() { - return LocalizedTextUtil.findDefaultText( + return localizedTextUtil.findDefaultText( "struts.exception.method-not-allowed", Locale.getDefault(), new String[]{method, actionName} @@ -214,12 +221,12 @@ public class DefaultActionProxy implements ActionProxy, Serializable { protected String getErrorMessage() { if ((namespace != null) && (namespace.trim().length() > 0)) { - return LocalizedTextUtil.findDefaultText( + return localizedTextUtil.findDefaultText( "xwork.exception.missing-package-action", Locale.getDefault(), new String[]{namespace, actionName}); } else { - return LocalizedTextUtil.findDefaultText( + return localizedTextUtil.findDefaultText( "xwork.exception.missing-action", Locale.getDefault(), new String[]{actionName}); http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/DefaultTextProvider.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/DefaultTextProvider.java b/core/src/main/java/com/opensymphony/xwork2/DefaultTextProvider.java index 1d72161..0e3f5b1 100644 --- a/core/src/main/java/com/opensymphony/xwork2/DefaultTextProvider.java +++ b/core/src/main/java/com/opensymphony/xwork2/DefaultTextProvider.java @@ -15,6 +15,7 @@ */ package com.opensymphony.xwork2; +import com.opensymphony.xwork2.inject.Inject; import com.opensymphony.xwork2.util.LocalizedTextUtil; import com.opensymphony.xwork2.util.ValueStack; @@ -37,15 +38,22 @@ public class DefaultTextProvider implements TextProvider, Serializable, Unchaina private static final Object[] EMPTY_ARGS = new Object[0]; + protected LocalizedTextUtil localizedTextUtil; + public DefaultTextProvider() { } + @Inject + public void setLocalizedTextUtil(LocalizedTextUtil localizedTextUtil) { + this.localizedTextUtil = localizedTextUtil; + } + public boolean hasKey(String key) { return getText(key) != null; } public String getText(String key) { - return LocalizedTextUtil.findDefaultText(key, ActionContext.getContext().getLocale()); + return localizedTextUtil.findDefaultText(key, ActionContext.getContext().getLocale()); } public String getText(String key, String defaultValue) { @@ -64,7 +72,7 @@ public class DefaultTextProvider implements TextProvider, Serializable, Unchaina params = EMPTY_ARGS; } - return LocalizedTextUtil.findDefaultText(key, ActionContext.getContext().getLocale(), params); + return localizedTextUtil.findDefaultText(key, ActionContext.getContext().getLocale(), params); } public String getText(String key, String[] args) { @@ -75,7 +83,7 @@ public class DefaultTextProvider implements TextProvider, Serializable, Unchaina params = EMPTY_ARGS; } - return LocalizedTextUtil.findDefaultText(key, ActionContext.getContext().getLocale(), params); + return localizedTextUtil.findDefaultText(key, ActionContext.getContext().getLocale(), params); } public String getText(String key, String defaultValue, List args) { @@ -136,7 +144,7 @@ public class DefaultTextProvider implements TextProvider, Serializable, Unchaina } public ResourceBundle getTexts(String bundleName) { - return LocalizedTextUtil.findResourceBundle(bundleName, ActionContext.getContext().getLocale()); + return localizedTextUtil.findResourceBundle(bundleName, ActionContext.getContext().getLocale()); } public ResourceBundle getTexts() { http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/TextProviderSupport.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/TextProviderSupport.java b/core/src/main/java/com/opensymphony/xwork2/TextProviderSupport.java index 3f20902..ab37e92 100644 --- a/core/src/main/java/com/opensymphony/xwork2/TextProviderSupport.java +++ b/core/src/main/java/com/opensymphony/xwork2/TextProviderSupport.java @@ -33,6 +33,7 @@ public class TextProviderSupport implements ResourceBundleTextProvider { private Class clazz; private LocaleProvider localeProvider; private ResourceBundle bundle; + private LocalizedTextUtil localizedTextUtil; /** * Default constructor @@ -85,6 +86,10 @@ public class TextProviderSupport implements ResourceBundleTextProvider { this.localeProvider = localeProvider; } + @Inject + public void setLocalizedTextUtil(LocalizedTextUtil localizedTextUtil) { + this.localizedTextUtil = localizedTextUtil; + } /** * Checks if a key is available in the resource bundles associated with this action. @@ -97,9 +102,9 @@ public class TextProviderSupport implements ResourceBundleTextProvider { public boolean hasKey(String key) { String message; if (clazz != null) { - message = LocalizedTextUtil.findText(clazz, key, getLocale(), null, new Object[0] ); + message = localizedTextUtil.findText(clazz, key, getLocale(), null, new Object[0] ); } else { - message = LocalizedTextUtil.findText(bundle, key, getLocale(), null, new Object[0]); + message = localizedTextUtil.findText(bundle, key, getLocale(), null, new Object[0]); } return message != null; } @@ -201,9 +206,9 @@ public class TextProviderSupport implements ResourceBundleTextProvider { public String getText(String key, String defaultValue, List args) { Object[] argsArray = ((args != null && !args.equals(Collections.emptyList())) ? args.toArray() : null); if (clazz != null) { - return LocalizedTextUtil.findText(clazz, key, getLocale(), defaultValue, argsArray); + return localizedTextUtil.findText(clazz, key, getLocale(), defaultValue, argsArray); } else { - return LocalizedTextUtil.findText(bundle, key, getLocale(), defaultValue, argsArray); + return localizedTextUtil.findText(bundle, key, getLocale(), defaultValue, argsArray); } } @@ -222,9 +227,9 @@ public class TextProviderSupport implements ResourceBundleTextProvider { */ public String getText(String key, String defaultValue, String[] args) { if (clazz != null) { - return LocalizedTextUtil.findText(clazz, key, getLocale(), defaultValue, args); + return localizedTextUtil.findText(clazz, key, getLocale(), defaultValue, args); } else { - return LocalizedTextUtil.findText(bundle, key, getLocale(), defaultValue, args); + return localizedTextUtil.findText(bundle, key, getLocale(), defaultValue, args); } } @@ -252,9 +257,9 @@ public class TextProviderSupport implements ResourceBundleTextProvider { locale = getLocale(); } if (clazz != null) { - return LocalizedTextUtil.findText(clazz, key, locale, defaultValue, argsArray, stack); + return localizedTextUtil.findText(clazz, key, locale, defaultValue, argsArray, stack); } else { - return LocalizedTextUtil.findText(bundle, key, locale, defaultValue, argsArray, stack); + return localizedTextUtil.findText(bundle, key, locale, defaultValue, argsArray, stack); } } @@ -282,9 +287,9 @@ public class TextProviderSupport implements ResourceBundleTextProvider { locale = getLocale(); } if (clazz != null) { - return LocalizedTextUtil.findText(clazz, key, locale, defaultValue, args, stack); + return localizedTextUtil.findText(clazz, key, locale, defaultValue, args, stack); } else { - return LocalizedTextUtil.findText(bundle, key, locale, defaultValue, args, stack); + return localizedTextUtil.findText(bundle, key, locale, defaultValue, args, stack); } } @@ -304,7 +309,7 @@ public class TextProviderSupport implements ResourceBundleTextProvider { * @return a resource bundle */ public ResourceBundle getTexts(String aBundleName) { - return LocalizedTextUtil.findResourceBundle(aBundleName, getLocale()); + return localizedTextUtil.findResourceBundle(aBundleName, getLocale()); } /** http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/conversion/impl/XWorkConverter.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/conversion/impl/XWorkConverter.java b/core/src/main/java/com/opensymphony/xwork2/conversion/impl/XWorkConverter.java index 9dfcd4d..771fc36 100644 --- a/core/src/main/java/com/opensymphony/xwork2/conversion/impl/XWorkConverter.java +++ b/core/src/main/java/com/opensymphony/xwork2/conversion/impl/XWorkConverter.java @@ -194,7 +194,8 @@ public class XWorkConverter extends DefaultTypeConverter { } public static String getConversionErrorMessage(String propertyName, ValueStack stack) { - String defaultMessage = LocalizedTextUtil.findDefaultText("xwork.default.invalid.fieldvalue", + LocalizedTextUtil localizedTextUtil = ActionContext.getContext().getContainer().getInstance(LocalizedTextUtil.class); + String defaultMessage = localizedTextUtil.findDefaultText("xwork.default.invalid.fieldvalue", ActionContext.getContext().getLocale(), new Object[]{ propertyName http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/interceptor/AliasInterceptor.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/interceptor/AliasInterceptor.java b/core/src/main/java/com/opensymphony/xwork2/interceptor/AliasInterceptor.java index a956aea..9d260c1 100644 --- a/core/src/main/java/com/opensymphony/xwork2/interceptor/AliasInterceptor.java +++ b/core/src/main/java/com/opensymphony/xwork2/interceptor/AliasInterceptor.java @@ -94,11 +94,12 @@ public class AliasInterceptor extends AbstractInterceptor { protected String aliasesKey = DEFAULT_ALIAS_KEY; protected ValueStackFactory valueStackFactory; - static boolean devMode = false; + protected LocalizedTextUtil localizedTextUtil; + protected boolean devMode = false; @Inject(XWorkConstants.DEV_MODE) - public static void setDevMode(String mode) { - devMode = "true".equals(mode); + public void setDevMode(String mode) { + this.devMode = Boolean.parseBoolean(mode); } @Inject @@ -106,6 +107,11 @@ public class AliasInterceptor extends AbstractInterceptor { this.valueStackFactory = valueStackFactory; } + @Inject + public void setLocalizedTextUtil(LocalizedTextUtil localizedTextUtil) { + this.localizedTextUtil = localizedTextUtil; + } + /** *

* Sets the name of the action parameter to look for the alias map. @@ -173,7 +179,7 @@ public class AliasInterceptor extends AbstractInterceptor { newStack.setValue(alias, value.get()); } catch (RuntimeException e) { if (devMode) { - String developerNotification = LocalizedTextUtil.findText(ParametersInterceptor.class, "devmode.notification", ActionContext.getContext().getLocale(), "Developer Notification:\n{0}", new Object[]{ + String developerNotification = localizedTextUtil.findText(ParametersInterceptor.class, "devmode.notification", ActionContext.getContext().getLocale(), "Developer Notification:\n{0}", new Object[]{ "Unexpected Exception caught setting '" + entry.getKey() + "' on '" + action.getClass() + ": " + e.getMessage() }); LOG.error(developerNotification); http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java b/core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java index c1c52d9..5eba95d 100644 --- a/core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java +++ b/core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java @@ -17,6 +17,7 @@ package com.opensymphony.xwork2.interceptor; import com.opensymphony.xwork2.ActionContext; import com.opensymphony.xwork2.ActionInvocation; +import com.opensymphony.xwork2.TextProvider; import com.opensymphony.xwork2.XWorkConstants; import com.opensymphony.xwork2.inject.Inject; import com.opensymphony.xwork2.security.AcceptedPatternsChecker; @@ -218,15 +219,19 @@ public class ParametersInterceptor extends MethodFilterInterceptor { } protected void notifyDeveloperParameterException(Object action, String property, String message) { - String developerNotification = LocalizedTextUtil.findText(ParametersInterceptor.class, "devmode.notification", - ActionContext.getContext().getLocale(), "Developer Notification:\n{0}", - new Object[]{ - "Unexpected Exception caught setting '" + property + "' on '" + action.getClass() + ": " + message - } - ); + String developerNotification = "Unexpected Exception caught setting '" + property + "' on '" + action.getClass() + ": " + message; + if (action instanceof TextProvider) { + TextProvider tp = (TextProvider) action; + developerNotification = tp.getText("devmode.notification", + "Developer Notification:\n{0}", + new String[]{ developerNotification } + ); + } + LOG.error(developerNotification); - // see https://issues.apache.org/jira/browse/WW-4066 + if (action instanceof ValidationAware) { + // see https://issues.apache.org/jira/browse/WW-4066 Collection messages = ((ValidationAware) action).getActionMessages(); messages.add(message); ((ValidationAware) action).setActionMessages(messages); http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/com/opensymphony/xwork2/interceptor/StaticParametersInterceptor.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/interceptor/StaticParametersInterceptor.java b/core/src/main/java/com/opensymphony/xwork2/interceptor/StaticParametersInterceptor.java index 8aab382..b697a64 100644 --- a/core/src/main/java/com/opensymphony/xwork2/interceptor/StaticParametersInterceptor.java +++ b/core/src/main/java/com/opensymphony/xwork2/interceptor/StaticParametersInterceptor.java @@ -93,6 +93,7 @@ public class StaticParametersInterceptor extends AbstractInterceptor { private static final Logger LOG = LogManager.getLogger(StaticParametersInterceptor.class); private ValueStackFactory valueStackFactory; + private LocalizedTextUtil localizedTextUtil; @Inject public void setValueStackFactory(ValueStackFactory valueStackFactory) { @@ -102,7 +103,12 @@ public class StaticParametersInterceptor extends AbstractInterceptor { @Inject(XWorkConstants.DEV_MODE) public void setDevMode(String mode) { devMode = BooleanUtils.toBoolean(mode); - } + } + + @Inject + public void setLocalizedTextUtil(LocalizedTextUtil localizedTextUtil) { + this.localizedTextUtil = localizedTextUtil; + } public void setParse(String value) { this.parse = BooleanUtils.toBoolean(value); @@ -168,7 +174,8 @@ public class StaticParametersInterceptor extends AbstractInterceptor { newStack.setValue(entry.getKey(), val); } catch (RuntimeException e) { if (devMode) { - String developerNotification = LocalizedTextUtil.findText(ParametersInterceptor.class, "devmode.notification", ActionContext.getContext().getLocale(), "Developer Notification:\n{0}", new Object[]{ + + String developerNotification = localizedTextUtil.findText(ParametersInterceptor.class, "devmode.notification", ActionContext.getContext().getLocale(), "Developer Notification:\n{0}", new Object[]{ "Unexpected Exception caught setting '" + entry.getKey() + "' on '" + action.getClass() + ": " + e.getMessage() }); LOG.error(developerNotification); http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/org/apache/struts2/components/Date.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/components/Date.java b/core/src/main/java/org/apache/struts2/components/Date.java index ac3d68b..46c193d 100644 --- a/core/src/main/java/org/apache/struts2/components/Date.java +++ b/core/src/main/java/org/apache/struts2/components/Date.java @@ -294,16 +294,18 @@ public class Date extends ContextBean { date = new java.util.Date((long) dateObject); } else { if (devMode) { - String developerNotification = LocalizedTextUtil.findText( - Date.class, - "devmode.notification", - ActionContext.getContext().getLocale(), - "Developer Notification:\n{0}", - new Object[]{ - "Expression [" + name + "] passed to tag which was evaluated to [" + dateObject + "](" - + (dateObject != null ? dateObject.getClass() : "null") + ") isn't instance of java.util.Date nor java.util.Calendar nor long!" - } - ); + TextProvider tp = findProviderInStack(); + String developerNotification = ""; + if (tp != null) { + developerNotification = findProviderInStack().getText( + "devmode.notification", + "Developer Notification:\n{0}", + new String[]{ + "Expression [" + name + "] passed to tag which was evaluated to [" + dateObject + "](" + + (dateObject != null ? dateObject.getClass() : "null") + ") isn't instance of java.util.Date nor java.util.Calendar nor long!" + } + ); + } LOG.warn(developerNotification); } else { LOG.debug("Expression [{}] passed to tag which was evaluated to [{}]({}) isn't instance of java.util.Date nor java.util.Calendar nor long!", http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/org/apache/struts2/components/I18n.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/components/I18n.java b/core/src/main/java/org/apache/struts2/components/I18n.java index a04840f..7a538b1 100644 --- a/core/src/main/java/org/apache/struts2/components/I18n.java +++ b/core/src/main/java/org/apache/struts2/components/I18n.java @@ -121,7 +121,7 @@ public class I18n extends Component { ResourceBundle bundle = defaultTextProvider.getTexts(name); if (bundle == null) { - bundle = LocalizedTextUtil.findResourceBundle(name, localeProvider.getLocale()); + bundle = container.getInstance(LocalizedTextUtil.class).findResourceBundle(name, localeProvider.getLocale()); } if (bundle != null) { http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java b/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java index 909e802..226ecfc 100644 --- a/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java +++ b/core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java @@ -442,15 +442,7 @@ public class Dispatcher { } private Container init_PreloadConfiguration() { - Container container = getContainer(); - - boolean reloadi18n = Boolean.valueOf(container.getInstance(String.class, StrutsConstants.STRUTS_I18N_RELOAD)); - LocalizedTextUtil.setReloadBundles(reloadi18n); - - boolean devMode = Boolean.valueOf(container.getInstance(String.class, StrutsConstants.STRUTS_DEVMODE)); - LocalizedTextUtil.setDevMode(devMode); - - return container; + return getContainer(); } private void init_CheckWebLogicWorkaround(Container container) { http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java b/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java index 02de7f2..f12f1b5 100644 --- a/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java +++ b/core/src/main/java/org/apache/struts2/dispatcher/multipart/JakartaMultiPartRequest.java @@ -337,11 +337,7 @@ public class JakartaMultiPartRequest extends AbstractMultiPartRequest { for (String name : names) { List items = files.get(name); for (FileItem item : items) { - if (LOG.isDebugEnabled()) { - String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file", - Locale.ENGLISH, "no.message.found", new Object[]{name, item}); - LOG.debug(msg); - } + LOG.debug("Removing file {} {}", name, item ); if (!item.isInMemory()) { item.delete(); } http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/org/apache/struts2/factory/StrutsActionProxy.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/factory/StrutsActionProxy.java b/core/src/main/java/org/apache/struts2/factory/StrutsActionProxy.java index 52f2efb..64323c3 100644 --- a/core/src/main/java/org/apache/struts2/factory/StrutsActionProxy.java +++ b/core/src/main/java/org/apache/struts2/factory/StrutsActionProxy.java @@ -67,7 +67,7 @@ public class StrutsActionProxy extends DefaultActionProxy { protected String getErrorMessage() { if ((namespace != null) && (namespace.trim().length() > 0)) { String contextPath = ServletActionContext.getRequest().getContextPath(); - return LocalizedTextUtil.findDefaultText( + return localizedTextUtil.findDefaultText( "struts.exception.missing-package-action.with-context", Locale.getDefault(), new String[]{namespace, actionName, contextPath} http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/core/src/main/java/org/apache/struts2/util/TokenHelper.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/util/TokenHelper.java b/core/src/main/java/org/apache/struts2/util/TokenHelper.java index d4ae8aa..c6d0f0f 100644 --- a/core/src/main/java/org/apache/struts2/util/TokenHelper.java +++ b/core/src/main/java/org/apache/struts2/util/TokenHelper.java @@ -188,7 +188,8 @@ public class TokenHelper { if (!token.equals(sessionToken)) { if (LOG.isWarnEnabled()) { - LOG.warn(LocalizedTextUtil.findText(TokenHelper.class, "struts.internal.invalid.token", ActionContext.getContext().getLocale(), "Form token {0} does not match the session token {1}.", new Object[]{ + LocalizedTextUtil localizedTextUtil = ActionContext.getContext().getContainer().getInstance(LocalizedTextUtil.class); + LOG.warn(localizedTextUtil.findText(TokenHelper.class, "struts.internal.invalid.token", ActionContext.getContext().getLocale(), "Form token {0} does not match the session token {1}.", new Object[]{ token, sessionToken })); } http://git-wip-us.apache.org/repos/asf/struts/blob/4ef64185/plugins/pell-multipart/src/main/java/org/apache/struts2/dispatcher/multipart/PellMultiPartRequest.java ---------------------------------------------------------------------- diff --git a/plugins/pell-multipart/src/main/java/org/apache/struts2/dispatcher/multipart/PellMultiPartRequest.java b/plugins/pell-multipart/src/main/java/org/apache/struts2/dispatcher/multipart/PellMultiPartRequest.java index 3c733c6..c91c52e 100644 --- a/plugins/pell-multipart/src/main/java/org/apache/struts2/dispatcher/multipart/PellMultiPartRequest.java +++ b/plugins/pell-multipart/src/main/java/org/apache/struts2/dispatcher/multipart/PellMultiPartRequest.java @@ -154,11 +154,7 @@ public class PellMultiPartRequest extends AbstractMultiPartRequest { String inputValue = (String) fileParameterNames.nextElement(); UploadedFile[] files = getFile(inputValue); for (UploadedFile currentFile : files) { - if (LOG.isInfoEnabled()) { - String msg = LocalizedTextUtil.findText(this.getClass(), "struts.messages.removing.file", Locale.ENGLISH, - "no.message.found", new Object[]{inputValue, currentFile}); - LOG.info(msg); - } + LOG.debug("Removing file {} {}", inputValue, currentFile); if ((currentFile != null) && currentFile.isFile()) { if (!currentFile.delete()) { LOG.warn("Resource Leaking: Could not remove uploaded file [{}]", currentFile.getAbsolutePath());