Merge pull request #368 from edx/db/fix-course-team-add-user

Fix course team JS to handle non-200 status codes

cms/templates/manage_users.html

@@ -67,6 +67,8 @@
 <%block name="jsextra">
 <script type="text/javascript">
   var $newUserForm;
+  var addUserPostbackUrl = "${add_user_postback_url}";
+  var removeUserPostbackUrl = "${remove_user_postback_url}";
 
   function showNewUserForm(e) {
     e.preventDefault();
@@ -91,16 +93,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);
+      }
     });
   }
 
@@ -115,7 +120,7 @@
 
     $('.remove-user').click(function() {
       $.ajax({
-        url: '${remove_user_postback_url}',
+        url: removeUserPostbackUrl,
         type: 'POST',
         dataType: 'json',
         contentType: 'application/json',