LocalDate and LocalDateTime don't work with ComparableAttribute

[njr-11]

Entity attribute types java.time.LocalDate and java.time.LocalDateTime are Comparable, but they implement Comparable<ChronoLocalDate> and Comparable<ChronoLocalDateTime> respectively rather than Comparable<LocalDate> and Comparable<LocalDateTime>.

I noticed this when attempting to use the static metamodel for LocalDate, where the following doesn't compile,

        ComparableAttribute<Author, LocalDate> birthday =
                ComparableAttribute.of(Author.class, BIRTHDAY, LocalDate.class);

[gavinking]

Ooops.

I think it works if we change every occurrence of extends Comparable<T>> to extends Comparable<?>>.

[otaviojava]

I remember that we removed the Comparable considering the LocalDate scenario

[gavinking]

I remember that we removed the Comparable considering the LocalDate scenario

I don't think we need to completely remove Comparable. We can just use a wildcard in the upper bound, unless I'm missing something.

[njr-11]

To be clear, I wasn't suggesting to remove Comparable, only to have this issue as a reminder so that LocalDate it isn't overlooked because it does require an update to the API (using Comparable<?> instead) before we release it.