You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Musachy Barroso (JIRA)" <ji...@apache.org> on 2009/03/27 14:50:05 UTC

[jira] Resolved: (WW-2850) paramsPrepareParams stack should execute checkbox interceptor earlier

     [ https://issues.apache.org/struts/browse/WW-2850?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Musachy Barroso resolved WW-2850.
---------------------------------

    Resolution: Fixed

"checkbox" interceptor was moved right before the first "params" interceptor in "paramsPrepareParamsStack"

> 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
>             Fix For: 2.1.7
>
>
> 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.