From ee65c8783db3c305914d19e6a39e4d02fdc4a1f2 Mon Sep 17 00:00:00 2001 From: Waheed Ahmed Date: Tue, 17 May 2016 18:41:22 +0500 Subject: [PATCH] Use CourseOverview instead of modulestore. --- lms/djangoapps/certificates/signals.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lms/djangoapps/certificates/signals.py b/lms/djangoapps/certificates/signals.py index 71f8f691b4..4cab739d5f 100644 --- a/lms/djangoapps/certificates/signals.py +++ b/lms/djangoapps/certificates/signals.py @@ -6,7 +6,8 @@ from django.dispatch.dispatcher import receiver from certificates.models import CertificateGenerationCourseSetting from opaque_keys.edx.keys import CourseKey -from xmodule.modulestore.django import SignalHandler, modulestore +from openedx.core.djangoapps.content.course_overviews.models import CourseOverview +from xmodule.modulestore.django import SignalHandler @receiver(SignalHandler.course_published) @@ -22,7 +23,7 @@ def _listen_for_course_publish(sender, course_key, **kwargs): # pylint: disable def enable_self_generated_certs(course_key): """Enable the self-generated certificates by default for self-paced courses.""" course_key = CourseKey.from_string(course_key) - course = modulestore().get_course(course_key) + course = CourseOverview.get_from_id(course_key) is_enabled_for_course = CertificateGenerationCourseSetting.is_enabled_for_course(course_key) if course.self_paced and not is_enabled_for_course: CertificateGenerationCourseSetting.set_enabled_for_course(course_key, True)