From 8977cac0f4921f5972162cd9060831c88c07e20f Mon Sep 17 00:00:00 2001 From: Thomas Rittson <31796059+eliykat@users.noreply.github.com> Date: Wed, 9 Apr 2025 10:33:31 +1000 Subject: [PATCH] Use switch expression instead of boolean MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: ✨ Audrey ✨ --- .../Requirements/ManageUsersRequirement.cs | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/src/Api/AdminConsole/Authorization/Requirements/ManageUsersRequirement.cs b/src/Api/AdminConsole/Authorization/Requirements/ManageUsersRequirement.cs index 3bfff00c48..169b356408 100644 --- a/src/Api/AdminConsole/Authorization/Requirements/ManageUsersRequirement.cs +++ b/src/Api/AdminConsole/Authorization/Requirements/ManageUsersRequirement.cs @@ -7,11 +7,14 @@ namespace Bit.Api.AdminConsole.Authorization.Requirements; public class ManageUsersRequirement : IOrganizationRequirement { - public async Task AuthorizeAsync( - CurrentContextOrganization? organizationClaims, - Func> isProviderUserForOrg) - => organizationClaims is - { Type: OrganizationUserType.Owner or OrganizationUserType.Admin } or - { Permissions.ManageUsers: true } - || await isProviderUserForOrg(); +public async Task AuthorizeAsync( + CurrentContextOrganization? organizationClaims, + Func> isProviderUserForOrg) + => organizationClaims switch + { + { Type: OrganizationUserType.Owner } => true, + { Type: OrganizationUserType.Admin } => true, + { Permissions.ManageUsers: true } => true, + _ => await isProviderUserForOrg() + }; }