Changeset 49505

Timestamp:

10/10/06 18:14:20 (3 years ago)

Author:

ogrisel

Message:

#1704 (related to): added two new properties to make special groups role:Authenticated and role:Anonymous optionaly behave as if they contain all the members of the portal instead of being empty by default.

Files:

• CPS3/products/CPSUserFolder/trunk/CHANGES (modified) (1 diff)
• CPS3/products/CPSUserFolder/trunk/CPSUserFolder.py (modified) (3 diffs)
• CPS3/products/CPSUserFolder/trunk/tests/testCPSUserFolder.py (modified) (2 diffs)

CPS3/products/CPSUserFolder/trunk/CHANGES

@@ -4 +4 @@
-New features
-~~~~~~~~~~~~
-
+
+
+
-Bug fixes
-~~~~~~~~~

CPS3/products/CPSUserFolder/trunk/CPSUserFolder.py

@@ -109 +109 @@
-        {'id': 'roles_members_field', 'type': 'string', 'mode': 'w',
-         'label': "Roles directory: members field"},
+        {'id': 'is_role_authenticated_empty', 'type': 'boolean', 'mode': 'w',
+         'label': "Consider role:Authenticated as an empty group"},
+        {'id': 'is_role_anonymous_empty', 'type': 'boolean', 'mode': 'w',
+         'label': "Consider role:Anonymous as an empty group"},
-        {'id': 'cache_timeout', 'type': 'int', 'mode': 'w',
-         'label': "Cache timeout"},
@@ -121 +125 @@
-    groups_members_field  = 'members'
-    roles_members_field = 'members'
+    is_role_authenticated_empty = True
+    is_role_anonymous_empty = True
-    cache_timeout = 300
-
@@ -542 +548 @@
-                    raise KeyError, groupname
-
-
-
-
+
+
-            else:
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-        else:
-            raise ValueError, "The directory %s doesn't exist" % self.groups_dir

CPS3/products/CPSUserFolder/trunk/tests/testCPSUserFolder.py

@@ -355 +355 @@
-        portal = self.portal
-        aclu = portal.aclu
+        mdir = portal.portal_directories.members
-        gdir = portal.portal_directories.groups
+
+        # Create some members
+        mdir.createEntry({'uid': 'mickey'})
+        mdir.createEntry({'uid': 'pluto'})
-
-        # Create a new group using the directory.
@@ -373 +378 @@
-
-        # Backward compatibility with UserFolderWithGroups
+        self.failUnless(aclu.is_role_authenticated_empty)
-        group = aclu.getGroupById('role:Authenticated')
-        self.assertEquals(group.getUsers(), ())
+
+        # now make role:Authenticated as a special group containing everybody
+        aclu.is_role_authenticated_empty = False
+        group = aclu.getGroupById('role:Authenticated')
+        self.assertEquals(group.getUsers(), ['mickey', 'pluto'])
+
+        # same behavior for role:Anonymous
+        self.failUnless(aclu.is_role_anonymous_empty)
+        group = aclu.getGroupById('role:Anonymous')
+        self.assertEquals(group.getUsers(), ())
+
+        # now make role:Anonymous as a special group containing everybody
+        aclu.is_role_anonymous_empty = False
+        group = aclu.getGroupById('role:Anonymous')
+        self.assertEquals(group.getUsers(), ['mickey', 'pluto'])
-
-    def test_group_API_security(self):