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 34103] - [configuration] Inconsistent behavior
Date Mon, 21 Mar 2005 15:07:52 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=34103>.
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=34103





------- Additional Comments From ittayd@qlusters.com  2005-03-21 16:07 -------
for an external client calling getProperty, yes, it will return the List.

But lets say I have a property named 'foo' whose value is set to be some list. 
Now, I've written a buggy code and use getString("foo"). instead of getting an 
error (exception), this will silently be ignored. furthermore, what if i 
intended to have the property value *converted* to string (get something like 
'1,2,3', instead i'll get '1').

==> resolveContainerStore should use what getProperty() returns, and not 
manipulate it. 

this relates to issue 34098 which i've opened. namely, things should be 
consistent:
getProperty(String) should throw a NoSuchElementException (as it sometimes does) 
if the key doesn't exists. if isThrowXXX is used, then it should always return 
null. so should the other getXXX methods (not sometimes an empty object, like in 
getProperties). 




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