[ 
https://issues.apache.org/jira/browse/WW-3753?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13216618#comment-13216618
 ] 

Rees Byars commented on WW-3753:
--------------------------------

Hey, I just wanted to follow-up with this, and see if I could maybe get some 
feedback.  I'm not familiar with the turnaround on these kinds of issues on 
this project.  Is there any likelihood that my problem and patch will get 
analyzed, or do I just need to move on?  
                
> The AnnotationActionValidatorManager does not adhere to the 
> ActionValidatorManager interface's contract
> -------------------------------------------------------------------------------------------------------
>
>                 Key: WW-3753
>                 URL: https://issues.apache.org/jira/browse/WW-3753
>             Project: Struts 2
>          Issue Type: Bug
>            Reporter: Rees Byars
>            Priority: Minor
>         Attachments: ww3753_patch
>
>
> An ActionValidatorManager accepts a java.util.String "context" parameter for 
> identifying the appropriate configurations.  In the 
> AnnotationActionValidatorManager's buildValidatorKey() method, however, 
> "config.getName()" is used instead of the passed-in context.  This violates 
> the contract of the interface and tightly couples the 
> AnnotationActionValidatorManager to the ValidationInterceptor.  
> I have a situation whereby I have created my own validation interceptor for a 
> special case that passes in a context not derived from 
> "proxy.getActionName()" (equivalent to config.getName() except for in the 
> case of wildcards), only to find that this context isn't used properly by the 
> manager.  I then created my own manager, changing only the 
> buildValidatorKey() to use the given context, and it works well. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to