[
https://issues.apache.org/struts/browse/WW-2850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=44876#action_44876
]
Sami Dalouche commented on WW-2850:
-----------------------------------
hmm... That is true..
However, I think we could solve this issues by applying the checkbox
interceptor twice, right ?
So, let's say
<interceptor-stack name="paramsPrepareParamsStack">
<interceptor-ref name="exception"/>
<interceptor-ref name="alias"/>
<interceptor-ref name="i18n"/>
<interceptor-ref name="checkbox"/>
<interceptor-ref name="params"/>
<interceptor-ref name="servletConfig"/>
<interceptor-ref name="prepare"/>
<interceptor-ref name="chain"/>
<interceptor-ref name="modelDriven"/>
<interceptor-ref name="fileUpload"/>
<interceptor-ref name="checkbox"/>
<interceptor-ref name="staticParams"/>
<interceptor-ref name="actionMappingParams"/>
<interceptor-ref name="params"/>
<interceptor-ref name="conversionError"/>
<interceptor-ref name="validation">
<param name="excludeMethods">input,back,cancel</param>
</interceptor-ref>
<interceptor-ref name="workflow">
<param name="excludeMethods">input,back,cancel</param>
</interceptor-ref>
</interceptor-stack>
That way, it should keep the same semantics, right ?
> paramsPrepareParams stack should execute checkbox interceptor earlier
> ---------------------------------------------------------------------
>
> Key: WW-2850
> URL: https://issues.apache.org/struts/browse/WW-2850
> Project: Struts 2
> Issue Type: Bug
> Components: Integration
> Affects Versions: 2.1.2
> Reporter: Sami Dalouche
>
> The current paramsPrepareParams interceptor executes the checkbox interceptor
> late in the chain, which results in the 2 params interceptor not applying the
> same parameter set if there are checkboxes.
> Instead, it should look like :
> <interceptor-stack name="paramsPrepareParamsStack">
> <interceptor-ref name="exception"/>
> <interceptor-ref name="alias"/>
> <interceptor-ref name="i18n"/>
> <interceptor-ref name="checkbox"/>
> <interceptor-ref name="params"/>
> <interceptor-ref name="servletConfig"/>
> <interceptor-ref name="prepare"/>
> <interceptor-ref name="chain"/>
> <interceptor-ref name="modelDriven"/>
> <interceptor-ref name="fileUpload"/>
> <!-- BAD <interceptor-ref name="checkbox"/>-->
> <interceptor-ref name="staticParams"/>
> <interceptor-ref name="actionMappingParams"/>
> <interceptor-ref name="params"/>
> <interceptor-ref name="conversionError"/>
> <interceptor-ref name="validation">
> <param name="excludeMethods">input,back,cancel</param>
> </interceptor-ref>
> <interceptor-ref name="workflow">
> <param name="excludeMethods">input,back,cancel</param>
> </interceptor-ref>
> </interceptor-stack>
> The additional bonus is that you get rid of stupid OGNL exceptions trying to
> set __checkbox_* parameters during the first params execution :)
> regards,
> Sami Dalouche
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
