You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by cn...@apache.org on 2015/10/12 12:55:43 UTC

[2/4] struts git commit: Make it easier to extend to override interceptor stack

Make it easier to extend to override interceptor stack

Just as createAction is split out from init, it would be great to split out creating the interceptor iterator creation so it can be overridden without having to override all of init().  (Needed this to always prepend an i18n interceptor to all stacks).

Project: http://git-wip-us.apache.org/repos/asf/struts/repo
Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/117f5fba
Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/117f5fba
Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/117f5fba

Branch: refs/heads/master
Commit: 117f5fbacc6193b1384f1cea3ee844252bfbdb3e
Parents: 41dd070
Author: Jasper <ja...@yahoo.com>
Authored: Fri Oct 9 07:41:05 2015 -0400
Committer: Jasper <ja...@yahoo.com>
Committed: Fri Oct 9 07:41:05 2015 -0400

----------------------------------------------------------------------
 .../java/com/opensymphony/xwork2/DefaultActionInvocation.java  | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/struts/blob/117f5fba/core/src/main/java/com/opensymphony/xwork2/DefaultActionInvocation.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/com/opensymphony/xwork2/DefaultActionInvocation.java b/core/src/main/java/com/opensymphony/xwork2/DefaultActionInvocation.java
index a5b4723..d6385c6 100644
--- a/core/src/main/java/com/opensymphony/xwork2/DefaultActionInvocation.java
+++ b/core/src/main/java/com/opensymphony/xwork2/DefaultActionInvocation.java
@@ -396,7 +396,11 @@ public class DefaultActionInvocation implements ActionInvocation {
         invocationContext = new ActionContext(contextMap);
         invocationContext.setName(proxy.getActionName());
 
-        // get a new List so we don't get problems with the iterator if someone changes the list
+        createInterceptors(proxy);
+    }
+
+    protected void createInterceptors(ActionProxy proxy) {
+        // Get a new List so we don't get problems with the iterator if someone changes the original list
         List<InterceptorMapping> interceptorList = new ArrayList<>(proxy.getConfig().getInterceptors());
         interceptors = interceptorList.iterator();
     }