Bulk editing all users as admin removes other admins

[Radiicall]

Description

When editing user perms as admin other admins are usually greyed out but if you bulk edit and click the all users checkbox then it will still edit the admins, thus removing their administrative permission.

Version

2.3.0

Steps to Reproduce

1. Go to Users AS AN ADMIN NOT OWNER
2. Click the checkbox for all users and then click bulk edit
3. Change the permissions to anything
4. Other admins have been demoted to normal users

Screenshots

Logs

Platform

desktop

Database

SQLite (default)

Device

Desktop

Operating System

Any

Browser

Zen Browser (Firefox based)

Additional Context

No response

Code of Conduct

• I agree to follow Jellyseerr's Code of Conduct

[fallenbagel]

What do you mean? This is working as intended?
When you bulk edit you're given a clean slate of permissions to choose and you're basically removing admin permission from them when you select everyone. Admins can edit other users permissions including other admins. When you edit a single admin if you untick admin permission that user will become a normal user. That's what happens when you bulk edit. You're removing the admin permission from that user.
Bulk edit is not gonna assign completely new permissions when you bulk edit them. Because there is no way to account for difference in permissions of every user sp when you bulk edit and choose everyone you're basically assigning whatever permission you select in the modal. So when you don't select admin you're basically removing admin permissions from them.

[Radiicall]

What do you mean? This is working as intended?

When you bulk edit you're given a clean slate of permissions to choose and you're basically removing admin permission from them when you select everyone. Admins can edit other users permissions including other admins. When you edit a single admin if you untick admin permission that user will become a normal user. That's what happens when you bulk edit. You're removing the admin permission from that user.

Bulk edit is not gonna assign completely new permissions when you bulk edit them. Because there is no way to account for difference in permissions of every user sp when you bulk edit and choose everyone you're basically assigning whatever permission you select in the modal. So when you don't select admin you're basically removing admin permissions from them.

The reason i thought this wasnt intended was because when trying to edit the other admin without using the bulk tool then every permission is completely grayed out and i cant remove their admin

[fallenbagel]

@Radiicall nevermind you're right. I got confused, I apologise 😅

Yes an admin shouldn't be able to remove another admins perms