Commit a2168c2c by Matt Drayer Committed by Jonathan Piacenti

mattdrayer/api-remove-isactive-user-filter: Removed registration check from older operations

parent 2f4defa6
...@@ -1144,7 +1144,7 @@ class CoursesUsersDetail(SecureAPIView): ...@@ -1144,7 +1144,7 @@ class CoursesUsersDetail(SecureAPIView):
DELETE /api/courses/{course_id}/users/{user_id} DELETE /api/courses/{course_id}/users/{user_id}
""" """
try: try:
user = User.objects.get(id=user_id, is_active=True) user = User.objects.get(id=user_id)
except ObjectDoesNotExist: except ObjectDoesNotExist:
return Response({}, status=status.HTTP_204_NO_CONTENT) return Response({}, status=status.HTTP_204_NO_CONTENT)
if not course_exists(request, request.user, course_id): if not course_exists(request, request.user, course_id):
......
...@@ -684,7 +684,7 @@ class UsersGroupsDetail(SecureAPIView): ...@@ -684,7 +684,7 @@ class UsersGroupsDetail(SecureAPIView):
""" """
DELETE /api/users/{user_id}/groups/{group_id} DELETE /api/users/{user_id}/groups/{group_id}
""" """
existing_user = User.objects.get(id=user_id, is_active=True) existing_user = User.objects.get(id=user_id)
existing_user.groups.remove(group_id) existing_user.groups.remove(group_id)
existing_user.save() existing_user.save()
return Response({}, status=status.HTTP_204_NO_CONTENT) return Response({}, status=status.HTTP_204_NO_CONTENT)
...@@ -828,7 +828,7 @@ class UsersCoursesDetail(SecureAPIView): ...@@ -828,7 +828,7 @@ class UsersCoursesDetail(SecureAPIView):
response_data = {} response_data = {}
response_data['uri'] = base_uri response_data['uri'] = base_uri
try: try:
user = User.objects.get(id=user_id, is_active=True) user = User.objects.get(id=user_id)
except ObjectDoesNotExist: except ObjectDoesNotExist:
return Response({}, status=status.HTTP_404_NOT_FOUND) return Response({}, status=status.HTTP_404_NOT_FOUND)
if not course_exists(request, user, course_id): if not course_exists(request, user, course_id):
...@@ -896,7 +896,7 @@ class UsersCoursesDetail(SecureAPIView): ...@@ -896,7 +896,7 @@ class UsersCoursesDetail(SecureAPIView):
DELETE /api/users/{user_id}/courses/{course_id} DELETE /api/users/{user_id}/courses/{course_id}
""" """
try: try:
user = User.objects.get(id=user_id, is_active=True) user = User.objects.get(id=user_id)
except ObjectDoesNotExist: except ObjectDoesNotExist:
return Response({}, status=status.HTTP_204_NO_CONTENT) return Response({}, status=status.HTTP_204_NO_CONTENT)
if not course_exists(request, user, course_id): if not course_exists(request, user, course_id):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment