2016-03-06 12:48 GMT+01:00 Greg Huber <gregh3...@gmail.com>:
> In DefaultConfiguration.buildFullActionConfig(..) it checks for the
> interceptors that were added and if it does not find any it adds the
> getFullDefaultInterceptorRef() which in my case is myStack.
>
> I guess the question is whether adding the message store interceptor ever
> included the default-interceptor-ref?  The code suggests that adding
> myStack is now required (unless catered for elsewhere).

Nope and it was that way from the very beginning, my refactoring
didn't change that. I was debugging it early and using just
MessageStoreInterceptor (with createSession interceptor) shows only
those two interceptors in scope of the action.

> List<InterceptorMapping> interceptors = new
> ArrayList<>(baseConfig.getInterceptors());
> if (interceptors.size() <= 0) {
>    String defaultInterceptorRefName =
> packageContext.getFullDefaultInterceptorRef();
>    if (defaultInterceptorRefName != null) {
>
> interceptors.addAll(InterceptorBuilder.constructInterceptorReference(new
> PackageConfig.Builder(packageContext), defaultInterceptorRefName,
>                         new LinkedHashMap<String, String>(),
> packageContext.getLocation(), objectFactory));
>   }
> }

Yap, default-interceptor-ref will be used if no other stack was
defined for an action.


Regards
-- 
Ɓukasz
+ 48 606 323 122 http://www.lenart.org.pl/

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org

Reply via email to