-
Notifications
You must be signed in to change notification settings - Fork 38.7k
Open
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)in: messagingIssues in messaging modules (jms, messaging)Issues in messaging modules (jms, messaging)in: webIssues in web modules (web, webmvc, webflux, websocket)Issues in web modules (web, webmvc, webflux, websocket)type: bugA general bugA general bug
Milestone
Description
Overview
While writing tests for #35342, I noticed that we have a discrepancy in our "method override" detection in MergedAnnotations
and AnnotatedMethod
.
Specifically, the MergedAnnotations
infrastructure supports finding annotations on methods that override non-public methods (such as a protected
or package-private method in a superclass).
However, AnnotatedMethod
will not find an annotation on a parameter in an overridden method unless the overridden method is public
.
Example
If we remove the public
modifier from the following declaration, the shouldFindAnnotationOnMethodParameterInGenericAbstractSuperclass()
test in AnnotatedMethodTests
will fail.
Line 106 in 5d325ca
public abstract void processTwo(@Param C value); |
Related Issues
Metadata
Metadata
Assignees
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)in: messagingIssues in messaging modules (jms, messaging)Issues in messaging modules (jms, messaging)in: webIssues in web modules (web, webmvc, webflux, websocket)Issues in web modules (web, webmvc, webflux, websocket)type: bugA general bugA general bug