[Zope-PAS] mutable property sheets
jens at dataflake.org
Sun Apr 16 12:18:32 EDT 2006
-----BEGIN PGP SIGNED MESSAGE-----
On 16 Apr 2006, at 17:46, Wichert Akkerman wrote:
> 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.
Ah, right, that's how something is retrieved that will actuate the
>> - - setting properties on a user will magically turn around and
>> 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.
I hate to say it, but GRUF isn't an example for anything right or
good when it comes to user folders. And that opinion goes right up to
the top in Plone circles.
So my own opinion is that I don't like the persistence magic for
setting properties. IMHO the methods on the properties plugins should
be called explicitly instead of automatically whenever anyone touches
one of these mutable property sheets.
This is just one opinion, I hope others will chime in as well.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.1 (Darwin)
-----END PGP SIGNATURE-----
More information about the Zope-PAS