From cffa01331427fb0b4a6698b538fc5c1774afcb0d Mon Sep 17 00:00:00 2001
From: David Baumgold <david@davidbaumgold.com>
Date: Thu, 11 Jul 2013 09:30:50 -0400
Subject: [PATCH] Fix course team JS to handle non-200 status codes

---
 cms/templates/manage_users.html | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/cms/templates/manage_users.html b/cms/templates/manage_users.html
index 462dd32c81..b39df0cede 100644
--- a/cms/templates/manage_users.html
+++ b/cms/templates/manage_users.html
@@ -67,6 +67,7 @@
 <%block name="jsextra">
 <script type="text/javascript">
   var $newUserForm;
+  var addUserPostbackUrl = "${add_user_postback_url}";
 
   function showNewUserForm(e) {
     e.preventDefault();
@@ -91,16 +92,19 @@
     e.preventDefault();
 
     $.ajax({
-      url: '${add_user_postback_url}',
+      url: addUserPostbackUrl,
       type: 'POST',
       dataType: 'json',
       contentType: 'application/json',
-      data:JSON.stringify({ 'email': $('#email').val()}),
-    }).done(function(data) {
-      if (data.ErrMsg != undefined)
-        $('#result').show().empty().append(data.ErrMsg);
-      else
+      data: JSON.stringify({ 'email': $('#email').val()}),
+      success: function(data) {
         location.reload();
+      },
+      notifyOnError: false,
+      error: function(jqXHR, textStatus, errorThrown) {
+        data = JSON.parse(jqXHR.responseText);
+        $('#result').show().empty().append(data.ErrMsg);
+      }
     });
   }