AnnotatedElementMetaDataLoader doesn't work for private meth

[wolfc]

With a hack, part of https://jira.jboss.org/jira/browse/EJBTHREE-1448, I have made advisor.annotationRepository authoritative on annotations. I did this by setting advisor.metaData to a dummy. (Which leaves the relationship between and advisor.annotationRepository and advisor.metaData as a nagging question in my mind. I guess that annotationRepository should be deprecated?)

This disables the regular fallback of the advisor to scan for annotations and leaves it to MDR.

But now if I want to look at an annotation of a private method I get nothing, because AnnotatedElementMetaDataLoader.getComponentMetaDataRetrieval(signature)
returns null when AnnotatedElementMetaDataLoader.this.annotated = class
A and signature = MethodSignature of a private Method of class A.

Dare I say, I consider this a bug.

[alesj]

Which MDR version is this?
Since this looks OK in MDR trunk:

 else if (signature instanceof MethodSignature)
 {
 MethodSignature methodSignature = (MethodSignature) signature;
 Method method = methodSignature.getMethod();
 if (method == null)
 method = ReflectionUtils.findMethod(clazz, signature.getName(), signature.getParametersTypes(clazz));

Where RU.findMethod is

 public static Method findMethod(Class<?> clazz, String name, Class<?>... parameterTypes)
 {
 if (clazz == null)
 return null;

 try
 {
 return clazz.getDeclaredMethod(name, parameterTypes);
 }
 catch(Exception ignored)
 {
 }
 return findMethod(clazz.getSuperclass(), name, parameterTypes);
 }

Looking up declared methods, means it includes private as well. ;-)

"wolfc" wrote:

Dare I say, I consider this a bug.

I see you're the same as vickyk:
- http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4168997#4168997
;-)

[wolfc]

At least I'm right about the part where it is considered a bug. :-P

https://jira.jboss.org/jira/browse/JBMDR-29