Hi,

I have created WW-5365 (https://issues.apache.org/jira/browse/WW-5365). Note 
that I'm not used to creating Jira issues for Apache projects, so I may have 
made some errors when creating the issue. Any improvements to the metadata 
and/or contents of this issue are welcome and appreciated.

Considering the effort that the workaround of manually adjusting the roughly 
140 s:radio tags in my project would take, I would be interested to know 
whether a rough timeframe for a possible resolution can be estimated. Assuming 
Apache is able to share such estimates, obviously. As a software developer I am 
aware that accurately estimating required time for work items can be 
challenging, and then there's also the release schedule to consider.

My thanks in advance for a hopefully speedy resolution of this issue.

Regards,

Nate

-----Original Message-----
From: Lukasz Lenart <lukaszlen...@apache.org> 
Sent: Thursday, 30 November 2023 16:52
To: Struts Users Mailing List <user@struts.apache.org>
Subject: Re: Struts 6.3 tags do not automatically evaluate the value field 
against the ognl stack

wt., 28 lis 2023 o 17:43 Yasser Zamani <yasserzam...@apache.org> napisał(a):
>
> Hi Nate,
>
> Thanks for the detailed info and the PoC which enabled me to debug the issue!
>
> The issue is introduced by [1] lines insertions which has changed the path of 
> the execution to findValue(expr, class) instead of findValue(expr). 
> findValue(expr, class) simply returns expr without evaluating it when class 
> is String and expr doesn't have %{. Before this change Struts used to use 
> findValue(expr) which evaluates expr.
>
> @Lukasz, what's your idea? looks like a regression right? Because Radio value 
> class looks like that can be Boolean according to Nate's sample.
>
> Regards,
> Yasser
>
> [1] 
> https://github.com/apache/struts/pull/577/files#diff-812c02fab412aaa58
> 8054c477048432288505ef1978d53d5b23ce9e5661f8492R83-R86

Reverting this change will break support for Enums, @Nate could you register a 
bug in JIRA so we can try to figure out a proper solution?


Thanks & regards
--
Łukasz

---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
For additional commands, e-mail: user-h...@struts.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscr...@struts.apache.org
For additional commands, e-mail: user-h...@struts.apache.org

Reply via email to