DO NOT REPLY [Bug 34103] New: - inconsistent behavior

[bu...@apache.org]

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

           Summary: inconsistent behavior
           Product: Commons
           Version: unspecified
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Configuration
        AssignedTo: commons-dev@jakarta.apache.org
        ReportedBy: ittayd@qlusters.com


in AbstractConfiguration.resolveContainerStore (javadoc):

  * Returns an object from the store described by the key. If the value is a
  * List object, replace it with the first object in the list.

but what if getProperty returns a List because this is the type of the property? 
 this code will silently grab the first elemen. I don't understand why. Probably 
the reason is that some class extending AbstractConfiguration returns List for 
properties. In this case I think the better approach is to have that class 
return the first element instead, rather than returning the List and letting 
AbstractConfiguration (which is used by many other implementations, including 
outside of the configuration package) handle it

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