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

Reply via email to