From d465d6f779fa6cd730ad5302b3d9097f7486accc Mon Sep 17 00:00:00 2001 From: Nimisha Asthagiri Date: Wed, 14 Jun 2017 12:57:23 -0400 Subject: [PATCH] Planning Prompts experiment: exclude non-English courses --- lms/djangoapps/courseware/views/views.py | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/lms/djangoapps/courseware/views/views.py b/lms/djangoapps/courseware/views/views.py index e564ccc191..b956a0c47d 100644 --- a/lms/djangoapps/courseware/views/views.py +++ b/lms/djangoapps/courseware/views/views.py @@ -525,7 +525,7 @@ class CourseTabView(EdxFragmentView): # TODO: (Experimental Code). See https://openedx.atlassian.net/wiki/display/RET/3.+Planning+Prompts -def _should_display_planning_prompt(request, course_overview): +def _should_display_planning_prompt(request, course): """ A planning prompt is enabled in the experiment for all enrollments whose content availability date is less than 14 days from today. @@ -533,12 +533,13 @@ def _should_display_planning_prompt(request, course_overview): The content availability date is defined as either the course start date or the enrollment date, whichever was most recent. """ - enrollment = CourseEnrollment.get_enrollment(request.user, course_overview.id) - if enrollment: - content_availability_date = max(course_overview.start, enrollment.created) - return content_availability_date > (datetime.now(utc) - timedelta(days=14)) - else: - return False + is_course_in_english = not course.language or course.language.lower() == u'en' + if is_course_in_english: + enrollment = CourseEnrollment.get_enrollment(request.user, course.id) + if enrollment and enrollment.is_active: + content_availability_date = max(course.start, enrollment.created) + return content_availability_date > (datetime.now(utc) - timedelta(days=14)) + return False # ENDTODO