fix url mapping bug in url.py and include import reference to remove_user_from_course_group

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)

cms/urls.py

@@ -25,7 +25,8 @@ urlpatterns = ('',
     url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/course/(?P<name>[^/]+)/add_user$',
         'contentstore.views.add_user', name='add_user'),
     url(r'^(?P<org>[^/]+)/(?P<course>[^/]+)/course/(?P<name>[^/]+)/remove_user$',
-        'contentstore.views.add_user', name='remove_user')
+        'contentstore.views.remove_user', name='remove_user')
+
 )
 
 # User creation and updating views