diff --git a/src/main/java/org/springframework/data/history/AnnotationRevisionMetadata.java b/src/main/java/org/springframework/data/history/AnnotationRevisionMetadata.java index 48c8ab659..868d536e1 100755 --- a/src/main/java/org/springframework/data/history/AnnotationRevisionMetadata.java +++ b/src/main/java/org/springframework/data/history/AnnotationRevisionMetadata.java @@ -76,7 +76,7 @@ public class AnnotationRevisionMetadata> implem @Deprecated public Optional getRevisionDate() { - return revisionDate.get().map(this::convertToLocalDateTime); + return revisionDate.get().map(AnnotationRevisionMetadata::convertToLocalDateTime); } /* @@ -84,33 +84,7 @@ public class AnnotationRevisionMetadata> implem * @see org.springframework.data.history.RevisionMetadata#getRevisionDate() */ public Optional getRevisionInstant() { - return revisionDate.get().map(this::convertToInstant); - } - - private LocalDateTime convertToLocalDateTime(TemporalAccessor temporalAccessor) { - - if (temporalAccessor instanceof LocalDateTime) { - return (LocalDateTime) temporalAccessor; - } - - if (temporalAccessor instanceof Instant) { - return LocalDateTime.ofInstant((Instant) temporalAccessor, ZoneOffset.systemDefault()); - } - - throw new IllegalArgumentException(String.format("Can't convert %s to LocalDateTime", temporalAccessor)); - } - - private Instant convertToInstant(TemporalAccessor temporalAccessor) { - - if (temporalAccessor instanceof Instant) { - return (Instant) temporalAccessor; - } - - if (temporalAccessor instanceof LocalDateTime) { - return ((LocalDateTime) temporalAccessor).atZone(ZoneOffset.systemDefault()).toInstant(); - } - - throw new IllegalArgumentException(String.format("Can't convert %s to LocalDateTime", temporalAccessor)); + return revisionDate.get().map(AnnotationRevisionMetadata::convertToInstant); } /* @@ -131,4 +105,30 @@ public class AnnotationRevisionMetadata> implem return Optional.ofNullable(callback.getValue(entity)); }); } + + private static LocalDateTime convertToLocalDateTime(TemporalAccessor temporalAccessor) { + + if (temporalAccessor instanceof LocalDateTime) { + return (LocalDateTime) temporalAccessor; + } + + if (temporalAccessor instanceof Instant) { + return LocalDateTime.ofInstant((Instant) temporalAccessor, ZoneOffset.systemDefault()); + } + + throw new IllegalArgumentException(String.format("Can't convert %s to LocalDateTime!", temporalAccessor)); + } + + private static Instant convertToInstant(TemporalAccessor temporalAccessor) { + + if (temporalAccessor instanceof Instant) { + return (Instant) temporalAccessor; + } + + if (temporalAccessor instanceof LocalDateTime) { + return ((LocalDateTime) temporalAccessor).atZone(ZoneOffset.systemDefault()).toInstant(); + } + + throw new IllegalArgumentException(String.format("Can't convert %s to LocalDateTime!", temporalAccessor)); + } }