[GitHub] [struts] brianandle commented on a diff in pull request #559: WW-5184 - Add optional parameter value check to ParametersInterceptor

[GitBox <gi...@apache.org>]

brianandle commented on code in PR #559:
URL: https://github.com/apache/struts/pull/559#discussion_r890405978


##########
core/src/main/java/com/opensymphony/xwork2/interceptor/ParametersInterceptor.java:
##########
@@ -354,7 +407,86 @@ protected boolean isExcluded(String paramName) {
         }
         return false;
     }
+    
 
+    public void setAcceptedValuePatterns(String commaDelimitedPatterns) {

Review Comment:
   No matter what I did I couldn't get Inject to work on these, and I had assumed that's because I was using it inappropriately. The other setters that are documented by Struts, setParamNameMaxLength, setAcceptParamNames, setExcludeParams don't utilize the @Inject annotation and are injected via the following params and thus I followed suit
   
   acceptParamNames, excludeParams, paramNameMaxLength
   
   
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@struts.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org