commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 20267] New: - PATCH - add switch to Validator.java to allow filtering out success results
Date Tue, 27 May 2003 17:23:19 GMT
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


Mime
View raw message