Polishing.

Lazily make methods and fields accessible. Introduce invoke method to AnnotationDetectionMethodCallback.

See #2569

Author: mp911de

src/main/java/org/springframework/data/history/AnnotationRevisionMetadata.java

@@ -100,7 +100,6 @@ public <T> T getDelegate() {
 		return (T) entity;
 	}
 
-	@SuppressWarnings("unchecked")
 	private static <T> Lazy<Optional<T>> detectAnnotation(Object entity, Class<? extends Annotation> annotationType) {
 
 		return Lazy.of(() -> {
@@ -109,7 +108,7 @@ private static <T> Lazy<Optional<T>> detectAnnotation(Object entity, Class<? ext
 					annotationType);
 			ReflectionUtils.doWithMethods(entity.getClass(), methodCallback);
 			if (methodCallback.getMethod() != null) {
-				return Optional.ofNullable((T) ReflectionUtils.invokeMethod(methodCallback.getRequiredMethod(), entity));
+				return Optional.ofNullable(methodCallback.invoke(entity));
 			}
 
 			AnnotationDetectionFieldCallback callback = new AnnotationDetectionFieldCallback(annotationType);

src/main/java/org/springframework/data/util/AnnotationDetectionFieldCallback.java

@@ -30,6 +30,7 @@
  *
  * @author Oliver Gierke
  * @author Christoph Strobl
+ * @author Mark Paluch
  */
 public class AnnotationDetectionFieldCallback implements FieldCallback {
 
@@ -55,8 +56,6 @@ public void doWith(Field field) throws IllegalArgumentException, IllegalAccessEx
 		}
 
 		if (AnnotatedElementUtils.findMergedAnnotation(field, annotationType) != null) {
-
-			ReflectionUtils.makeAccessible(field);
 			this.field = field;
 		}
 	}
@@ -129,6 +128,7 @@ public <T> T getValue(Object source) {
 			return null;
 		}
 
+		ReflectionUtils.makeAccessible(field);
 		return (T) ReflectionUtils.getField(field, source);
 	}
 }

src/main/java/org/springframework/data/util/AnnotationDetectionMethodCallback.java

@@ -123,9 +123,29 @@ public void doWith(Method method) throws IllegalArgumentException, IllegalAccess
 			}
 
 			this.annotation = foundAnnotation;
-
-			ReflectionUtils.makeAccessible(method);
 			this.foundMethod = method;
 		}
 	}
+
+	/**
+	 * Invokes the method using reflection.
+	 *
+	 * @param target can be {@literal null} for static method invocations.
+	 * @param args method arguments.
+	 * @return
+	 * @since 2.7
+	 */
+	@Nullable
+	@SuppressWarnings("unchecked")
+	public <T> T invoke(@Nullable Object target, Object... args) {
+
+		Method method = this.foundMethod;
+
+		if (method == null) {
+			return null;
+		}
+
+		ReflectionUtils.makeAccessible(method);
+		return (T) ReflectionUtils.invokeMethod(method, target, args);
+	}
 }