<div dir="ltr">Hey Guillaume,<div><br></div><div>I think that this is a matter of taste, but I don't think that an extra wrapper element is necessary. So I'm fine with the current way of doing it. Not sure if using a wrapper element is an "usual convention". But I'm not strong on this one.</div><div><br></div><div>Other thoughts?</div><div><br></div><div>Christian</div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-05-18 11:35 GMT+02:00 Guillaume Smet <span dir="ltr"><<a href="mailto:guillaume.smet@gmail.com" target="_blank">guillaume.smet@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Hi,<br><br></div>In the current revision of the spec, the value-extractor element is at the same level as message-interpolator for instance.<br><br></div>So, when you want to add several value-extractors, you end up having the following:<br><validation-config><br><br> <message-interpolator>org.<wbr>hibernate.beanvalidation.tck.<wbr>tests.integration.cdi.<wbr>managedobjects.<wbr>MessageInterpolatorUsingDepend<wbr>encyInjection</message-<wbr>interpolator><br><br> <value-extractor>package.<wbr>MapKeyValueExtractorUsingDepen<wbr>dencyInjection</value-<wbr>extractor><br> <value-extractor>package.<wbr>MapValueValueExtractorUsingDep<wbr>endencyInjection</value-<wbr>extractor><br><br></validation-config><br><br></div>I was thinking that we should maybe wrap it into a value-extractors element. It would be nicer and more consistent with the usual conventions.<br><br></div>What do you all think about it?<span class="HOEnZb"><font color="#888888"><br><br>-- <br></font></span></div><span class="HOEnZb"><font color="#888888">Guillaume<br></font></span></div>
<br>______________________________<wbr>_________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.<wbr>jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/<wbr>mailman/listinfo/<wbr>beanvalidation-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div>Christian Kaltepoth</div><div>Blog: <a href="http://blog.kaltepoth.de/" target="_blank">http://blog.kaltepoth.de/</a></div><div>Twitter: <a href="http://twitter.com/chkal" target="_blank">http://twitter.com/chkal</a></div><div>GitHub: <a href="https://github.com/chkal" target="_blank">https://github.com/chkal</a></div><div><br></div></div>
</div>