Warn when no relevant methods are found
This commit is contained in:
+11
-2
@@ -57,7 +57,7 @@ public class InterceptionInstrumenter {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void addHandlerClass(Class<?> handlerClass) {
|
private void addHandlerClass(Class<?> handlerClass) {
|
||||||
for (Field field : handlerClass.getFields()) {
|
for (final Field field : handlerClass.getFields()) {
|
||||||
MethodInterceptor annotation = field.getAnnotation(MethodInterceptor.class);
|
MethodInterceptor annotation = field.getAnnotation(MethodInterceptor.class);
|
||||||
if (annotation == null) continue;
|
if (annotation == null) continue;
|
||||||
|
|
||||||
@@ -74,7 +74,7 @@ public class InterceptionInstrumenter {
|
|||||||
throw new IllegalArgumentException("Interceptor is null: " + field);
|
throw new IllegalArgumentException("Interceptor is null: " + field);
|
||||||
}
|
}
|
||||||
|
|
||||||
Class<?> interceptorClass = interceptor.getClass();
|
final Class<?> interceptorClass = interceptor.getClass();
|
||||||
|
|
||||||
FieldData fieldData = getFieldData(field, interceptorClass);
|
FieldData fieldData = getFieldData(field, interceptorClass);
|
||||||
|
|
||||||
@@ -109,6 +109,15 @@ public class InterceptionInstrumenter {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (enterData.isEmpty() && normalReturnData.isEmpty() && exceptionData.isEmpty()) {
|
||||||
|
dumpTasks.add(new DumpAction() {
|
||||||
|
@Override
|
||||||
|
public void dump(PrintStream out) {
|
||||||
|
out.println("WARNING: No relevant methods found in " + field + " of type " + interceptorClass.getCanonicalName());
|
||||||
|
}
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
String nameFromAnnotation = annotation.methodName();
|
String nameFromAnnotation = annotation.methodName();
|
||||||
String methodName = nameFromAnnotation.isEmpty() ? field.getName() : nameFromAnnotation;
|
String methodName = nameFromAnnotation.isEmpty() ? field.getName() : nameFromAnnotation;
|
||||||
MethodInstrumenter instrumenter = new MethodInstrumenter(
|
MethodInstrumenter instrumenter = new MethodInstrumenter(
|
||||||
|
|||||||
Reference in New Issue
Block a user