Re: Struts 6.7.0, StrutsResultSupport conditionalParse uses com.opensymphony.xwork2.ActionInvocation invocation

Looks like I overlooked that specific method. And yes the old type can
be cast directly to the new type. ActionInvocation#adapt will allow
you to go the other way.

On Wed, Dec 11, 2024 at 7:48 PM [email protected]
<[email protected]> wrote:
>
> When upgrading from 6.6.1 to 6.7.0, I run into an issue with an
> overriding class extending StrutsResultSupport:
>
> on line 217 in org.apache.struts2.result.StrutsResultSupport the method
>
> protected String conditionalParse(String param, ActionInvocation
> invocation)
>
> seems to require as an argument
> (com.opensymphony.xwork2.ActionInvocation) invocation instead of the
> migrated (org.apache.struts2.ActionInvocation) invocation which, causes
> then a mismatch in my overriding class.
>
> Questions:
>
> - should the signature in org.apache.struts2.result.StrutsResultSupport
> not be changed to conditionalParse(String
> param, (org.apache.struts2.ActionInvocation) invocation)?
> - if not: is it save to use a simple cast like above, when overridding
> or should there be used something like ActionInvocation.adapt(invocation)?
>
> Thanks in advance!
>
> Best regards
> Markus