2012/11/8 Emmanuel Bernard <span dir="ltr"><<a href="mailto:emmanuel@hibernate.org" target="_blank">emmanuel@hibernate.org</a>></span><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
A property according to the JavaBeans specification is either:<br>
<br>
- a pair of [PropertyType] get[PropertyName]() / void set[PropertyName]([PropertyType] value)<br>
- a [PropertyType] get[PropertyName]()<br>
- a void set[PropertyName]([PropertyType] value)<br>
- if PropertyType is boolean, then boolean is[PropertyName]() is accepted (and preferred)<br>
<br>
There are ways to customize that further but that's out of the scope of this discussion. So to answer your question backing fields are an implementation detail.<br>
<br>
You and Gunnar seem to want a global VF level flag to activate/inactivate getters as constrained methods. I would like to know your reasons for that. When I put myself in situations and try and use method level validations, I can't seem to find it compelling to get a global flag. It would only let me chose between two evils instead of giving me choice on a per element basis.<br>
</blockquote><div><br></div><div>Generally I don't think that considering getters during method validation really is a problem. So I wouldn't really have a problem with this being the default behavior (which it also is in the proprietary API in HV 4.3). I'd prefer that over excluding certain named methods from method validation as this goes against the principle of least surprise and would feel like an inconsistency to me.</div>
<div><br></div><div>But if validating getters during method validation really is a problem, I'd expect this to be the case generally throughout an application or system. Taking the example of JPA entities which used to be validated upon persisting only and now would their property constraints get validated upon getter invocation, then I'd prefer to disable getter validation globally instead of marking each individual property with an annotation.</div>
<div><br></div><div>If I as a user really needed a configuration per element, I could do so using validation groups. If for instance single constraints should not be validated upon getter invocation, I could use a group like this:</div>
<br>public static class Foo {<br><br> //doesn't get validated during method validation of Default group<br> @NotNull(groups=AllConstraints.class)<br> private String getBar() { ... }<br><br> //does get validated during method validation<br>
@NotNull<br> private String getBaz() { ... }<br>}<br><br>//use this e.g. to validate upon entity persisting<br>public interface AllConstraints extends Default {}</div><div class="gmail_quote"><br></div><div class="gmail_quote">
Therefore I feel we shouldn't introduce a new mechanism such as <span style="font-family:arial,sans-serif;font-size:12.727272033691406px">@ValidateOnCall since the use case is already covered by existing concepts.</span></div>
<div class="gmail_quote"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px"><br></span></div><div class="gmail_quote"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px">Additionally, an annotation might not be the right vehicle for cases where only single constraints of a given getter shall be excluded from method validation while other constraints on the same getter should be considered.</span></div>
<div class="gmail_quote"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px"><br></span></div><div class="gmail_quote"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px">--Gunnar</span></div>
<div class="gmail_quote"><span style="font-family:arial,sans-serif;font-size:12.727272033691406px"><br></span></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
We already discussed the merits of having method validation activated by default vs the other way around in the past but if I follow your argument, I will ask developers who add constraints to their methods to also create a META-INF/validation.xml in their various modules to activate method validation ? That seems to go against the current trend to me.<br>
<span><font color="#888888"><br>
Emmanuel<br>
</font></span><div><div><br>
On 7 nov. 2012, at 12:36, Hardy Ferentschik <<a href="mailto:hardy@hibernate.org" target="_blank">hardy@hibernate.org</a>> wrote:<br>
<br>
> Hi,<br>
><br>
> I think I would like to clarify a few things first. We keep just talking about getters.<br>
> What does this actually mean? Do we talk about any method starting with 'get' or<br>
> do we mean getters for properties in the Java Bean sense (aka getters for fields with<br>
> the matching name)?<br>
><br>
> Do you want to exclude any method starting with 'get' from method validation or just<br>
> properties getters? If the latter, are we not introducing some quite arbitrary distinction<br>
> between methods starting with 'get'?<br>
><br>
> Also what's about methods starting with 'is'? Wouldn't we have to exclude them<br>
> as well in this case?<br>
><br>
> Also, in the current specification (1.0, 3.1.2. Field and property validation) is the intention<br>
> to only validate a getter when it is a Java Bean property (backed up by a matching field)?<br>
> I think this is not very clear and not explicitly tested in the TCK.<br>
><br>
><br>
> On 6 Jan 2012, at 6:53 PM, Emmanuel Bernard wrote:<br>
><br>
>> If we treat getters as regular methods, we would add a new<br>
>> behavior to all existing constrained beans. Things that were constrained<br>
>> at specific lifecycles boundaries would now be constrained every time a<br>
>> getter is called. That would break backward compatibility.<br>
><br>
> Backward compatibility is indeed an issue. However, instead of imo arbitrarily<br>
> considering a getXYZ not as a method, I would have just not enabled method validation<br>
> out of the box. I would have made it an active choice. Alternatively I would add an option<br>
> to switch between ignoring getters and taking them into consideration for method validation.<br>
> However, I don't think @ValidateOnCall is a good option though. This seems to me to<br>
> intrusive. I rather add a configuration option for the ValidatorFactory.<br>
><br>
>> Even if a getter was to be constrained on call - you would probably want<br>
>> to constrain on the setter<br>
><br>
> What if other method modify the state of the property (not just a setter). Maybe the intend<br>
> is to verify that the object is in a certain state when I request it.<br>
><br>
>> But I do like the simplicity of the rule claiming that all methods<br>
>> are constrained regardless of their similarity to the Bean<br>
>> specification. I do not currently thing that this argument alone<br>
>> outweighs the other problems.<br>
><br>
> I think it is a argument for consistency. Excluding getter methods from a general method validation<br>
> framework seems to introduce unnecessary inconsistency and might even exclude the framework<br>
> as general method validation framework of choice.<br>
><br>
>> ### Forcing a getter to behave as a regular method<br>
>><br>
>> To solve this use case, we can introduce a `@ValidateOnCall`<br>
>> annotation that should be placed on the getter method.<br>
><br>
> See above.<br>
><br>
> --Hardy<br>
><br>
><br>
> _______________________________________________<br>
> beanvalidation-dev mailing list<br>
> <a href="mailto:beanvalidation-dev@lists.jboss.org" target="_blank">beanvalidation-dev@lists.jboss.org</a><br>
> <a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
<br>
_______________________________________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org" target="_blank">beanvalidation-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
</div></div></blockquote></div><br></div>