permissions: add member policy generator

inveniosoftware · Jul 30, 2024 · a347562 · a347562
1 parent 1498d94
commit a347562
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 8 deletions.
diff --git a/invenio_communities/generators.py b/invenio_communities/generators.py
@@ -161,16 +161,36 @@ def __init__(self, then_, else_):
         """Initialize."""
         field = "record_policy"
         super().__init__(
-            lambda r: (
+            field_getter=lambda r: (
                 getattr(r.access, field, None)
                 if hasattr(r, "access")
                 else r.get("access", {}).get(field)
             ),  # needed for running permission check at serialization time and avoid db query
-            f"access.{field}",
-            "closed",
-            "open",
-            then_,
-            else_,
+            field_name=f"access.{field}",
+            then_value="closed",
+            else_value="open",
+            then_=then_,
+            else_=else_,
+        )
+
+
+class IfMemberPolicyClosed(IfRestrictedBase):
+    """If member policy is closed."""
+
+    def __init__(self, then_, else_):
+        """Initialize."""
+        field = "member_policy"
+        super().__init__(
+            field_getter=lambda r: (
+                getattr(r.access, field, None)
+                if hasattr(r, "access")
+                else r.get("access", {}).get(field)
+            ),  # needed for running permission check at serialization time and avoid db query
+            field_name=f"access.{field}",
+            then_value="closed",
+            else_value="open",
+            then_=then_,
+            else_=else_,
         )
 
 

diff --git a/invenio_communities/permissions.py b/invenio_communities/permissions.py
@@ -33,6 +33,7 @@
     CommunityOwners,
     CommunitySelfMember,
     IfCommunityDeleted,
+    IfMemberPolicyClosed,
     IfRecordPolicyClosed,
     IfRestricted,
     ReviewPolicy,
@@ -190,8 +191,7 @@ class CommunityPermissionPolicy(BasePermissionPolicy):
         IfConfig(
             "COMMUNITIES_ALLOW_MEMBERSHIP_REQUESTS",
             then_=[
-                IfPolicyClosed(
-                    "member_policy",
+                IfMemberPolicyClosed(
                     then_=[Disable()],
                     else_=[AuthenticatedButNotCommunityMembers()],
                 ),