diff --git a/common/lib/xmodule/xmodule/course_module.py b/common/lib/xmodule/xmodule/course_module.py index 73f33f7dc4..e595d6bd50 100644 --- a/common/lib/xmodule/xmodule/course_module.py +++ b/common/lib/xmodule/xmodule/course_module.py @@ -958,17 +958,6 @@ class CourseFields(object): scope=Scope.settings ) - bypass_home = Boolean( - display_name=_("Bypass Course Home"), - help=_( - "Bypass the course home tab when students arrive from the dashboard, " - "sending them directly to course content." - ), - default=False, - scope=Scope.settings, - deprecated=True - ) - enable_subsection_gating = Boolean( display_name=_("Enable Subsection Prerequisites"), help=_( diff --git a/common/lib/xmodule/xmodule/tests/test_course_module.py b/common/lib/xmodule/xmodule/tests/test_course_module.py index 84c1be9a85..2a2120e013 100644 --- a/common/lib/xmodule/xmodule/tests/test_course_module.py +++ b/common/lib/xmodule/xmodule/tests/test_course_module.py @@ -375,17 +375,6 @@ class SelfPacedTestCase(unittest.TestCase): self.assertFalse(self.course.self_paced) -class BypassHomeTestCase(unittest.TestCase): - """Tests for setting which allows course home to be bypassed.""" - - def setUp(self): - super(BypassHomeTestCase, self).setUp() - self.course = get_dummy_course('2012-12-02T12:00') - - def test_default(self): - self.assertFalse(self.course.bypass_home) - - class CourseDescriptorTestCase(unittest.TestCase): """ Tests for a select few functions from CourseDescriptor. diff --git a/lms/djangoapps/courseware/tests/test_views.py b/lms/djangoapps/courseware/tests/test_views.py index 98827a369f..cde89978b5 100644 --- a/lms/djangoapps/courseware/tests/test_views.py +++ b/lms/djangoapps/courseware/tests/test_views.py @@ -907,22 +907,12 @@ class ViewsTestCase(BaseViewsTestCase): def test_bypass_course_info(self): course_id = six.text_type(self.course_key) - self.assertFalse(self.course.bypass_home) - response = self.client.get(reverse('info', args=[course_id])) self.assertEqual(response.status_code, 200) response = self.client.get(reverse('info', args=[course_id]), HTTP_REFERER=reverse('dashboard')) self.assertEqual(response.status_code, 200) - self.course.bypass_home = True - self.store.update_item(self.course, self.user.id) - self.assertTrue(self.course.bypass_home) - - response = self.client.get(reverse('info', args=[course_id]), HTTP_REFERER=reverse('dashboard')) - - self.assertRedirects(response, reverse('courseware', args=[course_id]), fetch_redirect_response=False) - response = self.client.get(reverse('info', args=[course_id]), HTTP_REFERER='foo') self.assertEqual(response.status_code, 200) diff --git a/lms/djangoapps/courseware/views/views.py b/lms/djangoapps/courseware/views/views.py index 60562278f1..ae867e3d1b 100644 --- a/lms/djangoapps/courseware/views/views.py +++ b/lms/djangoapps/courseware/views/views.py @@ -460,13 +460,6 @@ def course_info(request, course_id): if not user_can_skip_entrance_exam(user, course): return redirect(reverse('courseware', args=[text_type(course.id)])) - # TODO: LEARNER-611: Remove deprecated course.bypass_home. - # If the user is coming from the dashboard and bypass_home setting is set, - # redirect them straight to the courseware page. - is_from_dashboard = reverse('dashboard') in request.META.get('HTTP_REFERER', []) - if course.bypass_home and is_from_dashboard: - return redirect(reverse('courseware', args=[course_id])) - # Construct the dates fragment dates_fragment = None