add error response when trying to remove a user which does not exist

2012-10-01 10:48:57 -04:00
parent ea8e875d7d
commit d241d5553c
1 changed files with 4 additions and 2 deletions
--- a/cms/djangoapps/contentstore/views.py
+++ b/cms/djangoapps/contentstore/views.py
@@ -601,8 +601,10 @@ def remove_user(request, org, course, name):
        raise PermissionDenied()

    user = get_user_by_email(email)
-    if user is not None:
-        remove_user_from_course_group(request.user, user, location, EDITOR_ROLE_NAME)
+    if user is None:
+        return create_json_response('Could not find user by email address \'{0}\'.'.format(email))
+
+    remove_user_from_course_group(request.user, user, location, EDITOR_ROLE_NAME)

    return create_json_response()