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

Gabriel Belingueres updated WW-2811:
------------------------------------

    Attachment: ListUIBean.patch

Attached a patch which would correct the issue. Just added the test to strip 
off the %{ } from the parameter.
However, I've seen this test in serveral places so in the long run I think will 
be better to refactor it somewhere.

> ListUIBean does not strip %{} off listKey
> -----------------------------------------
>
>                 Key: WW-2811
>                 URL: https://issues.apache.org/struts/browse/WW-2811
>             Project: Struts 2
>          Issue Type: Bug
>          Components: Expression Language
>    Affects Versions: 2.1.2
>            Reporter: Scott Stanlick
>            Priority: Minor
>         Attachments: ListUIBean.patch
>
>
> The ListUIBean method evaluateExtraParams() is stripping %{} off listValue 
> but *not* listKey!  Therefore, the following was failing:
> <s:select list="departments" listKey="%{departmentId}" 
> listValue="%{departmentName}"/>
> changing my code to this worked!
> <s:select list="departments" listKey="departmentId" 
> listValue="%{departmentName}"/>
> This crazy hybrid-expression language is messy enough without landmines like 
> this.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to