From 6f568d097c930c7284f354887cb7784e7ecb276f Mon Sep 17 00:00:00 2001
From: Calen Pennington <cale@edx.org>
Date: Thu, 28 May 2020 14:47:13 -0400
Subject: [PATCH] Set missed_deadlines in the course outlines template to
 simplify the logic in the dates_banner.html

---
 lms/templates/dates_banner.html                 | 17 +++--------------
 .../course-outline-fragment.html                |  2 +-
 .../course_experience/views/course_outline.py   |  3 +++
 3 files changed, 7 insertions(+), 15 deletions(-)

diff --git a/lms/templates/dates_banner.html b/lms/templates/dates_banner.html
index 984a82b10c..b774970378 100644
--- a/lms/templates/dates_banner.html
+++ b/lms/templates/dates_banner.html
@@ -104,20 +104,9 @@ additional_styling_class = 'on-mobile' if is_mobile_app else 'has-button'
         % endif
     </div>
 </%def>
-% if (missed_deadlines and (on_dates_tab_as_verified or on_courseware_page)):
-    % if is_verified:
-        ${reset_dates_banner()}
-    % elif is_audit:
-        ${upgrade_to_reset_banner()}
-    % endif
-% elif on_dates_tab_as_audit and has_locked_assignments:
-    % if missed_deadlines:
-        ${upgrade_to_reset_banner()}
-    % else:
-        ${upgrade_to_complete_graded_banner()}
-    % endif
-% elif on_course_outline_page and (is_audit or is_verified):
-## The outline page uses javascript to show the dates banner, so everything below is only displayed when a user has actually missed deadlines
+% if on_dates_tab and is_audit and has_locked_assignments and not missed_deadlines:
+    ${upgrade_to_complete_graded_banner()}
+% elif missed_deadlines:
     % if is_verified:
         ${reset_dates_banner()}
     % elif is_audit:
diff --git a/openedx/features/course_experience/templates/course_experience/course-outline-fragment.html b/openedx/features/course_experience/templates/course_experience/course-outline-fragment.html
index 5435b2370c..f27f24dd84 100644
--- a/openedx/features/course_experience/templates/course_experience/course-outline-fragment.html
+++ b/openedx/features/course_experience/templates/course_experience/course-outline-fragment.html
@@ -27,7 +27,7 @@ dates_banner_displayed = False
 %>
 <main role="main" class="course-outline" id="main" tabindex="-1">
     <div class="dates-banner-wrapper">
-        % if enrollment_mode and relative_dates_flag_is_enabled and self_paced and not is_course_staff:
+        % if relative_dates_flag_is_enabled and self_paced and not is_course_staff:
             <%include file="/dates_banner.html" />
         % endif
     </div>
diff --git a/openedx/features/course_experience/views/course_outline.py b/openedx/features/course_experience/views/course_outline.py
index ba98859c79..93769183fc 100644
--- a/openedx/features/course_experience/views/course_outline.py
+++ b/openedx/features/course_experience/views/course_outline.py
@@ -112,6 +112,9 @@ class CourseOutlineFragmentView(EdxFragmentView):
             user=request.user,
             course_key=course_key
         )
+        # We use javascript to check whether to actually display this banner, so we let the banner assume
+        # that deadlines have been missed.
+        context['missed_deadlines'] = True
 
         html = render_to_string('course_experience/course-outline-fragment.html', context)
         return Fragment(html)