[ 
https://issues.apache.org/struts/browse/WW-2993?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=46040#action_46040
 ] 

Wes Wannemacher commented on WW-2993:
-------------------------------------

Qiang, there are a few places to get the jar files. I forgot that xwork was a 
snapshot dependency, snapshot dependencies aren't always published. You can 
certainly build it as well, but it's probably easier to just download it... The 
latest one is available here - 

http://hudson.zones.apache.org/hudson/view/Struts/job/xwork2/32/com.opensymphony$xwork/

Just click on the one that says xwork-2.1.3-SNAPSHOT.jar... Then, you'll have 
to issue the command 'mvn install:install-file' just as it says. I'll check out 
the pom, etc. when I get a chance, but typically we don't worry too much about 
making sure that snapshots are available. We make pseudo-nightly builds 
available here - 

http://people.apache.org/builds/struts/nightlies/2.x/

But, it would be harder to make a patch from a zip. You'd have to keep a copy 
of the original and diff against it, etc. Plus, a patch generated that way is 
harder to apply. Thanks for helping out! Contributions from users are what 
makes open source what it is :)

> double Name expression gets escaped without being evaluated
> -----------------------------------------------------------
>
>                 Key: WW-2993
>                 URL: https://issues.apache.org/struts/browse/WW-2993
>             Project: Struts 2
>          Issue Type: Bug
>    Affects Versions: 2.0.14
>            Reporter: Qiang Zheng
>             Fix For: 2.1.7
>
>
> In DoubleListUIBean.java, there is one piece of code trying to populate 
> "doubleId" parameter. Looks like when "form!=null" this.doubleName get 
> escaped without being evaluated. The counter part in "form is null" case does 
> "findString" before do "escape". This cause problem when we pass in 
> doubleName as
> "%{#attr.name}", the lower part of "doubleId" became "_%{#attr_name}". Here 
> is the code.
> -----------------------------------------------------------------
> if (doubleId != null) {
> .....           
>         } else if (form != null) {
>             addParameter("doubleId", form.getParameters().get("id") + "_" 
> +escape(this.doubleName));
>         } else {
>             addParameter("doubleId", escape(doubleName !=null ? 
> findString(doubleName) : null));
>         }
> -----------------------------------------------------------------
> Should it acutally be:
> if (doubleId != null) {
> .....           
>         } else if (form != null) {
>             addParameter("doubleId", form.getParameters().get("id") + "_" 
> +escape(doubleName !=null ? findString(doubleName) : null));
>         } else {
>             addParameter("doubleId", escape(doubleName !=null ? 
> findString(doubleName) : null));
>         }

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