Changeset 6202

Timestamp:

26 May 2011, 23:34:04 (13 years ago)

Author:

uli

Message:

#37

Location:

main/waeup.sirp/trunk/src/waeup/sirp

Files:

• authentication.txt (modified) (1 diff)
• permissions.py (modified) (1 diff)
• permissions.txt (modified) (1 diff)
• users.py (modified) (1 diff)

main/waeup.sirp/trunk/src/waeup/sirp/authentication.txt

@@ -111 +111 @@
    {}
 
+If one notifies the machinery of a local role removal for an object
+that cannot have local roles, this will cause no trouble:
+
+   >>> mycontext = None #object()
+   >>> notify(LocalRoleSetEvent(
+   ...   mycontext, 'waeup.PortalManager', 'bob', granted=False
+   ... )) is None
+   True
+
 Logging in via side bar
 =======================

main/waeup.sirp/trunk/src/waeup/sirp/permissions.py

@@ -147 +147 @@
     roles map of the `context` object.
     """
-    role_map = IPrincipalRoleMap(context)
+    try:
+        role_map = IPrincipalRoleMap(context)
+    except TypeError:
+        # no map no roles.
+        raise StopIteration
     for local_role, user_name, setting in role_map.getPrincipalsAndRoles():
         user = grok.getSite()['users'].get(user_name,None)

main/waeup.sirp/trunk/src/waeup/sirp/permissions.txt

@@ -54 +54 @@
     >>> list(getWAeUPRoleNames())
     [u'waeup.ApplicationsOfficer', u'waeup.PortalManager', u'waeup.PortalUser']
+
+:func:`get_users_with_local_roles`
+----------------------------------
+
+We can get all users and their roles for a certain context
+object. This even works for objects that cannot have local roles as
+they are not stored in the ZODB:
+
+    >>> from waeup.sirp.permissions import get_users_with_local_roles
+    >>> mycontext = object()
+    >>> people_and_roles = get_users_with_local_roles(mycontext)
+    >>> people_and_roles
+    <generator object at 0x...>
+
+In this case, the result is empty:
+
+    >>> people_and_roles = list(people_and_roles)
+    >>> people_and_roles
+    []

main/waeup.sirp/trunk/src/waeup/sirp/users.py

@@ -63 +63 @@
 @grok.subscribe(Interface, grok.IObjectRemovedEvent)
 def handle_local_roles_on_obj_removal(obj, event):
-    role_map = IPrincipalRoleMap(obj)
+    try:
+        role_map = IPrincipalRoleMap(obj)
+    except TypeError:
+        # no map, no roles to remove
+        return
     for local_role, user_name, setting in role_map.getPrincipalsAndRoles():
         notify(LocalRoleSetEvent(