struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (WW-4139) Validation of non-default methods not being executed
Date Thu, 12 Nov 2015 13:58:11 GMT


ASF subversion and git services commented on WW-4139:

Commit ee562666abf01faf4dd73f65c9f4d77b079423d1 in struts's branch refs/heads/master from
[;h=ee56266 ]

Form.getValidators does not respect validateAnnotatedMethodOnly flag

WW-4139 removed method name from getValidators call.  While this works
fine for a case mentioned in that ticket it fails to account for a case
dynamicMethodCalls and validateAnnotatedMethodOnly are used. Without
method name actionValidatorManager.getValidators returns all validators
defined for a given field.
Expected behavior for dynamicMethodCalls with
validateAnnotatedMethodOnly case is to return validators for a given
field defined at the method level.

> Validation of non-default methods not being executed
> ----------------------------------------------------
>                 Key: WW-4139
>                 URL:
>             Project: Struts 2
>          Issue Type: Bug
>    Affects Versions: 2.3.15
>            Reporter: Bruno Klava
>            Assignee: Lukasz Lenart
>             Fix For: 2.3.16
> Code sample:
> In the example, in the index action/page, there are 2 forms: the difference is that the
form 1 submits to the action without setting a method (then calling the default execute()
method), while the form 2 submits to the same action but calling a specific method.
> The validator is set annotating the field setter method, so it should be available either
to the default execute method and to the non-default method. This is exactly the behaviour
obtained when using Struts
> After switching to Struts 2.3.15, the form 2 validation is not executed at all.
> As far as I can tell, this difference is due to changes made in org.apache.struts2.components.Form.getValidators(String

This message was sent by Atlassian JIRA

View raw message