From e852574a1718692dca8080fa97c19848082ea95d Mon Sep 17 00:00:00 2001
From: John Jarvis <jarv@edx.org>
Date: Mon, 6 May 2013 11:54:09 -0400
Subject: [PATCH] adding 404 handling for courses that do not exist

---
 lms/djangoapps/courseware/views.py            | 13 ++++++++--
 .../courseware/mktg_course_about.html         | 24 +++++++++++--------
 2 files changed, 25 insertions(+), 12 deletions(-)

diff --git a/lms/djangoapps/courseware/views.py b/lms/djangoapps/courseware/views.py
index 21bd2281ec..e5facee8f3 100644
--- a/lms/djangoapps/courseware/views.py
+++ b/lms/djangoapps/courseware/views.py
@@ -534,7 +534,15 @@ def course_about(request, course_id):
 @ensure_csrf_cookie
 @cache_if_anonymous
 def mktg_course_about(request, course_id):
-    course = get_course_with_access(request.user, course_id, 'see_exists')
+
+    coming_soon = False
+    try:
+        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)
 
     if has_access(request.user, course, 'load'):
@@ -549,7 +557,8 @@ def mktg_course_about(request, course_id):
                               {'course': course,
                                'registered': registered,
                                'course_target': course_target,
-                               'show_courseware_link': show_courseware_link})
+                               'show_courseware_link': show_courseware_link
+                               'coming_soon': coming_soon})
 
 
 
diff --git a/lms/templates/courseware/mktg_course_about.html b/lms/templates/courseware/mktg_course_about.html
index dc90b4e98c..d647d92887 100644
--- a/lms/templates/courseware/mktg_course_about.html
+++ b/lms/templates/courseware/mktg_course_about.html
@@ -63,18 +63,22 @@
 
   <ul class="list-actions register">
     <li class="item">
-      %if user.is_authenticated():
-        %if registered:
-          <!-- signed in, registered -->
-          <a class="action action-register is-registered" href="${course_target}">Registered for <strong>${course.number}</strong></a>
+    %if coming_soon:
+      <a class="action action-register is-registered" href="#">Coming Soon</a>
+    %else:
+        %if user.is_authenticated():
+          %if registered:
+            <!-- signed in, registered -->
+            <a class="action action-register is-registered" href="${course_target}">Registered for <strong>${course.number}</strong></a>
+          %else:
+          <!-- signed in, not registered -->
+            <a href="" class="action action-register register">Register for ${course.number}</a> <div id="register_message"></div>
+          %endif
         %else:
-        <!-- signed in, not registered -->
-          <a href="" class="action action-register register">Register for ${course.number}</a> <div id="register_message"></div>
+        <!-- not signed in -->
+            <a class="action action-register is-not-signedin" href="${reverse('register_user')}">Register for <strong>${course.number}</strong></a>
         %endif
-      %else:
-      <!-- not signed in -->
-          <a class="action action-register is-not-signedin" href="${reverse('register_user')}">Register for <strong>${course.number}</strong></a>
-      %endif
+    %endif
     </li>
   </ul>