diff --git a/src/plugins/workspace/server/saved_objects/workspace_saved_objects_client_wrapper.ts b/src/plugins/workspace/server/saved_objects/workspace_saved_objects_client_wrapper.ts index 5b2b45b353d7..dbde03f91150 100644 --- a/src/plugins/workspace/server/saved_objects/workspace_saved_objects_client_wrapper.ts +++ b/src/plugins/workspace/server/saved_objects/workspace_saved_objects_client_wrapper.ts @@ -73,16 +73,15 @@ export class WorkspaceSavedObjectsClientWrapper { if (!workspaceId) { return; } - if ( - !(await this.permissionControl.validate( - request, - { - type: WORKSPACE_TYPE, - id: workspaceId, - }, - this.formatWorkspacePermissionModeToStringArray(permissionMode) - )) - ) { + const validateResult = await this.permissionControl.validate( + request, + { + type: WORKSPACE_TYPE, + id: workspaceId, + }, + this.formatWorkspacePermissionModeToStringArray(permissionMode) + ); + if (!validateResult?.result) { throw generateWorkspacePermissionError(); } } @@ -96,16 +95,15 @@ export class WorkspaceSavedObjectsClientWrapper { return; } for (const workspaceId of workspaces) { - if ( - !(await this.permissionControl.validate( - request, - { - type: WORKSPACE_TYPE, - id: workspaceId, - }, - this.formatWorkspacePermissionModeToStringArray(permissionMode) - )) - ) { + const validateResult = await this.permissionControl.validate( + request, + { + type: WORKSPACE_TYPE, + id: workspaceId, + }, + this.formatWorkspacePermissionModeToStringArray(permissionMode) + ); + if (!validateResult?.result) { throw generateWorkspacePermissionError(); } } @@ -121,16 +119,15 @@ export class WorkspaceSavedObjectsClientWrapper { } let permitted = false; for (const workspaceId of workspaces) { - if ( - await this.permissionControl.validate( - request, - { - type: WORKSPACE_TYPE, - id: workspaceId, - }, - this.formatWorkspacePermissionModeToStringArray(permissionMode) - ) - ) { + const validateResult = await this.permissionControl.validate( + request, + { + type: WORKSPACE_TYPE, + id: workspaceId, + }, + this.formatWorkspacePermissionModeToStringArray(permissionMode) + ); + if (validateResult?.result) { permitted = true; break; }