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 35621] - [configuration] Included properties w/ relative path fails in v1.1
Date Sat, 09 Jul 2005 09:18:53 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=35621>.
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=35621





------- Additional Comments From oliver.heger@t-online.de  2005-07-09 11:18 -------
I fully agree with you. It's a bug that the load() method is called before some
member fields that can influence the load process have been initialized. I will
see what I can do about it.

The fact that the includesAllowed flag is modified by the setBasePath() method
seems to me like a quick hack. Now the following can happen:

1. A PropertiesConfiguration object is created
2. setIncludesAllowed(false) is called on this object
3. The configuration file is set (which invokes setBasePath)
4. load() is called

Step 3 might enable the includesAllowed flag again, so that the results might be
different than expected. This behavior of setBasePath() is documented, but I
think it is indeed very confusing and it should be possible to find a better
solution.

Would anybody object if I changed this? This will be an incompatible change and
can have impact on existing code, but IMO it would make usage of this API clearer.

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