[
https://issues.apache.org/jira/browse/WW-3488?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12909087#action_12909087
]
Maurizio Cucchiara commented on WW-3488:
----------------------------------------
Lukasz,
I cannot test the example attached to the previous bug at this time, but I'm
pretty sure that it doesn't work.
I think that ChainInterceptor sees inner actions like ActionChainResult, and
root.size() is always >2 in these cases.
By Chainable IF I mean a dual IF of UnChainable.
If a specific class Implements this one, then CI copy the stack.
Something like that:
private boolean shouldCopy(Object o) {
return o != null && (o instanceof Chainable);
}
> ChainingInterceptor doesn't copy the value stack to the target action
> ---------------------------------------------------------------------
>
> Key: WW-3488
> URL: https://issues.apache.org/jira/browse/WW-3488
> Project: Struts 2
> Issue Type: Bug
> Components: Core Interceptors
> Affects Versions: 2.2.1
> Reporter: Mika Laukka
> Assignee: Lukasz Lenart
> Fix For: 2.2.2
>
> Attachments: WW-3488.patch
>
>
> This can be easily reproduced by running the bundled struts2-showcase web
> application.
> http://localhost:8080/struts2-showcase-2.2.1/actionchaining/actionChain1!input.action
> results in this:
> Action Chain 1 Property 1:
> Action Chain 2 Property 1:
> Action Chain 3 Property 1: Property set in Action Chain 3
> So clearly the properties set in action 1 and action 2 are lost in the
> chaining process.
> For comparison, version 2.0.14 output
> http://localhost:8080/struts2-showcase-2.0.14/actionchaining/actionChain1!input.action
> results in this:
> Action Chain 1 Property 1: Property Set In Action Chain 1
> Action Chain 2 Property 1: Property Set in Action Chain 2
> Action Chain 3 Property 1: Property set in Action Chain 3
> Which is the expected result. This bug may be related to fixing of WW-2869.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
