<div dir="ltr">>�<span style="font-family:arial,sans-serif;font-size:13px">I'd like BootstrapConfiguration to mimic the name we use in the XSD.</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">Does that change your proposal?</span><div>
<span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="font-family:arial,sans-serif;font-size:13px">Maybe isExecutableValidationEnabled() then? I guess this would match <</span><span style="font-family:arial,sans-serif;font-size:13px">executable-validation enabled="true|false"> quite closely.</span></div>
<div style><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="font-family:arial,sans-serif;font-size:13px">>�</span><span style="font-family:arial,sans-serif;font-size:13px">How would that be designed in BootstrapConfiguration?</span></div>
<div style><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="font-family:arial,sans-serif;font-size:13px">Two methods, isExecutableValidationEnabled() and getDefaultValidatedExecutableTypes()? It would be in the responsibility of the caller to make use of the latter only if the former returned true.</span></div>
<div style><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div style><span style="font-family:arial,sans-serif;font-size:13px"><br></span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
2013/3/13 Emmanuel Bernard <span dir="ltr"><<a href="mailto:emmanuel@hibernate.org" target="_blank">emmanuel@hibernate.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'd like BootstrapConfiguration to mimic the name we use in the XSD.<br>
Does that change your proposal?<br>
<br>
I thought about using choice but it forces people to comment out their<br>
default validated executable section.<br>
<br>
That being said, I like your proposal for<br>
<executable-validation enabled="false"/> with a default to true.<br>
<br>
How would that be designed in BootstrapConfiguration?<br>
<span class="HOEnZb"><font color="#888888"><br>
Emmanuel<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed 2013-03-13 11:54, Gunnar Morling wrote:<br>
> What I like about approach #2 is that it bundles the configuration related<br>
> to executable validation in one element.<br>
><br>
> Maybe an attribute could be used for the flag:<br>
><br>
> � � <executable-validation enabled="true|false"><br>
> � � � � <default-validated-executables><br>
> � � � � � � <executable-type>NONE</executable-type><br>
> � � � � </default-validated-executables><br>
> � � </executable-validation><br>
><br>
> Generally I like "positive" flag names better than negated ones (i.e.<br>
> "enabled" instead of "disabled").<br>
><br>
> If you want to stick to two elements instead of an attribute, a xs:choice<br>
> could be used to express that only one of the elements may be present in a<br>
> given instance document.<br>
><br>
> > Would isValidationExecutionDisabled() not a better name?<br>
><br>
> Or isValidationOnExecutionEnabled()?<br>
><br>
> --Gunnar<br>
><br>
><br>
><br>
> 2013/3/13 Hardy Ferentschik <<a href="mailto:hardy@hibernate.org">hardy@hibernate.org</a>><br>
><br>
> ><br>
> > On 13 Jan 2013, at 8:37 AM, Emmanuel Bernard <<a href="mailto:emmanuel@hibernate.org">emmanuel@hibernate.org</a>><br>
> > wrote:<br>
> ><br>
> > > <a href="https://gist.github.com/emmanuelbernard/2191bad0c7d8742a8eea" target="_blank">https://gist.github.com/emmanuelbernard/2191bad0c7d8742a8eea</a><br>
> > ><br>
> > > Approach #1 has my preference I believe.<br>
> ><br>
> > +1 A single flag works.<br>
> ><br>
> > > Likewise BootstrapConfiguration would get a Boolean<br>
> > > getDisableValidationExecution() method that is ignored by Bean<br>
> > > Validation engines proper.<br>
> ><br>
> > Would isValidationExecutionDisabled() not a better name?<br>
> ><br>
> > --Hardy<br>
> ><br>
> > _______________________________________________<br>
> > beanvalidation-dev mailing list<br>
> > <a href="mailto:beanvalidation-dev@lists.jboss.org">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>
> _______________________________________________<br>
> beanvalidation-dev mailing list<br>
> <a href="mailto:beanvalidation-dev@lists.jboss.org">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">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>