You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by bu...@apache.org on 2003/05/27 19:23:19 UTC
DO NOT REPLY [Bug 20267] New: -
PATCH - add switch to Validator.java to allow filtering out success results
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20267>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND
INSERTED IN THE BUG DATABASE.
http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20267
PATCH - add switch to Validator.java to allow filtering out success results
Summary: PATCH - add switch to Validator.java to allow filtering
out success results
Product: Commons
Version: unspecified
Platform: All
OS/Version: All
Status: NEW
Severity: Enhancement
Priority: Other
Component: Validator
AssignedTo: commons-dev@jakarta.apache.org
ReportedBy: DavidNeuer@nascopgh.com
Here's a patch we have applied to our local tree to allow filtering out
success results (is there a use case for knowing that your input *did* pass
validation rules that wouldn't be better handled by trace debugging? No new is
good news...)
AFAICT, it should be perfectly binary- and source- compatible w/ existing
clients.
Please feel free to include it in the distribution.
There are other changes I was thinking of making as well, but thought they
might prove to me rather more controversial:
1) In ValidatorResult, there is an unused protected member called Field --
unused except by the constructor, which requires clients to pass it in even
though it will never be used. I don't know whether there are clients out there
subclassing ValidatorResult and using this, but if there are good design says
they should be refactored and add it to their own subclass.
LEGAL MUMBO-JUMBO: I have received an explicit waiver of any Nationwide
Appraisal Services Corp. IP rights over the software expressed in the patch
from Kevin Kelly, VP and MIS director of Nationwide Appraisal Services.
---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org