Skip to content

WW-3714 Deprecate and repackage common APIs part 1#1079

Merged
kusalk merged 3 commits into
masterfrom
kusal-depr-apis
Oct 21, 2024
Merged

WW-3714 Deprecate and repackage common APIs part 1#1079
kusalk merged 3 commits into
masterfrom
kusal-depr-apis

Conversation

@kusalk

@kusalk kusalk commented Oct 17, 2024

Copy link
Copy Markdown
Member

WW-3714

This PR deprecates and provides repackaged replacements for the following APIs, ahead of their permanent migration in Struts 7.0:

  • com.opensymphony.xwork2.ActionContext
  • com.opensymphony.xwork2.Action
  • com.opensymphony.xwork2.Result
  • com.opensymphony.xwork2.interceptor.Interceptor
  • com.opensymphony.xwork2.interceptor.ConditionalInterceptor

Note that these changes alone are not sufficient for defining Interceptors/Results implementing the new interfaces in XML configuration files. Factory support is also necessary which is introduced in #1087.

@kusalk kusalk force-pushed the kusal-depr-apis branch 2 times, most recently from fd3463c to 74af7e7 Compare October 17, 2024 02:35
@kusalk kusalk changed the title WW-3714 Deprecate and repackage common APIs WW-3714 Deprecate and repackage common APIs part 1 Oct 17, 2024
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@kusalk kusalk marked this pull request as ready for review October 21, 2024 10:06
@lukaszlenart

lukaszlenart commented Oct 21, 2024

Copy link
Copy Markdown
Member

Is it ready for review?

@kusalk

kusalk commented Oct 21, 2024

Copy link
Copy Markdown
Member Author

@lukaszlenart Thanks for reviewing - yep I wasn't sure whether there was a Struts 6.7 planned

@kusalk kusalk merged commit efa447a into master Oct 21, 2024
@kusalk kusalk deleted the kusal-depr-apis branch October 21, 2024 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants