From d241d5553c04ddd2ed7ca465f8081c060fa4acaf Mon Sep 17 00:00:00 2001 From: Chris Dodge Date: Mon, 1 Oct 2012 10:48:57 -0400 Subject: [PATCH] add error response when trying to remove a user which does not exist --- cms/djangoapps/contentstore/views.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/cms/djangoapps/contentstore/views.py b/cms/djangoapps/contentstore/views.py index e8e359f0f1..9a9653a0c1 100644 --- 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()