Hi Azeez, In side Axis2 I have not seen any valid requirement for getProperties method , what we are using is getProperty. So I am +1 on your approach where we remove getProperties method and introduced new method to get the property names.
> When it comes to replicating properties in the context hierarchy, we > should only transmit the differences, i.e. only > additions/updates/removals to/from the properties object should be > sent to the group. > > However, AbstractContext#getProperties() is public. Anybody who gets > hold of this Map can add items to it, remove items from it, or update > items in it. This makes it impossible for us to track the changes to > the properties Map. Also, this is bad encapsulation. Users can add any > object as the key to this Map which will break many things. I suggest > we remove this method(or deprecate it?) and introduce the following; > > public Iterator getPropertyNames(); // Will return only the property > names (keys in the Map) > public void removeProperty(String key); > public Object getProperty(String key); > public void setProperty(String key, Object value); > > Without such a change, we may not be able to properly handle > replication of the properties. > -- > Thanks > Afkham Azeez > > http://www.wso2.org > GPG Fingerprint: 643F C2AF EB78 F886 40C9 B2A2 4AE2 C887 665E 0760 --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
