Merge pull request #15058 from edx/diana/course-date-info-fix

Only try to render dates when user is authenticated.

lms/djangoapps/courseware/tests/test_date_summary.py

@@ -96,10 +96,17 @@ class CourseDateSummaryTest(SharedModuleStoreTestCase):
     def test_course_info_feature_flag(self):
         SelfPacedConfiguration(enable_course_home_improvements=False).save()
         self.setup_course_and_user()
+        self.client.login(username='mrrobot', password='test')
         url = reverse('info', args=(self.course.id,))
         response = self.client.get(url)
         self.assertNotIn('date-summary', response.content)
 
+    def test_course_info_logged_out(self):
+        self.setup_course_and_user()
+        url = reverse('info', args=(self.course.id,))
+        response = self.client.get(url)
+        self.assertEqual(200, response.status_code)
+
     # Tests for which blocks are enabled
     def assert_block_types(self, expected_blocks):
         """Assert that the enabled block types for this course are as expected."""

lms/djangoapps/courseware/views/views.py

@@ -334,10 +334,10 @@ def course_info(request, course_id):
 
         # Construct the dates fragment
         dates_fragment = None
-        if SelfPacedConfiguration.current().enable_course_home_improvements:
-            dates_fragment = CourseDatesFragmentView().render_to_fragment(request, course_id=course_id)
 
         if request.user.is_authenticated():
+            if SelfPacedConfiguration.current().enable_course_home_improvements:
+                dates_fragment = CourseDatesFragmentView().render_to_fragment(request, course_id=course_id)
             show_upgrade_notification = False
             if request.GET.get('upgrade', 'false') == 'true':
                 store_upgrade_cookie = True