Bugs item #1556326, was opened at 2006-09-11 14:40
Message generated for change (Comment added) made by drieseng
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=402868&aid=1556326&group_id=31650
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core
Group: 0.85
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Roger Lipscombe (rlipscombe)
>Assigned to: Gert Driesen (drieseng)
Summary: PropertyDictionary.Remove doesn't remove read-only-ness
Initial Comment:
If, in a task, you call
PropertyDictionary.AddReadOnlyProperty, it is added
to this.Dictionary, and to _readOnlyProperties.
If you later call PropertyDictionary.Remove, the
property is removed from this.Dictionary, but not
from _readOnlyProperties.
This means that, if you later still, attempt to add a
different property with the same name, it silently
fails.
FIX: The Remove function should be changed to the
following:
public void Remove(string name) {
Dictionary.Remove(name);
_readOnlyProperties.Remove(name);
}
----------------------------------------------------------------------
>Comment By: Gert Driesen (drieseng)
Date: 2006-09-13 23:17
Message:
Logged In: YES
user_id=707851
This is now fixed in cvs.
Thanks for the report!
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=402868&aid=1556326&group_id=31650
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
nant-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/nant-developers