Commit e852574a by John Jarvis

adding 404 handling for courses that do not exist

parent bf5ac6c2
...@@ -534,7 +534,15 @@ def course_about(request, course_id): ...@@ -534,7 +534,15 @@ def course_about(request, course_id):
@ensure_csrf_cookie @ensure_csrf_cookie
@cache_if_anonymous @cache_if_anonymous
def mktg_course_about(request, course_id): def mktg_course_about(request, course_id):
coming_soon = False
try:
course = get_course_with_access(request.user, course_id, 'see_exists') course = get_course_with_access(request.user, course_id, 'see_exists')
except: Http404 as e:
# if a course does not exist yet, display a coming
# soon button
coming_soon = True
registered = registered_for_course(course, request.user) registered = registered_for_course(course, request.user)
if has_access(request.user, course, 'load'): if has_access(request.user, course, 'load'):
...@@ -549,7 +557,8 @@ def mktg_course_about(request, course_id): ...@@ -549,7 +557,8 @@ def mktg_course_about(request, course_id):
{'course': course, {'course': course,
'registered': registered, 'registered': registered,
'course_target': course_target, 'course_target': course_target,
'show_courseware_link': show_courseware_link}) 'show_courseware_link': show_courseware_link
'coming_soon': coming_soon})
......
...@@ -63,6 +63,9 @@ ...@@ -63,6 +63,9 @@
<ul class="list-actions register"> <ul class="list-actions register">
<li class="item"> <li class="item">
%if coming_soon:
<a class="action action-register is-registered" href="#">Coming Soon</a>
%else:
%if user.is_authenticated(): %if user.is_authenticated():
%if registered: %if registered:
<!-- signed in, registered --> <!-- signed in, registered -->
...@@ -75,6 +78,7 @@ ...@@ -75,6 +78,7 @@
<!-- not signed in --> <!-- not signed in -->
<a class="action action-register is-not-signedin" href="${reverse('register_user')}">Register for <strong>${course.number}</strong></a> <a class="action action-register is-not-signedin" href="${reverse('register_user')}">Register for <strong>${course.number}</strong></a>
%endif %endif
%endif
</li> </li>
</ul> </ul>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment