Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor public course check to only call if needed

Browse Source
This commit is contained in:
Adam Butterworth
2020-04-03 09:55:28 -04:00
parent ff33cc54dd
commit 4363b1ede0
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
openedx/core/djangoapps/courseware_api/views.py
View File

@@ -89,12 +89,11 @@ class CoursewareInformation(RetrieveAPIView):
if not has_access(user, 'load', overview):
return False
has_public_access = allow_public_access(overview, [COURSE_VISIBILITY_PUBLIC])
if user.is_anonymous and not has_public_access:
return False
if not CourseEnrollment.is_enrolled(user, overview.id) and not has_public_access:
return False
# Anonymous or unenrolled users
if user.is_anonymous or not CourseEnrollment.is_enrolled(user, overview.id):
# do not have access if the course is not public
if not allow_public_access(overview, [COURSE_VISIBILITY_PUBLIC]):
return False
# if is_survey_required_and_unanswered(user, course):
# TODO: This.