I can see the advantage of consistency (though this is establishing a
pattern to which we'd be expected to stick going forward). Also, removing
the additional annotation element does mean no #init() method for the
implementation, for whatever that's worth. I don't feel strongly about it,
but I don't really have any negative arguments.
Matt
On Jun 21, 2017 5:00 AM, "Gunnar Morling" <gunnar(a)hibernate.org> wrote:
Hi all,
A colleague of mine reviewing the BV 2.0 changes pointed out an
inconsistency in the added/modified constraints.
As per the community poll we have now @NegativeOrZero and
@PositiveOrZero, whereas we still have the orPresent() option on @Past
and @Future. Should it be @PastOrPresent and @FutureOrPresent instead
for the sake of consistency?
Now one could argue that having only attributes (orPresent(),
orZero()) is most consistent with other constraints such as
@DigitalMin. But indeed the explicit annotations grew on me over time
as they are a bit shorter to use and, more importantly, better to
read.
It's late in the game, but I feel inclined to extract @PastOrPresent
and @FutureOrPresent. What do you think?
Thanks,
--Gunnar
_______________________________________________
beanvalidation-dev mailing list
beanvalidation-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/beanvalidation-dev