On 30 May 2005, at 21:35, Jean-Marc Orliaguet wrote:
I've done the last fixes on PluggableUserFolder, now the changes are:

- local roles were not merged from the parent folders
- mergedLocalRoles() according to PluginInterface getUsersWithRoles()
takes  an object as argument.
- mergedLocalRoles(): cannot expand a tuple, so we use an empty list
instead.
- mergedLocalRoles(): the 'result' dict was not defined before assignment.
- moved the 'No plugins able to identify user' log message from ERROR
to DEBUG level.
- getRolesInContext() did not take group roles ('roles:Authenticated')
into account
- mergedLocalRole(): optimization in dictionary access and fix:
group:roles where counted several times.
- mergedLocalRolesWithPath(): group:roles where counted several times.

I am going to add some tests, but at least the site's behaviour is
normal now.

PS: There is a feature in CPSUserFolder (in getRolesInContext()) where
the 'Anonymous' role is assigned to authenticated users, why?

I thought other user folders did that when I coded it in CPSUserFolder. It's actually not the case. It doesn't seem to do harm though.

Florent

--
Florent Guillaume, Nuxeo (Paris, France)   CTO, Director of R&D
+33 1 40 33 71 59   http://nuxeo.com   [EMAIL PROTECTED]


_______________________________________________
cps-devel mailing list
http://lists.nuxeo.com/mailman/listinfo/cps-devel

Reply via email to