Merge pull request ckan#8449 from tino097/8438-user-details-break-gro…

…up-page User details break group display
PublicaMundi · Sep 27, 2024 · 7e5f6a8 · 7e5f6a8
2 parents f13a00b + 66ac1f4
commit 7e5f6a8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/changes/8438.bugfix b/changes/8438.bugfix
@@ -0,0 +1,2 @@
+Template helper `member_count` will return 0
+for unauthorized users.
diff --git a/ckan/lib/helpers.py b/ckan/lib/helpers.py
@@ -2018,7 +2018,10 @@ def member_count(group: str) -> int:
         u'id': group,
         u'object_type': u'user'
     }
-    return len(logic.get_action(u'member_list')(context, data_dict))
+    try:
+        return len(logic.get_action(u'member_list')(context, data_dict))
+    except logic.NotAuthorized:
+        return 0
 
 
 @core_helper
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Template helper `member_count` will return 0
		for unauthorized users.