[Zope-PAS] mutable property sheets

[Wichert Akkerman]

Previously Jens Vagelpohl wrote:
> - - IMutablePropertiesPlugin.getPropertiesForUser deviates from the  
> original contract in IPropertiesPlugin.getPropertiesForUser and  
> returns a property sheet object instead of a mapping. IMHO it should  
> return a mapping.

It has to return an object so it can persist property changes. 

> - - IPropertiesPlugin.setPropertiesForUser claims to only accept a  
> propertysheet object, but it would probably work fine with a simple  
> mapping (eyeballing the implementation in the  
> ZODBMutablePropertyProvider class). The contract could be changed to  
> accept mappings.

Indeed, I'll update the contract to that effect.

> - - setting properties on a user will magically turn around and persist  
> the change. This would be the first user folder I have seen where  
> this happens. Personally, I'm a bit uncomfortable with this magic.

setProperty on GRUFUser will persist the change as well.

Wichert.

-- 
Wichert Akkerman <wichert at wiggy.net>    It is simple to make things.
http://www.wiggy.net/                   It is hard to make things simple.