You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@wicket.apache.org by "Emond Papegaaij (Updated) (JIRA)" <ji...@apache.org> on 2011/11/28 16:14:45 UTC

[jira] [Updated] (WICKET-4234) Remove AbstractValidator because its mostly noise

     [ https://issues.apache.org/jira/browse/WICKET-4234?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Emond Papegaaij updated WICKET-4234:
------------------------------------

    Attachment: 0002-some-util-methods.patch

It would be nice if the RangeValidators got meaningful static factory methods. Instantiating the validators with null-arguments really isn't intuitive and makes the code hard to understand. This patch introduces a few for StringValidator, I can create patches for the others as well if you want me to.
                
> Remove AbstractValidator because its mostly noise
> -------------------------------------------------
>
>                 Key: WICKET-4234
>                 URL: https://issues.apache.org/jira/browse/WICKET-4234
>             Project: Wicket
>          Issue Type: Improvement
>            Reporter: Igor Vaynberg
>            Assignee: Igor Vaynberg
>             Fix For: 6.0.0
>
>         Attachments: 0002-some-util-methods.patch
>
>
> with the introduction of ValidationError, AbstractValidator has been mostly superseded because ValidationError makes it trivial to construct errors and let subclasses tweak them.
> this issue is to track the progress of removal of AbstractValidator

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira