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

Merge pull request #17872 from edx/revert-17796-aj/COURSE_LOADING_ERROR

Browse Source 
Revert "Logging: Error getting the course from modulestore"
This commit is contained in:
Awais Jibran
2018-04-04 19:14:06 +05:00
committed by GitHub
parent 315c06c633 e12120befe
commit 02973250ea
1 changed files with 0 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
openedx/core/djangoapps/course_groups/cohorts.py
View File

@@ -19,8 +19,6 @@ from eventtracking import tracker
from openedx.core.djangoapps.request_cache import clear_cache, get_cache
from openedx.core.djangoapps.request_cache.middleware import request_cached
from student.models import get_user_by_username_or_email
from xmodule.course_module import CourseDescriptor
from xmodule.error_module import ErrorDescriptor
from .models import (
CohortMembership,
@@ -283,12 +281,6 @@ def get_random_cohort(course_key):
If there are no existing cohorts of type RANDOM in the course, one will be created.
"""
course = courses.get_course(course_key)
if not isinstance(course, CourseDescriptor):
course_or_error_msg = course.error_msg if isinstance(course, ErrorDescriptor) else unicode(course)
log.error(
"COURSE_LOADING_ERROR: Error while loading course %s from the module store: %s",
unicode(course_key), course_or_error_msg
)
cohorts = get_course_cohorts(course, assignment_type=CourseCohort.RANDOM)
if cohorts:
cohort = local_random().choice(cohorts)