Skip to content

Commit 98fed27

Browse files
committed
Fixed remove user error
1 parent ee7bc25 commit 98fed27

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

Alpha/backend/lib/views/admin_area.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,8 @@ def post(self, request ,*args, **kwargs):
5858
def delete_user(request, user_id):
5959
try:
6060
if request.user.is_staff:
61-
user = CustomUser.objects.get(id=int(user_id))
62-
if len(CustomUser.objects.filter(is_staff=True)) < 2:
61+
user = CustomUser.objects.get(id=int(user_id)) # Get user
62+
if (len(CustomUser.objects.filter(is_staff=True)) < 2) and (user.is_staff == True):
6363
return JsonResponse({'success': False, 'result_user': 'Problem deleting user: Make sure there is more than one admin account on the system'})
6464
activity = GetProject.create_activity(user=request.user, description=f'Admin #{request.user.id} delete user ID #{user_id}')
6565
user.delete()

Alpha/backend/lib/views/members.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,6 @@ def delete_member(request, project_id, user_id):
9494
user = CustomUser.objects.get(id=int(user_id))
9595
except:
9696
return JsonResponse({'success': False, 'message': 'User not found'})
97-
#print(user.username)
9897
project.members.remove(user)
9998
activity = GetProject.create_activity(project = project, user=request.user, description=f'USER #{request.user.id} remove a member ID #{user_id} from project #{project_id}')
10099
return JsonResponse({'success': True, 'message': 'Member deleted.'})

0 commit comments

Comments
 (0)