Return-Path: X-Original-To: apmail-struts-commits-archive@minotaur.apache.org Delivered-To: apmail-struts-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 818DE17B3F for ; Thu, 18 Jun 2015 09:53:45 +0000 (UTC) Received: (qmail 13865 invoked by uid 500); 18 Jun 2015 09:53:45 -0000 Delivered-To: apmail-struts-commits-archive@struts.apache.org Received: (qmail 13822 invoked by uid 500); 18 Jun 2015 09:53:45 -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 13813 invoked by uid 99); 18 Jun 2015 09:53:45 -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; Thu, 18 Jun 2015 09:53:45 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 30AA2E10A9; Thu, 18 Jun 2015 09:53:45 +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 Message-Id: X-Mailer: ASF-Git Admin Mailer Subject: struts git commit: WW-4443 Renames scope DEFAULT into PROTOTYPE Date: Thu, 18 Jun 2015 09:53:45 +0000 (UTC) Repository: struts Updated Branches: refs/heads/master e2347f165 -> 356b5c57b WW-4443 Renames scope DEFAULT into PROTOTYPE Project: http://git-wip-us.apache.org/repos/asf/struts/repo Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/356b5c57 Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/356b5c57 Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/356b5c57 Branch: refs/heads/master Commit: 356b5c57b269e6ba7cfe267d4517e5f7a27ce200 Parents: e2347f1 Author: Lukasz Lenart Authored: Thu Jun 18 11:53:34 2015 +0200 Committer: Lukasz Lenart Committed: Thu Jun 18 11:53:34 2015 +0200 ---------------------------------------------------------------------- .../config/providers/XWorkConfigurationProvider.java | 4 ++-- .../config/providers/XmlConfigurationProvider.java | 4 ++-- .../opensymphony/xwork2/inject/ContainerBuilder.java | 14 +++++++------- .../java/com/opensymphony/xwork2/inject/Scope.java | 2 +- .../struts2/config/DefaultBeanSelectionProvider.java | 10 +++++----- core/src/main/resources/struts-default.xml | 10 +++++----- 6 files changed, 22 insertions(+), 22 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/struts/blob/356b5c57/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java b/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java index 4ade520..89e8f97 100644 --- a/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java +++ b/core/src/main/java/com/opensymphony/xwork2/config/providers/XWorkConfigurationProvider.java @@ -177,8 +177,8 @@ public class XWorkConfigurationProvider implements ConfigurationProvider { .factory(NumberConverter.class, Scope.SINGLETON) .factory(StringConverter.class, Scope.SINGLETON) - .factory(ExcludedPatternsChecker.class, DefaultExcludedPatternsChecker.class, Scope.DEFAULT) - .factory(AcceptedPatternsChecker.class, DefaultAcceptedPatternsChecker.class, Scope.DEFAULT) + .factory(ExcludedPatternsChecker.class, DefaultExcludedPatternsChecker.class, Scope.PROTOTYPE) + .factory(AcceptedPatternsChecker.class, DefaultAcceptedPatternsChecker.class, Scope.PROTOTYPE) ; props.setProperty(XWorkConstants.DEV_MODE, Boolean.FALSE.toString()); http://git-wip-us.apache.org/repos/asf/struts/blob/356b5c57/core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java b/core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java index 0cf7059..d3d524c 100644 --- a/core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java +++ b/core/src/main/java/com/opensymphony/xwork2/config/providers/XmlConfigurationProvider.java @@ -193,8 +193,8 @@ public class XmlConfigurationProvider implements ConfigurationProvider { String scopeStr = child.getAttribute("scope"); boolean optional = "true".equals(child.getAttribute("optional")); Scope scope = Scope.SINGLETON; - if ("default".equals(scopeStr)) { - scope = Scope.DEFAULT; + if ("prototype".equals(scopeStr)) { + scope = Scope.PROTOTYPE; } else if ("request".equals(scopeStr)) { scope = Scope.REQUEST; } else if ("session".equals(scopeStr)) { http://git-wip-us.apache.org/repos/asf/struts/blob/356b5c57/core/src/main/java/com/opensymphony/xwork2/inject/ContainerBuilder.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/inject/ContainerBuilder.java b/core/src/main/java/com/opensymphony/xwork2/inject/ContainerBuilder.java index 54f5be6..c37f6c7 100644 --- a/core/src/main/java/com/opensymphony/xwork2/inject/ContainerBuilder.java +++ b/core/src/main/java/com/opensymphony/xwork2/inject/ContainerBuilder.java @@ -150,22 +150,22 @@ public final class ContainerBuilder { /** * Convenience method. Equivalent to {@code factory(type, name, factory, - * Scope.DEFAULT)}. + * Scope.PROTOTYPE)}. * * @see #factory(Class, String, Factory, Scope) */ public ContainerBuilder factory(Class type, String name, Factory factory) { - return factory(type, name, factory, Scope.DEFAULT); + return factory(type, name, factory, Scope.PROTOTYPE); } /** * Convenience method. Equivalent to {@code factory(type, - * Container.DEFAULT_NAME, factory, Scope.DEFAULT)}. + * Container.DEFAULT_NAME, factory, Scope.PROTOTYPE)}. * * @see #factory(Class, String, Factory, Scope) */ public ContainerBuilder factory(Class type, Factory factory) { - return factory(type, Container.DEFAULT_NAME, factory, Scope.DEFAULT); + return factory(type, Container.DEFAULT_NAME, factory, Scope.PROTOTYPE); } /** @@ -215,7 +215,7 @@ public final class ContainerBuilder { * instances using the container, recursively injecting dependencies. *

*

Sets scope to value from {@link Scoped} annotation on the - * implementation class. Defaults to {@link Scope#DEFAULT} if no annotation + * implementation class. Defaults to {@link Scope#PROTOTYPE} if no annotation * is found. * * @param type of dependency @@ -226,7 +226,7 @@ public final class ContainerBuilder { public ContainerBuilder factory(final Class type, String name, final Class implementation) { Scoped scoped = implementation.getAnnotation(Scoped.class); - Scope scope = scoped == null ? Scope.DEFAULT : scoped.value(); + Scope scope = scoped == null ? Scope.PROTOTYPE : scoped.value(); return factory(type, name, implementation, scope); } @@ -419,7 +419,7 @@ public final class ContainerBuilder { } }; - return factory(Key.newInstance(type, name), factory, Scope.DEFAULT); + return factory(Key.newInstance(type, name), factory, Scope.PROTOTYPE); } /** http://git-wip-us.apache.org/repos/asf/struts/blob/356b5c57/core/src/main/java/com/opensymphony/xwork2/inject/Scope.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/com/opensymphony/xwork2/inject/Scope.java b/core/src/main/java/com/opensymphony/xwork2/inject/Scope.java index 62443f7..08d7c49 100644 --- a/core/src/main/java/com/opensymphony/xwork2/inject/Scope.java +++ b/core/src/main/java/com/opensymphony/xwork2/inject/Scope.java @@ -28,7 +28,7 @@ public enum Scope { /** * One instance per injection. */ - DEFAULT { + PROTOTYPE { @Override InternalFactory scopeFactory(Class type, String name, InternalFactory factory) { http://git-wip-us.apache.org/repos/asf/struts/blob/356b5c57/core/src/main/java/org/apache/struts2/config/DefaultBeanSelectionProvider.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/struts2/config/DefaultBeanSelectionProvider.java b/core/src/main/java/org/apache/struts2/config/DefaultBeanSelectionProvider.java index 1f66d16..15cb16d 100644 --- a/core/src/main/java/org/apache/struts2/config/DefaultBeanSelectionProvider.java +++ b/core/src/main/java/org/apache/struts2/config/DefaultBeanSelectionProvider.java @@ -383,13 +383,13 @@ public class DefaultBeanSelectionProvider 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.DEFAULT); + alias(TextProvider.class, StrutsConstants.STRUTS_XWORKTEXTPROVIDER, builder, props, Scope.PROTOTYPE); alias(LocaleProvider.class, StrutsConstants.STRUTS_LOCALE_PROVIDER, builder, props); alias(ActionProxyFactory.class, StrutsConstants.STRUTS_ACTIONPROXYFACTORY, builder, props); alias(ObjectTypeDeterminer.class, StrutsConstants.STRUTS_OBJECTTYPEDETERMINER, builder, props); alias(ActionMapper.class, StrutsConstants.STRUTS_MAPPER_CLASS, builder, props); - alias(MultiPartRequest.class, StrutsConstants.STRUTS_MULTIPART_PARSER, builder, props, Scope.DEFAULT); + alias(MultiPartRequest.class, StrutsConstants.STRUTS_MULTIPART_PARSER, builder, props, Scope.PROTOTYPE); alias(FreemarkerManager.class, StrutsConstants.STRUTS_FREEMARKER_MANAGER_CLASSNAME, builder, props); alias(VelocityManager.class, StrutsConstants.STRUTS_VELOCITY_MANAGER_CLASSNAME, builder, props); alias(UrlRenderer.class, StrutsConstants.STRUTS_URL_RENDERER, builder, props); @@ -407,9 +407,9 @@ public class DefaultBeanSelectionProvider extends AbstractBeanSelectionProvider alias(DispatcherErrorHandler.class, StrutsConstants.STRUTS_DISPATCHER_ERROR_HANDLER, builder, props); - /** Checker is used mostly in interceptors, so there be one instance of checker per interceptor with Scope.DEFAULT **/ - alias(ExcludedPatternsChecker.class, StrutsConstants.STRUTS_EXCLUDED_PATTERNS_CHECKER, builder, props, Scope.DEFAULT); - alias(AcceptedPatternsChecker.class, StrutsConstants.STRUTS_ACCEPTED_PATTERNS_CHECKER, builder, props, Scope.DEFAULT); + /** Checker is used mostly in interceptors, so there be one instance of checker per interceptor with Scope.PROTOTYPE **/ + alias(ExcludedPatternsChecker.class, StrutsConstants.STRUTS_EXCLUDED_PATTERNS_CHECKER, builder, props, Scope.PROTOTYPE); + alias(AcceptedPatternsChecker.class, StrutsConstants.STRUTS_ACCEPTED_PATTERNS_CHECKER, builder, props, Scope.PROTOTYPE); switchDevMode(props); http://git-wip-us.apache.org/repos/asf/struts/blob/356b5c57/core/src/main/resources/struts-default.xml ---------------------------------------------------------------------- diff --git a/core/src/main/resources/struts-default.xml b/core/src/main/resources/struts-default.xml index f9c9ff4..5b82fe0 100644 --- a/core/src/main/resources/struts-default.xml +++ b/core/src/main/resources/struts-default.xml @@ -81,8 +81,8 @@ - - + + @@ -112,7 +112,7 @@ - + @@ -162,8 +162,8 @@ - - + +