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

Check if effective_user is anonymous before trying to get grade

Browse Source 
I saw locally that it caused an error if you hit the endpoint directly
so just being defensive.
This commit is contained in:
Dillon Dumesnil
2020-10-06 12:49:03 -04:00
parent 67cddb7ce7
commit fa2277bb6b
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -210,9 +210,11 @@ class CoursewareMeta:
@property @property
def user_has_passing_grade(self): def user_has_passing_grade(self):
""" Returns a boolean on if the effective_user has a passing grade in the course """ """ Returns a boolean on if the effective_user has a passing grade in the course """
course = get_course_by_id(self.course_key) if not self.effective_user.is_anonymous:
user_grade = CourseGradeFactory().read(self.effective_user, course).percent course = get_course_by_id(self.course_key)
return user_grade >= course.lowest_passing_grade user_grade = CourseGradeFactory().read(self.effective_user, course).percent
return user_grade >= course.lowest_passing_grade
return False
@property @property
def course_exit_page_is_active(self): def course_exit_page_is_active(self):