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 37704] - [validator] API Bug in org.apache.commons.validator.ValidatorResult
Date Sun, 08 Jan 2006 22:22:02 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=37704>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=37704





------- Additional Comments From niallp@apache.org  2006-01-08 23:22 -------
OK good - that gives you a solution for the moment. However your original 
assertion that there is a problem with a Map of ResultStatus is still true and 
the question is what to do about them. Heres the possible options...

1) Do nothing
2) Do as you requested and make ResultStatus a public class.
3) Change the getActionMap to return a Map of "Boolean" values, as it did 
originally.
4) Deprecate getActionMap and provide an Iterator for the action names.
5) Provide an isValid() method which checks all the contained actions.

My preference is to do both 3) and 4). I'd rather not do 2) because I'm not 
convinced there is a need for ResultStatus and making it public will just make 
it harder to remove at a later date.

Unless there are objections I'll go ahead with 2) and 4).

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
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