[nant-dev] [ nant-Bugs-1556326 ] PropertyDictionary.Remove doesn't remove read-only-ness

[SourceForge.net <no...@so...>]

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