On 27 août 2012, at 10:51, Gunnar Morling wrote:

For the “getter“ approach, getMethodValidator() would be defined on ValidatorFactory, right? At least that was the original idea.

Ah. I was thinking `getMethodValidator()` was hosted to `Validator`. The negative I could see are:

1. `ValidatorContext` needs to be updated too

2. we would require to allow injection of `MethodValidator` in injectable environments

3. we will need to duplicate `getConstraintsForClass` and `unwrap` in `MethodValidator`

Independently, they don't weight much (except point 3) but as a whole I wonder if that's the right approach.

What would be the negative points of putting it on `Validator`? It requires one extra method call. Any other?

Emmanuel