Skip to content

Commit 0d5fbf4

Browse files
committed
Fixed a problem with trying to use None as a Q object.
- Legacy-Id: 8202
1 parent 8727626 commit 0d5fbf4

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

ietf/ietfauth/utils.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,6 @@ def is_authorized_in_doc_stream(user, doc):
107107

108108
# must be authorized in the stream or group
109109

110-
group_req = None
111-
112110
if (not doc.stream or doc.stream.slug == "ietf") and has_role(user, ["Area Director"]):
113111
return True
114112

@@ -124,6 +122,8 @@ def is_authorized_in_doc_stream(user, doc):
124122
group_req = Q(group__acronym=doc.stream.slug) | Q(group=doc.group)
125123
elif doc.stream.slug in ("iab", "ise"):
126124
group_req = Q(group__acronym=doc.stream.slug)
125+
else:
126+
group_req = Q()
127127

128-
return bool(Role.objects.filter(Q(name__in=("chair", "secr", "delegate", "auth"), person__user=user) & group_req))
128+
return Role.objects.filter(Q(name__in=("chair", "secr", "delegate", "auth"), person__user=user) & group_req)
129129

0 commit comments

Comments
 (0)