I never liked the idea that byProvider() was taking a Configuration subclass and not the actual ValidationProvider class.
This is now fixed using some additional generics magic. It has also the nice side effect of simplifying the contract between the bootstrap module and the providers.
In a nutshell, instead of
ValidatorFactory factory = Validation
.byProvider( ACMEConfiguration.class ) //chose a specific provider
.configure()
.messageInterpolator( new ContainerMessageInterpolator() ) //default configuration option
.addConstraint(Address.class, customConstraintDescriptor) //ACME specific method
.buildValidatorFactory();
we have
ValidatorFactory factory = Validation
.byProvider( ACMEProvider.class ) //chose a specific provider
.configure()
.messageInterpolator( new ContainerMessageInterpolator() ) //default configuration option
.addConstraint(Address.class, customConstraintDescriptor) //ACME specific method
.buildValidatorFactory();
The ACME provider becomes
/**
* ACME validation provider
* Note how ACMEConfiguration and ACMEProvider are linked together
* via the generic parameter.
*/
public class ACMEProvider implements ValidationProvider<ACMEConfiguration> {
...
}
Validation.byProvider() now is
/**
* Build a <code>Configuration</code> for a particular provider implementation.
* Optionally overrides the provider resolution strategy used to determine the provider.
* <p/>
* Used by applications targeting a specific provider programmatically.
* <p/>
* <pre>
* ACMEConfiguration configuration =
* Validation.byProvider(ACMEProvider.class)
* .providerResolver( new MyResolverStrategy() )
* .configure();
* </pre>,
* where <code>ACMEConfiguration</code> is the
* <code>Configuration</code> sub interface uniquely identifying the
* ACME Bean Validation provider. and ACMEProvider is the ValidationProvider
* implementation of the ACME provider.
*
* @param providerType the <code>ValidationProvider</code> implementation type
*
* @return instance building a provider specific <code>Configuration</code>
* sub interface implementation.
*/
public static <T extends Configuration<T>, U extends ValidationProvider<T>>
ProviderSpecificBootstrap<T> byProvider(Class<U> providerType) {
[...]
}
ValidationProvider now is
package javax.validation.spi;
/**
* Contract between the validation bootstrap mechanism and the provider engine.
* <p/>
* Implementations must have a public no-arg constructor. The construction of a provider
* should be as "lightweight" as possible.
*
* <code>T</code> represents the provider specific Configuration subclass
* which typically host provider's additional configuration methods.
*
* @author Emmanuel Bernard
* @author Hardy Ferentschik
*/
public interface ValidationProvider<T extends Configuration<T>> {
/**
* Returns a Configuration instance implementing <code>T</code>,
* the <code>Configuration</code> subinterface.
* The returned Configuration instance must use the current provider (<code>this</code>)
* to build the ValidatorFactory instance.
* <p/>
*
* @param state bootstrap state
*
* @return specific Configuration implementation
*/
T createSpecializedConfiguration(BootstrapState state);
/**
* Returns a Configuration instance. This instance is not bound to
* use the current provider. The choice of provider follows the algorithm described
* in {@link javax.validation.Configuration}
* <p/>
* The ValidationProviderResolver used by <code>Configuration</code>
* is provided by <code>state</code>.
* If null, the default ValidationProviderResolver is used.
*
* @param state bootstrap state
*
* @return Non specialized Configuration implementation
*/
Configuration<?> createGenericConfiguration(BootstrapState state);
/**
* Build a ValidatorFactory using the current provider implementation. The
* ValidatorFactory is assembled and follows the configuration passed
* via ConfigurationState.
* <p>
* The returned ValidatorFactory is properly initialized and ready for use.
* </p>
*
* @param configurationState the configuration descriptor
*
* @return the instanciated ValidatorFactory
* @throws javax.validation.ValidationException if the ValidatorFactory cannot be built
*/
ValidatorFactory buildValidatorFactory(ConfigurationState configurationState);
}
esp, we no longer have isSuitable.
Any objection? The diff for the spec is attached