Skip to content

Commit d98cecc

Browse files
committed
refactor: replace has_access with administrative_accesses_to_course_for_user
1 parent c4e07da commit d98cecc

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

openedx/core/djangoapps/course_groups/permissions.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
from rest_framework import permissions
77

88
from common.djangoapps.student.roles import GlobalStaff
9-
from lms.djangoapps.courseware.access import has_access
9+
from lms.djangoapps.courseware.access import administrative_accesses_to_course_for_user
1010
from lms.djangoapps.discussion.django_comment_client.utils import get_user_role_names
1111
from openedx.core.djangoapps.django_comment_common.models import (
1212
FORUM_ROLE_ADMINISTRATOR,
@@ -34,5 +34,5 @@ def has_permission(self, request, view):
3434
FORUM_ROLE_COMMUNITY_TA,
3535
}
3636
)
37-
has_course_team_access = has_access(request.user, "staff", course_key).has_access
38-
return has_course_team_access or has_discussion_privileges and request.method == "GET"
37+
_, staff_access, instructor_access = administrative_accesses_to_course_for_user(request.user, course_key)
38+
return staff_access or instructor_access or (has_discussion_privileges and request.method == "GET")

0 commit comments

Comments
 (0)