Github user svenmeier commented on the issue:

    https://github.com/apache/wicket/pull/246
  
    Actually I'm wondering whether we should just add another method to 
ITagModifier (possibly with better names):
    
    ```
    public interface ITagModifier<T extends Annotation>
    {
      void modify(FormComponent<?> component, T annotation);
        
      void modify(FormComponent<?> component, ComponentTag tag, T annotation);
    }
    ```
    
    This way we can use the identical lookup 
BeanValidationConfiguration#getTagModifier() to alter the component itself from 
#onConfigure() and the component tag from #onComponentTag().
    
    All that's required for the new annotations is then:
    
    ```
    register(Size.class, new SizeTagModifier());
    register(NotNull.class, new RequiredTagModifier<NotNull>());
    register(NotEmpty.class, new RequiredTagModifier<NotEmpty>());
    register(NotBlank.class, new RequiredTagModifier<NotBlank>());
    ```


---

Reply via email to