[prev in list] [next in list] [prev in thread] [next in thread] 

List:       struts-dev
Subject:    [jira] Commented: (WW-2811) ListUIBean does not strip %{} off
From:       "Scott Stanlick (JIRA)" <jira () apache ! org>
Date:       2008-09-25 12:41:26
Message-ID: 1199967084.1222346486165.JavaMail.jira () brutus
[Download RAW message or body]


    [ https://issues.apache.org/struts/browse/WW-2811?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=44753#action_44753 \
] 

Scott Stanlick commented on WW-2811:
------------------------------------

Cool!  Actually, maybe a single contact between S2 and OGNL that
checks/strips just prior to searching the stack.  I too have seen checks for
this many places.

Peace,
Scott

On Wed, Sep 24, 2008 at 1:06 PM, Gabriel Belingueres (JIRA) <jira@apache.org



> 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.


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic