From b2af990d3cf0d4b9a0b074de28c87328ced891d6 Mon Sep 17 00:00:00 2001 From: Usman Khalid Date: Mon, 23 Sep 2013 15:19:32 +0000 Subject: [PATCH] If setting COURSES_ARE_BROWSABLE is off return 404 on /courses --- lms/djangoapps/branding/views.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lms/djangoapps/branding/views.py b/lms/djangoapps/branding/views.py index 6a8ef8056b..cdf7aa5eda 100644 --- a/lms/djangoapps/branding/views.py +++ b/lms/djangoapps/branding/views.py @@ -1,5 +1,6 @@ from django.conf import settings from django.core.urlresolvers import reverse +from django.http import Http404 from django.shortcuts import redirect from django_future.csrf import ensure_csrf_cookie from mitxmako.shortcuts import render_to_response @@ -48,6 +49,9 @@ def courses(request): if settings.MITX_FEATURES.get('ENABLE_MKTG_SITE', False): return redirect(marketing_link('COURSES'), permanent=True) + if not settings.MITX_FEATURES.get('COURSES_ARE_BROWSABLE'): + raise Http404 + university = branding.get_university(request.META.get('HTTP_HOST')) if university == 'edge': return render_to_response('university_profile/edge.html', {})