diff --git a/cms/djangoapps/contentstore/views.py b/cms/djangoapps/contentstore/views.py index 8d5c32b78d..e8e359f0f1 100644 --- a/cms/djangoapps/contentstore/views.py +++ b/cms/djangoapps/contentstore/views.py @@ -40,7 +40,8 @@ from xmodule.contentstore.content import StaticContent from cache_toolbox.core import set_cached_content, get_cached_content, del_cached_content from auth.authz import is_user_in_course_group_role, get_users_in_course_group_by_role -from auth.authz import get_user_by_email, add_user_to_course_group, ADMIN_ROLE_NAME, EDITOR_ROLE_NAME +from auth.authz import get_user_by_email, add_user_to_course_group, remove_user_from_course_group +from auth.authz import ADMIN_ROLE_NAME, EDITOR_ROLE_NAME from .utils import get_course_location_for_item log = logging.getLogger(__name__) @@ -598,7 +599,7 @@ def remove_user(request, org, course, name): # check that logged in user has admin permissions on this course if not has_access(request.user, location, role=ADMIN_ROLE_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) diff --git a/cms/urls.py b/cms/urls.py index 8868e923de..141abac2ac 100644 --- a/cms/urls.py +++ b/cms/urls.py @@ -25,7 +25,8 @@ urlpatterns = ('', url(r'^(?P[^/]+)/(?P[^/]+)/course/(?P[^/]+)/add_user$', 'contentstore.views.add_user', name='add_user'), url(r'^(?P[^/]+)/(?P[^/]+)/course/(?P[^/]+)/remove_user$', - 'contentstore.views.add_user', name='remove_user') + 'contentstore.views.remove_user', name='remove_user') + ) # User creation and updating views