[ 
https://issues.apache.org/jira/browse/WW-5352?focusedWorklogId=897748&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-897748
 ]

ASF GitHub Bot logged work on WW-5352:
--------------------------------------

                Author: ASF GitHub Bot
            Created on: 03/Jan/24 03:54
            Start Date: 03/Jan/24 03:54
    Worklog Time Spent: 10m 
      Work Description: kusalk commented on code in PR #829:
URL: https://github.com/apache/struts/pull/829#discussion_r1440046409


##########
core/src/main/java/org/apache/struts2/action/NoParameters.java:
##########
@@ -23,7 +23,6 @@
  * them automatically. This may be useful if one is using the action tag and 
want to supply
  * the parameters to the action manually using the param tag. It may also be 
useful if one for
  * security reasons wants to make sure that parameters cannot be set by 
malicious users.
- *
  */
-public interface NoParameters extends 
com.opensymphony.xwork2.interceptor.NoParameters {

Review Comment:
   Reverse the delegations for easier deletion of deprecated classes





Issue Time Tracking
-------------------

    Worklog Id:     (was: 897748)
    Time Spent: 0.5h  (was: 20m)

> Implement annotation mechanism for injectable fields via parameters
> -------------------------------------------------------------------
>
>                 Key: WW-5352
>                 URL: https://issues.apache.org/jira/browse/WW-5352
>             Project: Struts 2
>          Issue Type: Improvement
>          Components: Core, Core Interceptors
>            Reporter: Kusal Kithul-Godage
>            Priority: Minor
>             Fix For: 6.4.0
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> struts.parameters.requireAnnotations
>  
> Require an explicit annotation '@StrutsParameter' on one of: 
> Getter/Setter/Field/ReturnType for injecting parameters.
>  
> This mechanism is intended to be a more usable replacement for 
> 'ParameterNameAware'



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to