Issue Details (XML | Word | Printable)

Key: XFIRE-429
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Trivial Trivial
Assignee: Dan Diephouse
Reporter: Holger Hoffstätte
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
XFire

Client should have removeProperty() method

Created: 27/May/06 08:57 AM   Updated: 30/May/06 09:03 AM
Component/s: Core
Affects Version/s: 1.1
Fix Version/s: 1.1.1

Time Tracking:
Not Specified


 Description  « Hide
Client has get/setProperty() methods, but no removeProperty(). This implicitly assumes that a setProperty(key, null) is actually allowed on the underlying HashMap (which was a horrible mistake, as Josh Bloch has recently admitted).
An alternative approach could be to implement setProperty() to remove the key/value entry on null value instead of putting null into the map, but I do not know how this might might affect other xfire use cases, therefore removeProperty() would probably be the solution with the least surprises.

 All   Comments   Work Log   Change History      Sort Order: Ascending order - Click to sort in descending order
Dan Diephouse added a comment - 30/May/06 09:03 AM
Fixed this in SVN. Thanks.